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.

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

Add scopes as required field of OAuth Flow Object #2673

Merged
merged 1 commit into from
Aug 19, 2021

Conversation

char0n
Copy link
Contributor

@char0n char0n commented Aug 10, 2021

This makes metaschema consistent with the 3.0.x spec.

Refs #2666

Copy link
Member

@karenetheridge karenetheridge left a comment

Choose a reason for hiding this comment

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

Please update schema.yaml as well.

@char0n char0n force-pushed the char0n/scopes-required-3-0 branch from e6b3e7a to cd79b6f Compare August 11, 2021 05:58
@char0n
Copy link
Contributor Author

char0n commented Aug 11, 2021

@karenetheridge schema.yaml is updated as well now.

@hkosova
Copy link
Contributor

hkosova commented Aug 11, 2021

Should the schema id also be updated to the current date?

-id: https://spec.openapis.org/oas/3.0/schema/2020-02-25
+id: https://spec.openapis.org/oas/3.0/schema/2021-08-10

This was done the last time when the 3.0 schema was updated:
https://github.com/OAI/OpenAPI-Specification/pull/2158/files

@webron
Copy link
Member

webron commented Aug 11, 2021

Update procedure is in the README 😃

@char0n
Copy link
Contributor Author

char0n commented Aug 11, 2021

Right, I'll amend the PR to reflect the README

@webron
Copy link
Member

webron commented Aug 11, 2021

That was also a nudge to @MikeRalphson as he likes automating things ;) You can keep the json version too, I don't see it as a problem.

This makes metaschema consistent with the 3.0.x spec.

Refs OAI#2666
@char0n char0n force-pushed the char0n/scopes-required-3-0 branch from cd79b6f to 35b8638 Compare August 12, 2021 06:34
@char0n
Copy link
Contributor Author

char0n commented Aug 12, 2021

Ok now it's done precisely according to the README ;]

@webron webron merged commit 4b1433d into OAI:main Aug 19, 2021
jdesrosiers added a commit to jdesrosiers/OpenAPI-Specification that referenced this pull request Sep 28, 2021
jdesrosiers added a commit to jdesrosiers/OpenAPI-Specification that referenced this pull request Sep 28, 2021
jdesrosiers added a commit to jdesrosiers/OpenAPI-Specification that referenced this pull request Sep 29, 2021
webron pushed a commit that referenced this pull request Oct 7, 2021
* Update 3.1 schema id for patch release

* Generate JSON for new version of 3.1 schemas

* Catchup v3.0 schema with #2158

* Catchup v3.0 schemas with #2673

* Catchup v3.0 schemas with #2706

* Update schema id for 3.0 release
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.

5 participants