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

[7.x] Add lineItemPriceHook hook #1157

Merged
merged 5 commits into from
Sep 18, 2024
Merged

[7.x] Add lineItemPriceHook hook #1157

merged 5 commits into from
Sep 18, 2024

Conversation

SteJW
Copy link
Contributor

@SteJW SteJW commented Sep 18, 2024

I made the change for the lineItem hook as talked about here: #1156

Please have a look, when you have time.

@duncanmcclean duncanmcclean changed the title Created price hook from a lineItem perspective [7.x] Add lineItemPriceHook hook Sep 18, 2024
Copy link
Owner

@duncanmcclean duncanmcclean left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

I've removed the documentation for the new hook, since I'm going to be changing how these hooks work in the next version anyway.

@duncanmcclean duncanmcclean merged commit 5f71d13 into duncanmcclean:7.x Sep 18, 2024
6 checks passed
Copy link

Released as part of v7.5.0.

@SteJW
Copy link
Contributor Author

SteJW commented Sep 18, 2024

Awesome! Thanks for releasing.

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.

2 participants