Skip to content

[Backport 2025.01.xx] Fix #11459 unload resources at logout (#11480)#11491

Merged
allyoucanmap merged 1 commit intogeosolutions-it:2025.01.xxfrom
allyoucanmap:issue_11459_2025.01.xx
Sep 17, 2025
Merged

[Backport 2025.01.xx] Fix #11459 unload resources at logout (#11480)#11491
allyoucanmap merged 1 commit intogeosolutions-it:2025.01.xxfrom
allyoucanmap:issue_11459_2025.01.xx

Conversation

@allyoucanmap
Copy link
Copy Markdown
Contributor

fix issue #11459

Define new actions, reducers, epic in the plugin ResourcesCatalog.
around the action LOGOUT, introduce new action UNLOAD_RESOURCES triggered to clear the state section that refers to the selected resource for unlogged users, clear this state sections:
resources.initialSelectedResource
resources.selectedResource
This new feature fixes some issues related to inconsistency state for non-logged in users.

Includes also a patch that hides the edit/save button in the info panel for non-logged users (this is based on the user's logged condition because the canEdit field is not enough to hide the button, the resource canCopy field for public resources override the canEdit condition (see details in the linked issues)
(previous exposed here: #11459 (comment))

  • added new epic unloadResources in ResourcesCatalog
  • added unit test for resources epic
  • hide save button in infopanel for no-logged users

Description

Please check if the PR fulfills these requirements

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

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

Issue

What is the current behavior?
#11459

What is the new behavior?

Hides the correct items after on logout.

pixes.mp4

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

---------

Co-authored-by: allyoucanmap <stefano.bovio@geosolutionsgroup.com>
@allyoucanmap allyoucanmap added this to the 2025.01.02 milestone Sep 17, 2025
@allyoucanmap allyoucanmap marked this pull request as ready for review September 17, 2025 10:46
@allyoucanmap allyoucanmap linked an issue Sep 17, 2025 that may be closed by this pull request
1 task
@allyoucanmap allyoucanmap merged commit 37183a4 into geosolutions-it:2025.01.xx Sep 17, 2025
9 of 10 checks passed
@allyoucanmap
Copy link
Copy Markdown
Contributor Author

@ElenaGallo please test this fix on QA, thanks

@stefanocudini stefanocudini mentioned this pull request Sep 22, 2025
12 tasks
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.

Issue on top bar and Permission panel for anonymous user and normal user

3 participants