Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate reimbursement hooks #3541

Conversation

spaghetticode
Copy link
Member

The introduction of dedicated events for succesful and failed reimbursement made the existing hooks implemented as class attributes obsolete.
They are now deprecated, waiting for a major release before their complete removal.

Checklist:

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have updated Guides and README accordingly to this change (if needed)
  • I have added tests to cover this change (if needed)
  • I have attached screenshots to this PR for visual changes (if needed)

@spaghetticode spaghetticode self-assigned this Mar 6, 2020
The introduction of dedicated events for succesful and failed reimbursement
made the existing hooks implemented as class attributes obsolete. They are
now deprecated, waiting for a major release before their complete removal.
@spaghetticode spaghetticode force-pushed the spaghetticode/deprecate-reimbursement-hooks branch from 96dfef8 to 90bf905 Compare March 6, 2020 15:03
Copy link
Member

@kennyadsl kennyadsl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes a lot of sense, we should do the same for order hooks!

@aldesantis aldesantis merged commit 16dc131 into solidusio:master Mar 26, 2020
@aldesantis aldesantis deleted the spaghetticode/deprecate-reimbursement-hooks branch March 26, 2020 07:43
@aldesantis
Copy link
Member

Awesome @spaghetticode, thank you!

@spaghetticode spaghetticode changed the title Depreacate reimbursement hooks Deprecate reimbursement hooks Jul 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants