Skip to content

Add OpenAPI scopes - #42888

Merged
nickvergessen merged 2 commits into
masterfrom
techdebt/noid/openapi-scopes
Jan 19, 2024
Merged

Add OpenAPI scopes#42888
nickvergessen merged 2 commits into
masterfrom
techdebt/noid/openapi-scopes

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

@provokateurinprovokateurin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The appropriate scopes should be set before this is merged

@nickvergessen
nickvergessen marked this pull request as draft January 17, 2024 15:21
@nickvergessen
nickvergessenforce-pushed the techdebt/noid/openapi-scopes branch 2 times, most recently from 7365a7d to f8d9679CompareJanuary 17, 2024 15:34
@provokateurin

Copy link
Copy Markdown
Member

This should also be backported to 28

Signed-off-by: provokateurin <kate@provokateurin.de>
Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin
provokateurinforce-pushed the techdebt/noid/openapi-scopes branch from a3495c8 to c5808d0CompareJanuary 18, 2024 15:43
@provokateurinprovokateurin changed the title fix(openapi): Regenerate OpenAPIAdd OpenAPI scopesJan 18, 2024
@provokateurin
provokateurin changed the base branch from master to chore/deps/update-openapi-extractorJanuary 18, 2024 15:44
@provokateurin

Copy link
Copy Markdown
Member

/skjnldsv-backport to stable28

@provokateurin
provokateurin marked this pull request as ready for review January 18, 2024 15:45
@provokateurinprovokateurin added this to the Nextcloud 29 milestone Jan 18, 2024
Base automatically changed from chore/deps/update-openapi-extractor to masterJanuary 18, 2024 22:31
@nickvergessen
nickvergessen merged commit 6c8b1fa into masterJan 19, 2024
@nickvergessen
nickvergessen deleted the techdebt/noid/openapi-scopes branch January 19, 2024 08:03
@backportbotbackportbotBot mentioned this pull request Jan 19, 2024
@skjnldsv

Copy link
Copy Markdown
Member

/skjnldsv-backport to stable28

(Testing)

@skjnldsv

This comment was marked as resolved.

@skjnldsv

This comment was marked as resolved.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsenhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@nickvergessen@provokateurin@skjnldsv@bigcat88