Skip to content

Conversation

@rowheat02
Copy link
Contributor

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

Done as part of this issue #11374. Reported here #11375 (comment)

What is the current behavior?

When entering invalid credentials, user was not able to re-login

What is the new behavior?

The user will see the Relogin button when invalid credentials is entered.
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.02.00 milestone Oct 24, 2025
@rowheat02 rowheat02 linked an issue Oct 24, 2025 that may be closed by this pull request
1 task
@tdipisa tdipisa requested a review from allyoucanmap October 24, 2025 13:15
@tdipisa tdipisa added the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Nov 4, 2025
Copy link
Contributor

@allyoucanmap allyoucanmap left a comment

Choose a reason for hiding this comment

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

@rowheat02 it will be better to show directly credentials form with the error message instead of showing a button to reload the inputs, e.g. :

image

@rowheat02
Copy link
Contributor Author

rowheat02 commented Nov 4, 2025

Now, when the user enters invalid credentials, the credential form and error message will be shown directly.
image

  • Cancel Button will not be shown on the credential from when wrong credentials are entered already.

@allyoucanmap allyoucanmap merged commit 5b493fb into geosolutions-it:master Nov 4, 2025
6 checks passed
@allyoucanmap
Copy link
Contributor

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

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

Labels

BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve the Street Smart plugin view for 3D maps visualization

3 participants