-
Notifications
You must be signed in to change notification settings - Fork 57
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 3.2.1 #111
Release 3.2.1 #111
Conversation
Hlavtox
commented
Sep 25, 2024
Questions | Answers |
---|---|
Description? | Release 3.2.1 |
Type? | |
BC breaks? | |
Deprecations? | |
Fixed ticket? | |
Sponsor company | |
How to test? |
Merging as it was tested here #110 and there was only one change. |
PR merged, well done! Message to @PrestaShop/committers: do not forget to milestone it before the merge. |
hello @ShaiMagal it's QA decision to make, not yours |
@kpodemski I discussed this with @florine2623 and she tested the PR few hours ago. |
Am I wrong here? |
Hi @Hlavtox In that case it's all clear, it should be written in the merge message or @florine2623 could put QA Approved label |
@kpodemski OK, I will make sure to note it for the police next time 🤣 |