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

Disable 'verify if module is latest' for shops under 1.7.0.0 for 4.14.x and bump version to 4.14.3 #626

Merged
merged 4 commits into from Sep 13, 2023

Conversation

matks
Copy link
Contributor

@matks matks commented Sep 8, 2023

Questions Answers
Description? Disable 'verify if module is latest' for shops under 1.7.0.0 for 4.14.x and bump version to 4.14.3
Type? improvement
BC breaks? no
Deprecations? no
Fixed ticket?
Sponsor company
How to test? See below

Context

The upgrade module has a feature where it checks whether there is a more recent version of itself and if it does, it tells the user "please use the latest version, you are using an outdated version".

Capture d’écran 2023-09-08 à 23 09 45

Currently if someone installs the module 4.14.2 on a shop, the module will detect there is a 4.16.3 version and says to user "please don't use me! use the latest 4.16.3 version instead!"

Consequently this PR disables this check, this was a suggestion from @Hlavtox.

How to test

Create a test autoupgrade release from this PR. Install this test module

  • on a shop 1.6.1.24 : you do NOT see the warning Your current version of the module is out of date. Update now (because this PR disables the check on versions under 1.7.0.0)
  • on a shop 1.7.8.10 : you see the warning Your current version of the module is out of date. Update now

Important

Did you notice target branch is 4.14.x? So if you rebase the PR against target branch with git prc be careful to select the right branch

@matks
Copy link
Contributor Author

matks commented Sep 8, 2023

bd9fc65
As the branch 4.14.x is only for updates 1.6 to 1.7 I set CI to test againt latest 1.7.x version

@Hlavtox
Copy link
Contributor

Hlavtox commented Sep 9, 2023

@matks It should not perform this check for versions lower than 1.7, not lower than 8.

For people on 1.7, we want them to upgrade to 4.16. :)

@matks
Copy link
Contributor Author

matks commented Sep 11, 2023

True. I guess sending this PR at 22pm on a Friday evening was not the best way to obtain a good result. I will submit the changes.

@matks
Copy link
Contributor Author

matks commented Sep 12, 2023

@Hlavtox Fixed in 1a92a6b

@matks matks changed the title Disable 'verify if module is latest' for shops under 8.0.0 for 4.14.x and bump version to 4.14.3 Disable 'verify if module is latest' for shops under 1.7.0.0 for 4.14.x and bump version to 4.14.3 Sep 12, 2023
@matks
Copy link
Contributor Author

matks commented Sep 12, 2023

For your information, I discussed with @Robin-Fischer-PS and @MatShir and this is what we suggest for release 4.14.3:

  • this PR test is OK, no need for further release testing as the only difference between 4.14.2 and 4.14.3 will be this PR
  • upgrading from 1.6.x to 1.7.x shops is not supported anymore so this 4.14.3 release will be out of scope
  • this means that 4.14.3 will be published and available for anyone to use but we make no warranties, express or implied
  • basically the release will be provided "as is", without warranty

Copy link

@aniszr aniszr left a comment

Choose a reason for hiding this comment

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

Hello @matks

I tested your PR with two PS versions as mentioned :

  • PS 1.6.1.24
    The checks on versions of Autoupgrade module are disabled ✔️

See attached screenshot:

image

image

  • PS 1.7.8.10
    The checks on versions of Autoupgrade module are enabled ✔️

See attached screenshot:
image

image

Qa ✔️

Thank you!

@aniszr aniszr self-assigned this Sep 12, 2023
@MatShir
Copy link

MatShir commented Sep 12, 2023

Hi @aniszr,
The check is inside the module (not in the module list). When you want to upgrade the shop, the module check if there is a new version available.

@aniszr
Copy link

aniszr commented Sep 12, 2023

Hello @MatShir

Thanks for your feedback !

See attachet screenshot for PS 1.7.8.10

image

Thank you!

@MatShir
Copy link

MatShir commented Sep 12, 2023

👌 sweet ! thanks @aniszr we can merge and release the version !

@matks
Copy link
Contributor Author

matks commented Sep 13, 2023

Thank you @MatShir @aniszr and @Hlavtox

@matks matks merged commit d8df0f3 into PrestaShop:4.14.x Sep 13, 2023
17 checks passed
@matks matks deleted the 4.14.3 branch September 13, 2023 07:23
@matks matks added this to the 4.14.3 milestone Sep 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
5 participants