Skip to content

Orval implementation#34

Draft
rishikamtam12 wants to merge 13 commits intomainfrom
orval_implementation
Draft

Orval implementation#34
rishikamtam12 wants to merge 13 commits intomainfrom
orval_implementation

Conversation

@rishikamtam12
Copy link
Copy Markdown
Collaborator

note:

Generated output structure differs slightly from the original expectation.Orval generates schema-based type files with a shared index rather than a single flat generated.types.ts file.

Why this approach was kept:

  • Aligns with Orval’s default and recommended output
  • Scales better as the API grows (one file per schema instead of one large file)

Copy link
Copy Markdown
Collaborator

@bamarler bamarler left a comment

Choose a reason for hiding this comment

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

Great work, this looks nice. Just please make sure that the checks pass.

Also, in the issue I said that we should use a zod plugin. Ideally Orval can generate that, but if you found that it's not possible we can discuss if it's necessary.

@bamarler bamarler marked this pull request as draft February 12, 2026 02:29
@bamarler bamarler linked an issue Mar 19, 2026 that may be closed by this pull request
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.

Add Orval to Frontend

3 participants