Skip to content

BUG: User Management #11274#11275

Merged
offtherailz merged 2 commits intogeosolutions-it:masterfrom
rowheat02:bug/usermangement
Jul 3, 2025
Merged

BUG: User Management #11274#11275
offtherailz merged 2 commits intogeosolutions-it:masterfrom
rowheat02:bug/usermangement

Conversation

@rowheat02
Copy link
Copy Markdown
Contributor

@rowheat02 rowheat02 commented Jul 3, 2025

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:

fixes #11274

What is the current behavior?

#11274

What is the new behavior?

  1. Page will not crash while logging out when on the User management page
  2. User details even attributes are misssing.
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

@rowheat02 rowheat02 added this to the 2025.01.01 milestone Jul 3, 2025
@rowheat02 rowheat02 self-assigned this Jul 3, 2025
@rowheat02 rowheat02 added the bug label Jul 3, 2025
@tdipisa tdipisa added the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Jul 3, 2025
@offtherailz offtherailz merged commit 3985f9e into geosolutions-it:master Jul 3, 2025
6 checks passed
@offtherailz
Copy link
Copy Markdown
Member

offtherailz commented Jul 3, 2025

@ElenaGallo, could you please test this on DEV ? Thank you

@tdipisa
Copy link
Copy Markdown
Member

tdipisa commented Jul 3, 2025

@ElenaGallo, could you please test this on DEV ? Thank you

@offtherailz from my understanding this is fixing a behavior with default users. Correct if I'm wrong. If so, how to test this in DEV?

@offtherailz
Copy link
Copy Markdown
Member

offtherailz commented Jul 3, 2025

Right, @ElenaGallo it can be tested running npm start. Sorry, I have a pre-defined anwer for this :|
Thank you @tdipisa

@ElenaGallo
Copy link
Copy Markdown
Contributor

ElenaGallo commented Jul 3, 2025

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

rowheat02 added a commit to rowheat02/MapStore2 that referenced this pull request Jul 3, 2025
@rowheat02
BUG: fixes geosolutions-it#11274 fix user info window (geosolutions-i…
6198d34 
…t#11275)
@rowheat02 rowheat02 mentioned this pull request Jul 3, 2025
Merged
tdipisa pushed a commit that referenced this pull request Jul 3, 2025
@rowheat02
BUG: fixes #11274 fix user info window (#11275) (#11277)
449d2b1
@tdipisa tdipisa removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Jul 3, 2025
MV88 pushed a commit that referenced this pull request Sep 11, 2025
@rowheat02 @MV88
BUG: fixes #11274 fix user info window (#11275)
b00a9f1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@offtherailz offtherailz offtherailz approved these changes

Assignees

@rowheat02 rowheat02

Labels

bug

Projects

None yet

Milestone

2025.01.01

Development

Successfully merging this pull request may close these issues.

BUG: User Management

4 participants

@rowheat02 @offtherailz @tdipisa @ElenaGallo