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

Adapt ps_crossselling for multiples versions of PrestaShop #44

Merged
merged 3 commits into from Feb 6, 2023

Conversation

Progi1984
Copy link
Contributor

Questions Answers
Description? Adapt ps_crossselling for multiples versions of PrestaShop
Type? refacto
BC breaks? no
Deprecations? no
Fixed ticket? Relative to PrestaShop/PrestaShop#28988
How to test? Try this module in version < 1.7.4.0 and in recent version (8.0.0)

@Progi1984 Progi1984 added this to the 2.0.2 milestone Feb 3, 2023
@Progi1984 Progi1984 requested a review from a team February 3, 2023 15:08
@Progi1984 Progi1984 added the waiting for QA Status: Waiting for QA feedback label Feb 6, 2023
@HanaRebaiQA HanaRebaiQA self-assigned this Feb 6, 2023
Copy link

@HanaRebaiQA HanaRebaiQA left a comment

Choose a reason for hiding this comment

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

Hello @Progi1984

I have checked your PR.

Check on PS1733 : OK
Check on PS8.0.1 : OK
Check with multistore: OK
Check with different php versions (php7.1 and php8.1) : OK

So, it's QA ✔️

Thank You!

@HanaRebaiQA HanaRebaiQA added QA ✔️ and removed waiting for QA Status: Waiting for QA feedback labels Feb 6, 2023
@Hlavtox Hlavtox merged commit ceeae70 into PrestaShop:dev Feb 6, 2023
@Hlavtox
Copy link
Contributor

Hlavtox commented Feb 6, 2023

Thanks everyone!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants