Skip to content

PB-1872: term of use in english iframe menu returns a 404#1388

Merged
ltkum merged 2 commits intodevelopfrom
fix-PB-1872-change-iframe-term-of-use-link
Aug 5, 2025
Merged

PB-1872: term of use in english iframe menu returns a 404#1388
ltkum merged 2 commits intodevelopfrom
fix-PB-1872-change-iframe-term-of-use-link

Conversation

@ltkum
Copy link
Contributor

@ltkum ltkum commented Aug 5, 2025

Issue: the terms of use link in the iframe integration menu, in english, returns a 404 since the page doesn't exist

Fix: We link to the german one.

Test link

@github-actions github-actions bot added the bug label Aug 5, 2025
@ltkum ltkum requested a review from pakb August 5, 2025 09:20
@cypress
Copy link

cypress bot commented Aug 5, 2025

web-mapviewer    Run #5545

Run Properties:  status check failed Failed #5545  •  git commit 9bc58b79cd: PB-1872: term of use in english iframe menu returns a 404
Project web-mapviewer
Branch Review fix-PB-1872-change-iframe-term-of-use-link
Run status status check failed Failed #5545
Run duration 06m 04s
Commit git commit 9bc58b79cd: PB-1872: term of use in english iframe menu returns a 404
Committer Martin Künzi
View all properties for this run ↗︎

Test results
Tests that failed  Failures 1
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 253
View all changes introduced in this branch ↗︎

Tests for review

Failed  tests/cypress/tests-e2e/importToolFile.cy.js • 1 failed test • e2e/chrome/mobile

View Output

Test Artifacts
The Import File Tool > Import GPX file Test Replay Screenshots

Issue: the terms of use link in the iframe integration menu, in english, returns a 404 since the page doesn't exist

Fix: We link to the german one.
@pakb pakb force-pushed the fix-PB-1872-change-iframe-term-of-use-link branch from 9bc58b7 to aee8e7a Compare August 5, 2025 10:57
thanks to @bariod for finding them :)
@ltkum ltkum mentioned this pull request Aug 5, 2025
@ltkum ltkum merged commit 0eda963 into develop Aug 5, 2025
4 checks passed
@ltkum ltkum deleted the fix-PB-1872-change-iframe-term-of-use-link branch August 5, 2025 12:05
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