8000 Improve web archive retrieval by M0rgan01 · Pull Request #1304 · PrestaShop/autoupgrade · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Improve web archive retrieval #1304

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

Merged
merged 1 commit into from
Apr 15, 2025

Conversation

M0rgan01
Copy link
Contributor
@M0rgan01 M0rgan01 commented Apr 9, 2025
Questions Answers
Description? We no longer display versions lower than the current one in the archives, we no longer display versions that do not have the corresponding xml or zip in the archive
Type? improvement
BC breaks? no
Deprecations? no
Fixed ticket? -
Sponsor company -
How to test? use the update:check-new-version CLI command and the local channel on the web ui.

@github-project-automation github-project-automation bot moved this to Ready for review in PR Dashboard Apr 9, 2025
@M0rgan01 M0rgan01 added this to the 7.1.0 milestone Apr 9, 2025
@M0rgan01 M0rgan01 force-pushed the improve-web-archive branch 3 times, most recently from 6be5f85 to 7af2e9d Compare April 9, 2025 15:27
self::TYPE_ZIP => $zipList,
self::TYPE_XML => $xmlList,
];
}
Copy link
Member

Choose a reason for hiding this comment

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

Can you please add a unit test for this new function?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will do it, of course :)

@M0rgan01 M0rgan01 force-pushed the improve-web-archive branch from 7af2e9d to 283a4ee Compare April 10, 2025 07:53
Copy link

@AureRita AureRita self-assigned this Apr 15, 2025
Copy link
Contributor
@AureRita AureRita left a comment

Choose a reason for hiding this comment

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

Hi @M0rgan01

Thank you for your PR, I tested it and it seems to works as you can see :

image

Capture.video.du.2025-04-15.14-19-40.mp4

Tested from :
8.0.5 to 8.1.2
8.1.2 to 8.2.1
8.0.5 to 8.2.1

Because the PR seems to works as expected, It's QA ✔️

Thank you

@Quetzacoalt91 Quetzacoalt91 merged commit da58dce into PrestaShop:dev Apr 15, 2025
51 of 52 checks passed
@github-project-automation github-project-automation bot moved this from Ready for review to Merged in PR Dashboard Apr 15, 2025
@Quetzacoalt91 Quetzacoalt91 deleted the improve-web-archive branch April 15, 2025 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Type: Improvement QA ✔️
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants
0