Skip to content

PB-1318 : give a better error message on deleted KMLs#1276

Merged
pakb merged 1 commit intodevelopfrom
feat-PB-1318-better-error-message-on-deleted-kml
Mar 21, 2025
Merged

PB-1318 : give a better error message on deleted KMLs#1276
pakb merged 1 commit intodevelopfrom
feat-PB-1318-better-error-message-on-deleted-kml

Conversation

@pakb
Copy link
Contributor

@pakb pakb commented Mar 21, 2025

Trying to better convey why the KML couldn't be loaded, to help reduce questions on our helpdesk team about it.

Test link

Trying to better convey why the KML couldn't be loaded, to help reduce questions on our helpdesk team about it.
@cypress
Copy link

cypress bot commented Mar 21, 2025

web-mapviewer    Run #4838

Run Properties:  status check passed Passed #4838  •  git commit c78ea8f558: Merge pull request #1276 from geoadmin/feat-PB-1318-better-error-message-on-dele...
Project web-mapviewer
Branch Review develop
Run status status check passed Passed #4838
Run duration 01m 39s
Commit git commit c78ea8f558: Merge pull request #1276 from geoadmin/feat-PB-1318-better-error-message-on-dele...
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 1
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 48
View all changes introduced in this branch ↗︎

@pakb pakb requested a review from ltkum March 21, 2025 08:47
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.

Seems explicit to me :)

@pakb pakb merged commit c78ea8f into develop Mar 21, 2025
6 checks passed
@pakb pakb deleted the feat-PB-1318-better-error-message-on-deleted-kml branch March 21, 2025 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants