Skip to content

Street smart openid support#10817

Merged
offtherailz merged 10 commits intoc027-2024.01.xxfrom
streetsmart_openid
Feb 17, 2025
Merged

Street smart openid support#10817
offtherailz merged 10 commits intoc027-2024.01.xxfrom
streetsmart_openid

Conversation

@offtherailz
Copy link
Copy Markdown
Member

Description

This PR provides for a specific branch used for the current integration, the support for streetsmart OIDC.
This will be contributed to master too.

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?

Support OIDC in street smart API

What is the new behavior?

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

@offtherailz offtherailz self-assigned this Feb 17, 2025
@tdipisa tdipisa requested a review from MV88 February 17, 2025 15:00
Copy link
Copy Markdown
Contributor

@MV88 MV88 left a comment

Choose a reason for hiding this comment

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

apart from logout string in italian I think this pr is good as it is

@offtherailz
Copy link
Copy Markdown
Member Author

Yes, Your observation is correct.
I should use "logout" anyway because is used in italian, and "disconnetti" may be misleading to in this case.

@offtherailz offtherailz merged commit 55ab964 into c027-2024.01.xx Feb 17, 2025
offtherailz added a commit to offtherailz/MapStore2 that referenced this pull request Feb 26, 2025
@offtherailz offtherailz deleted the streetsmart_openid branch November 10, 2025 08:43
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.

2 participants