Give user alert to confirm browser refresh #81
Labels
Good First Issue
Good for newcomers
[Type] Enhancement
New feature or request
[Type] UI / UX / User Experience
Comments
artemiomorales
added
UI/UX
[Type] Enhancement
|
Hey @hlashbrooke , |
|
@akansjain there you go! |
Closed
This was referenced Apr 22, 2025
bgrgicak
added a commit
that referenced
this issue
May 7, 2025
#81) ## Motivation for the change, related issues Before this PR the `setSiteLanguage` step automatically tried to resolve the WordPress core translation version, but some combinations of versions and languages don't have translation packages. This PR uses the WordPress.org translations API to get the correct translation URL for the given WordPress version and language. If a translation package doesn't exist, the step will still fail. ## Implementation details Instead of inferring the WordPress version, `setSiteLanguage` now uses the `api.wordpress.org/translations/core/1.0` endpoint to get the correct translation URL for the given WordPress version and language. This approach ensures we return an existing translation URL whenever possible and moves the WP version resolution to the WordPress.org translations API. ## Testing Instructions (or ideally a Blueprint) - CI
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
To prevent accidental page refreshes that will lose the user's data and progress, it would be great to have a basic browser alert that confirms if the user does want to refresh before it actually happens.
The text was updated successfully, but these errors were encountered: