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 6.0 #180

Merged
merged 54 commits into from
Aug 23, 2023
Merged

Release 6.0 #180

merged 54 commits into from
Aug 23, 2023

Conversation

M0rgan01
Copy link
Contributor

@M0rgan01 M0rgan01 commented Aug 4, 2023

Questions Answers
Description? Release 6.0.0
Type? improvement
BC breaks? no
Deprecations? no
Fixed ticket? -
How to test? Please perform a full testing of the module as this is a new major release.

leemyong pakvn and others added 30 commits March 15, 2023 13:15
…ination

Stop using jquery simple pagination
…3_phpdevtools-v3.4tov3.16

Update github actions v1v2 to v3, phpdevtools v3.4 to v3.16
…nts.css

Remove an unused css file from version 4.0.0
Update build-release.yml using Organization config
…7_moduleVersionTo600

Bump min compatibility to PHP 7.1.3 and PS 1.7.7, module version to 6.0.0
@M0rgan01
Copy link
Contributor Author

Hello @kpodemski @leemyongpakvn , this release is blocking for the issue PrestaShop/PrestaShop#33459. We need to unblock the review to make progress on Doctrine. Do you think the PR #177 can be merged quickly? If not, we can consider proposing another release that includes it.

@leemyongpakvn
Copy link
Contributor

leemyongpakvn commented Aug 21, 2023

@M0rgan01 PR 177 was reviewed by @matks and @kpodemski. I answered all questions and applied some suggestions so it's their turn now. And I'm pretty sure that matks don't like bumping to v7.0.0 right after v6.0.0 much ;)

@M0rgan01 M0rgan01 dismissed kpodemski’s stale review August 23, 2023 13:04

PR #177 is now in release

@M0rgan01 M0rgan01 closed this Aug 23, 2023
@M0rgan01 M0rgan01 reopened this Aug 23, 2023
Copy link
Contributor

@matks matks left a comment

Choose a reason for hiding this comment

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

Let's go 💪

@hibatallahAouadni hibatallahAouadni self-assigned this Aug 23, 2023
Copy link

@hibatallahAouadni hibatallahAouadni left a comment

Choose a reason for hiding this comment

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

Hello @M0rgan01

I checked every functionalities in the module on 1.7.8.10 and 8.1.1 and it works 🎉
LGTM, QA ✅

Thanks!

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

Successfully merging this pull request may close these issues.