-
Notifications
You must be signed in to change notification settings - Fork 119
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
Rework of module updates : Introducing Sources and their Providers #850
Conversation
b673c6b
to
4051ac4
Compare
efaf05e
to
4795c6c
Compare
d7acfd8
to
25af9df
Compare
…ta from the Marketplace API
…dates from the prestashop release
b6eb0a6
to
16b9800
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your PR, I tested it and it seems to works as you can see :
Tested from :
1.7.8.9 to 8.1.7
8.0.5 to 8.1.7
8.1.5 to 8.1.7
8.1.7 to 9.0.0
8.1.5 to 9.0.0
8.0.5 to 9.0.0
All issues are tested on it and corrected
Because the PR seems to works as expected, It's QA ✔️
Thank you
PR merged, well done! Message to @PrestaShop/committers: do not forget to milestone it before the merge. |
Current state of logs
No updates
No migration files to run
With migration files
Work in progress:
Provider of sources from Distribution APIComing in another PR as new featureRefacto:
AbstractSourceProvider