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

Release version 4.16.2 #610

Merged
merged 27 commits into from
Aug 2, 2023
Merged

Release version 4.16.2 #610

merged 27 commits into from
Aug 2, 2023

Conversation

Hlavtox
Copy link
Contributor

@Hlavtox Hlavtox commented Aug 2, 2023

Questions Answers
Description? Release version 4.16.2
Type?
BC breaks?
Deprecations?
Fixed ticket?
Sponsor company
How to test? Make one last upgrade from 1.7.6 to 8.1 and check issues below are fixed. The rest of changes is just wording and some safety precautions.

What to check

  • Can you open new product page after upgrade?
  • Can you successfully save new redirect types on product page?

Change list

#604: Add empty values for MPN fields @Hlavtox
#599: Add key check by @Hlavtox
#609: Add force removing system for some files by @boherm
#608: Avoid deleting a popular payment module "stripejs" by @nexustotalsolutions
#607: Add more info to autoupgrade process by @Hlavtox
#601: Fix product redirect_type by @Hlavtox
#606: Clean opcache when cleaning cache by @Hlavtox
#603: Add upgrade helptext by @Hlavtox
#602: Move script to proper location by @Hlavtox

Hlavtox and others added 24 commits July 30, 2023 13:43
Move script to proper location
Clean opcache when cleaning cache
Hello,

I'm the contributor for the module "Stripejs" payment module and I've got lots of complains that the module does gets deleted while upgrading Prestashop. So I opened an issue to fix it but nothing was done after long time.
PrestaShop/PrestaShop#27561
So I decided to do it myself.

The code which I removed from this autupgrade module, is responsible to delete my module during the upgrade. 

Please consider this change at high priority because thousands of customers are constantly facing this serious issue with  my module.
Best Regards,
Sitender
Deleted whole foreach{} for clear code.
Fix product redirect_type
Add more info to autoupgrade process
Avoid deleting a popular payment module "stripejs"
Add empty values for MPN fields
@Hlavtox Hlavtox added this to the 4.16.2 milestone Aug 2, 2023
boherm
boherm previously approved these changes Aug 2, 2023
kpodemski
kpodemski previously approved these changes Aug 2, 2023
matks
matks previously approved these changes Aug 2, 2023
@florine2623 florine2623 self-assigned this Aug 2, 2023
@kpodemski kpodemski dismissed stale reviews from matks, boherm, and themself via 830a720 August 2, 2023 14:52
Copy link

@florine2623 florine2623 left a comment

Choose a reason for hiding this comment

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

Hello @Hlavtox ,

Tested the release of autoupgrade with @hibatallahAouadni .

Bump to version 4.16.2 OK.

All upgrades went well.
1.7.8.9 -> 8.1.1 ✅
1.7.8.10 -> 8.1.1 ✅
8.0.4 -> 8.1.1 ✅
8.0.5 -> 8.1.1 ✅
8.1.0 -> 8.1.1 ✅

For all the upgrades, checked the all the PR that were concerned.
Rollback is OK too ✅

It is QA ✅ !
Thanks!

@Hlavtox Hlavtox merged commit 5c49dcb into master Aug 2, 2023
52 checks passed
@Hlavtox
Copy link
Contributor Author

Hlavtox commented Aug 2, 2023

Thank you!!!!! @florine2623 & @hibatallahAouadni 🌷🌷🌷

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
8 participants