Skip to content

Removed authorization toggles #1913

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.

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

Merged
merged 6 commits into from
Jul 3, 2025

Conversation

ThomasMinkeRitense
Copy link
Contributor

Describe the changes

Link to the related Github issue:

Relevant comments:

Breaking changes

  • The contribution only contains changes that are not breaking.

Documentation

  • Release notes have been written for these changes.

Link to the pull request in the Valtimo documentation repository:

New features or changes that have been introduced have been documented.

  • Yes
  • Not applicable

Tests

Unit tests have been added that cover these changes

  • Yes
  • Not applicable

Integration tests have been added that cover these changes

  • Yes
  • Not applicable

Security

The Secure by Design principle has been applied to these changes

  • Yes
  • Not applicable

Added or changed REST API endpoints have authentication and authorization in place

  • Yes
  • Not applicable

Valtimo access control checks have been implemented

  • Yes
  • Not applicable

Dependencies

Newly added dependencies do not introduce known vulnerabilities/CVE's and are in line with the Valtimo license

  • Yes
  • Not applicable

@ThomasMinkeRitense ThomasMinkeRitense marked this pull request as ready for review July 2, 2025 19:35
@ThomasMinkeRitense ThomasMinkeRitense requested a review from a team as a code owner July 2, 2025 19:35
@ThomasMinkeRitense ThomasMinkeRitense merged commit 8a0d851 into next-major Jul 3, 2025
139 checks passed
@ThomasMinkeRitense ThomasMinkeRitense deleted the feat/removed-authorization-toggles branch July 3, 2025 08:24
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.

3 participants