-
Notifications
You must be signed in to change notification settings - Fork 22
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
add missing hooks when upgrading from old versions #55
Conversation
kpodemski
commented
Mar 8, 2023
Questions | Answers |
---|---|
Description? | To prevent issues with upgraded installations, we should hook this module during its upgrade. |
Type? | bug fix |
BC breaks? | no |
Deprecations? | no |
Fixed ticket? | Related to the contactform fix |
How to test? | You'd need to have the version below < 3.3.1 installed to see that hooks are registered during an upgrade of the module. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @kpodemski ,
I've followed these steps :
- Uninstall ps_contactinfo
v3.3.1
- Install ps_contactinfo
v3.3.0
- Install this PR
- Upgrade to ps_contactinfo
v3.3.2
- Check in FO :
The store information block should not be hook here, am I right ?
@florine2623 Hi, you can retest, I removed the extra hook. Krystian knows. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @Hlavtox ,
Tested and it works like a charm !
QA ✅
Thanks!
Thank @kpodemski ! |