Skip to content

New Release v1.57.3 - #patch#1380

Merged
pakb merged 17 commits intomasterfrom
develop
Jul 10, 2025
Merged

New Release v1.57.3 - #patch#1380
pakb merged 17 commits intomasterfrom
develop

Conversation

@pakb
Copy link
Contributor

@pakb pakb commented Jul 10, 2025

ismailsunni and others added 17 commits July 9, 2025 14:09
…lity

PB-1784: Use eye for KML description visibility and add tooltip for other buttons
and trying to keep the fix for Firefox print working

Network first means the Service-Worker will attempt to load the resource online regardless of the cache status, and if the network responds it will update the cache with the new answer.
We need a good e2e test coverage (with something like Playwright) to be confident enough to change the strategy back to StaleWhileRevalidate

Giving a full URL to the "Manifest generation function" is a bad idea, and the service worker isn't loading anymore, reverting to relative `index.html` path, and editing the API print regex to match a regex found in the VitePWA documentation

Adding a "BETA" badge to the feature, to have a better "expectation management" and signify that this could have some bugs still.
PB-1834: Layer "Notifications for maps an geodata" not loading
@github-actions github-actions bot changed the title #patch New Release v1.57.3 - #patch Jul 10, 2025
@pakb pakb requested a review from ismailsunni July 10, 2025 11:12
@cypress
Copy link

cypress bot commented Jul 10, 2025

web-mapviewer    Run #5524

Run Properties:  status check passed Passed #5524  •  git commit 274bc89ad9: Merge pull request #1378 from geoadmin/fix-PB-1834-wrong-call-to-log
Project web-mapviewer
Branch Review develop
Run status status check passed Passed #5524
Run duration 05m 30s
Commit git commit 274bc89ad9: Merge pull request #1378 from geoadmin/fix-PB-1834-wrong-call-to-log
Committer Pascal Barth
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 20
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 254
View all changes introduced in this branch ↗︎

@pakb pakb requested a review from ltkum July 10, 2025 11:23
Copy link
Contributor

@ltkum ltkum left a comment

Choose a reason for hiding this comment

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

LGTM

@pakb pakb merged commit ab04af1 into master Jul 10, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants