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

restructure routes & conform to zod-openapi spec #60

Closed
dromzeh opened this issue Nov 5, 2023 · 1 comment
Closed

restructure routes & conform to zod-openapi spec #60

dromzeh opened this issue Nov 5, 2023 · 1 comment
Assignees
Labels
db documentation Improvements or additions to documentation enhancement New feature or request priority: high Issue needs to be fixed ASAP

Comments

@dromzeh
Copy link
Member

dromzeh commented Nov 5, 2023

related to #51 - instead of routes being /v2/asset/upload-asset.ts, we can restructure like so:

/v2/asset/upload/route.ts (logic)
/v2/asset/upload/openapi.ts (openapi definition)
/v2/asset/upload/schema.ts (zod schema)

for routes which require an id or name, we can have /v2/user/get/[name]/route.ts, etc

@dromzeh dromzeh added documentation Improvements or additions to documentation enhancement New feature or request priority: high Issue needs to be fixed ASAP db labels Nov 5, 2023
@dromzeh dromzeh self-assigned this Nov 5, 2023
@dromzeh dromzeh added this to Pipeline Nov 5, 2023
@dromzeh dromzeh moved this to In Progress in Pipeline Nov 5, 2023
@dromzeh
Copy link
Member Author

dromzeh commented Jan 22, 2024

#51

@dromzeh dromzeh closed this as completed Jan 22, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in Pipeline Jan 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
db documentation Improvements or additions to documentation enhancement New feature or request priority: high Issue needs to be fixed ASAP
Projects
Status: Done
Development

No branches or pull requests

1 participant