Skip to content

Closes #332 - Fix scopes section#334

Merged
iamseanodentity merged 3 commits into
openid:mainfrom
jischr:main
Jun 2, 2026
Merged

Closes #332 - Fix scopes section#334
iamseanodentity merged 3 commits into
openid:mainfrom
jischr:main

Conversation

@jischr

@jischr jischr commented May 26, 2026

Copy link
Copy Markdown
Contributor

Addressing Issue #332 by fixing the wording.

@jischr jischr requested a review from a team as a code owner May 26, 2026 15:35

@appsdesh appsdesh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The new wording makes the scores mutually exclusive. The managed scope should also allow all CRUD operations.

@jischr

jischr commented May 27, 2026

Copy link
Copy Markdown
Contributor Author

@appsdesh thanks for the comment. can you take a look at the updated wording and let me know what you think?

@jischr jischr requested a review from appsdesh May 27, 2026 15:38
@appsdesh

appsdesh commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

@appsdesh thanks for the comment. can you take a look at the updated wording and let me know what you think?

LGTM! thanks

@iamseanodentity iamseanodentity left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ship it

@iamseanodentity iamseanodentity merged commit da0634f into openid:main Jun 2, 2026
2 checks passed
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.

4 participants