Skip to content

Access Restrictions for Maps Created from Unadvertised Contexts #11748#11752

Merged
allyoucanmap merged 1 commit intogeosolutions-it:masterfrom
rowheat02:fix/brokencontextEnhancement
Dec 2, 2025
Merged

Access Restrictions for Maps Created from Unadvertised Contexts #11748#11752
allyoucanmap merged 1 commit intogeosolutions-it:masterfrom
rowheat02:fix/brokencontextEnhancement

Conversation

@rowheat02
Copy link
Copy Markdown
Contributor

Description

From the discussion and conclusion here, it was decided to make the non-advertised context's map clickable to open in the default viewer, along with showing a warning.

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Enhancement

#11748

What is the current behavior?

#11748
For a normal user, the user can not open a non-advertised context's map even if they have edit/view permission.

What is the new behavior?

The user can click and open a non-advertised context's map in the default viewer. Warning about permission will be as it is.
image

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • Yes, and I documented them in migration notes
  • No

Other useful information

The warning message has also been updated to not include: "This resource can't be open"

@rowheat02 rowheat02 linked an issue Dec 1, 2025 that may be closed by this pull request
1 task
@rowheat02 rowheat02 added the bug label Dec 1, 2025
@tdipisa tdipisa requested a review from allyoucanmap December 1, 2025 16:08
@tdipisa tdipisa added this to the 2025.02.00 milestone Dec 1, 2025
@allyoucanmap allyoucanmap merged commit 1b6d336 into geosolutions-it:master Dec 2, 2025
12 checks passed
github-actions bot pushed a commit that referenced this pull request Dec 2, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 2, 2025

@allyoucanmap
Copy link
Copy Markdown
Contributor

@ElenaGallo please test on dev and let us know if we can backport on 2025.02.xx, thanks

@ElenaGallo
Copy link
Copy Markdown
Contributor

Test passed, @rowheat02 please backport to 2025.02.xx. Thanks

@tdipisa
Copy link
Copy Markdown
Member

tdipisa commented Dec 2, 2025

backport already there @ElenaGallo @rowheat02 as we have the automation now. I'm going to review and merge

tdipisa pushed a commit that referenced this pull request Dec 2, 2025
…ntexts (#11752) (#11755)

(cherry picked from commit 1b6d336)

Co-authored-by: RowHeat <40065760+rowheat02@users.noreply.github.com>
allyoucanmap pushed a commit that referenced this pull request Dec 2, 2025
…ntexts (#11752) (#11755)

(cherry picked from commit 1b6d336)

Co-authored-by: RowHeat <40065760+rowheat02@users.noreply.github.com>
dsuren1 added a commit that referenced this pull request Dec 4, 2025
…request

* commit 'a0da62e93ec7c7f9049c3343e410d4bea1db33c1':
  Update User Guide - Itinerary plugin (#11768)
  Fix the dashboard save, legend dependency and maximize issue (#11715)
  Update User Guide - Isochrone plugin (#11769)
  #11525 - Improve Maps in Dashboards (#11625)
  Update User Guide - Cascading search filter (#11766)
  Update User Guide - Show the current time and Null values options on Chart Widget (#11764)
  Fix #11754 - Moved assemply to bin-war module (#11761)
  Update User Guide - 3D Tiles imagery layers (#11758)
  Update User Guide - Update Tags menu section (#11756)
  Fix #11748 Access Restrictions for Maps Created from Unadvertised Contexts (#11752)
mahesh-wor pushed a commit to mahesh-wor/MapStore2 that referenced this pull request Dec 17, 2025
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.

Access Restrictions for Maps Created from Unadvertised Contexts

4 participants