Skip to content

fix(openapi)!: oauth scopes with dashes in name#7853

Merged
soyuka merged 3 commits intoapi-platform:4.3from
thomaschaaf:patch-1
Mar 24, 2026
Merged

fix(openapi)!: oauth scopes with dashes in name#7853
soyuka merged 3 commits intoapi-platform:4.3from
thomaschaaf:patch-1

Conversation

@thomaschaaf
Copy link
Contributor

Q A
Branch main because of potential breaking change
License MIT

Scopes:
this-is-a-test is currently normalized to this_is_a_test.

@soyuka soyuka changed the title fix: Allow setting oauth scopes with dashes in name fix(openapi)!: oauth scopes with dashes in name Mar 21, 2026
@soyuka
Copy link
Member

soyuka commented Mar 21, 2026

I think we should move this to 4.3 anyways as its hard to circumvent if you need this no?

@thomaschaaf
Copy link
Contributor Author

Okay then I will create a new PR against 4.3. 🙂

@soyuka
Copy link
Member

soyuka commented Mar 22, 2026

just change the target of this PR and rebase :)

@thomaschaaf thomaschaaf changed the base branch from main to 4.3 March 22, 2026 09:21
@soyuka soyuka merged commit 2d6e474 into api-platform:4.3 Mar 24, 2026
149 of 151 checks passed
@soyuka
Copy link
Member

soyuka commented Mar 24, 2026

I don't think this will have much impact and I think its quite a mandatory patch therefore merging on 4.3 is nicer. Let me know if this is an issue in the future.

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