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.

Sign up for GitHub

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

Jump to bottom

Migrate all apps to generated OpenAPI specs #537

Open
1 of 5 tasks
provokateurin opened this issue Aug 10, 2023 · 6 comments
Open
1 of 5 tasks

Migrate all apps to generated OpenAPI specs #537

provokateurin opened this issue Aug 10, 2023 · 6 comments
Labels
feature New feature or request openapi OpenAPI specifications package: nextcloud

Comments

@provokateurin
Copy link
Member

provokateurin commented Aug 10, 2023 •
edited
Loading

They need to have the server side changes merged:
@provokateurin provokateurin added feature New feature or request package: nextcloud openapi OpenAPI specifications labels Aug 10, 2023
@provokateurin provokateurin self-assigned this Aug 10, 2023
@provokateurin
Copy link
Member Author

This is also a nice idea for the OpenAPI workshop. I think I have WIP stuff laying around for all of them, but it's pretty outdated. Sadly the Notes app needs some refactoring

@provokateurin provokateurin added this to the Nextcloud Conference Hackathon milestone Aug 10, 2023
@provokateurin provokateurin removed their assignment Aug 12, 2023
@provokateurin provokateurin mentioned this issue Sep 4, 2023
Merged
@Leptopoda
Copy link
Member

we might want to annotate those apis with @experimental so changing them after the release wouldn't count es breaking

@provokateurin
Copy link
Member Author

Or we manage to migrate them before releasing. I think that should be possible. Each of those apps shouldn't take much time to adapt. Only review might take some time.

@provokateurin provokateurin removed this from the Nextcloud Conference Hackathon milestone Sep 19, 2023
@provokateurin provokateurin mentioned this issue Oct 28, 2023
Merged
@provokateurin
Copy link
Member Author

At least News and Notes require quite some heavy refactoring to allow openapi-extractor to operate on them. I don't know if the required amount is reasonable for the outcome. The handwritten specs have proven to be quite reliable until now.

@Leptopoda
Copy link
Member

At least for cookbook it will take quite some time until we have an ocs api.

@provokateurin
Copy link
Member Author

I added it to the list

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature New feature or request openapi OpenAPI specifications package: nextcloud
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Leptopoda @provokateurin