Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: update the SecurityProviderType enum in platform-client #923

Merged
merged 1 commit into from
Mar 13, 2025
Merged

feat: update the SecurityProviderType enum in platform-client #923

merged 1 commit into from
Mar 13, 2025
+9 −1

Conversation

paulgerold
Copy link
Contributor

Update the SecurityProviderType enum to match the values of the SecurityProviderType in the security-cache-service

Acceptance Criteria

  • My changes are publicly available, documented, and deployed in production. (i.e. on Swagger)
  • JSDoc annotates each property added in the exported interfaces
  • The proposed changes are covered by unit tests
  • Commits containing breaking changes a properly identified as such
  • README.md is adjusted to reflect the proposed changes (if relevant)
  • My merge commit message will be conventional (See Conventional Commit)

Sorry, something went wrong.

@paulgerold
feat: update the securityprovidertype enum in platform-client

Verified

This commit was signed with the committer’s verified signature.
paulgerold Paul Gerold
SSH Key Fingerprint: NrJQ1+y/uzR/S0aJK/v2AAi9USRL7/2PK9Q2rHk1RGQ
Verified
Learn about vigilant mode
Loading
Loading status checks…
07e04ac
@paulgerold paulgerold marked this pull request as ready for review March 12, 2025 17:28
@paulgerold paulgerold requested review from a team as code owners March 12, 2025 17:28
@paulgerold paulgerold added this pull request to the merge queue Mar 13, 2025
Merged via the queue into master with commit c70bc52 Mar 13, 2025
6 checks passed
@paulgerold paulgerold deleted the ADUI-10538-update-the branch March 13, 2025 14:28
@github-actions GitHub Actions
Copy link

🎉 This PR is included in version 58.12.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

@GermainBergeron GermainBergeron

@toofff toofff

@FelixBlaisThon FelixBlaisThon

@alexprudhomme alexprudhomme

@olamothe olamothe

@fpbrault fpbrault

@mtheriault-git mtheriault-git

Assignees
No one assigned
Labels
released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@paulgerold @GermainBergeron @toofff @FelixBlaisThon @alexprudhomme