-
Notifications
You must be signed in to change notification settings - Fork 29
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
Update API-design-guidelines.md-with-wildcard-scopes-icm-agreement #221
Conversation
@AxelNennker and @jpengar Could you kindly check if the section "API-level scopes" reflects correctly the agreement made in camaraproject/IdentityAndConsentManagement#95 @PedroDiez and @bigludo7 : I have simply duplicated the content of #177 into the section 11.6.1 to ensure consistency. Could you kindly take a look? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM in advance. I've just added some minor suggestions.
Co-authored-by: Jesús Peña García-Oliva <jesus.penagarcia-oliva@telefonica.com>
Co-authored-by: Jesús Peña García-Oliva <jesus.penagarcia-oliva@telefonica.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co-authored-by: Rafal Artych <121048129+rartych@users.noreply.github.com>
Co-authored-by: Jesús Peña García-Oliva <jesus.penagarcia-oliva@telefonica.com>
Co-authored-by: Rafal Artych <121048129+rartych@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
What type of PR is this?
Add one of the following kinds:
What this PR does / why we need it:
This PR includes the following changes:
Documents the agreement in ICM on wildcard scopes discussed under the issue -(Scope at "API level" feedback from country market implementation IdentityAndConsentManagement#95) into the design guidelines to ensure completeness of the doc,
The explicit subscriptions related scope changes were documented in the PR update-design-doc-with-explicit-sub-scope-changes #177 . The scope naming is described in 2 sections in the design doc - 10.2 and 11.6.1. The earlier PR only updated section 10.2 (b). This PR adds the needed content also in section 11.6.1
Which issue(s) this PR fixes:
Fixes #184
Special notes for reviewers:
Changelog input
Additional documentation
This section can be blank.