Skip to content

Conversation

@glughi
Copy link
Contributor

@glughi glughi commented Nov 14, 2025

What this PR does

Fixes #7269
Replace PR #7270

Test me

Using, ie, https://terriamap.com/#emg202305, you can load some data from emg202305.json.
Without this fix, the sharing link from https://terriamap.com/#emg202305 will be something like
https://terriamap.com/#share=g-8feface8fefd6fe09ac81f22f21ec5d8
so you can't reload data from emg202305.json.
Instead, with this fix, the link will be something like
https://terriamap.com/#emg202305&share=g-8feface8fefd6fe09ac81f22f21ec5d8

Checklist

  • There are unit tests to verify my changes are correct or unit tests aren't applicable (if so, write quick reason why unit tests don't exist)
  • I've updated relevant documentation in doc/.
  • I've updated CHANGES.md with what I changed.
  • I've provided instructions in the PR description on how to test this PR.

@glughi glughi mentioned this pull request Nov 14, 2025
4 tasks
) {
const uri = new URI(document.baseURI).fragment("").search("");

const usefulSubs = new URL(document.URL).hash
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we find a more descriptive name for this variable?

@niktayv
Copy link
Collaborator

niktayv commented Nov 15, 2025

It would be helpful to describe how to test this feature.
Is there a minimal way to reproduce a failure?

@niktayv
Copy link
Collaborator

niktayv commented Nov 15, 2025

By the way, I inspected the CI/build failure that occurs at yarn gulp test-firefox.
Locally, this test passes, so I asked the core team to help resolve this issue.

@glughi
Copy link
Contributor Author

glughi commented Nov 17, 2025

Updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Share using initfile

2 participants