Skip to content

fix(mobile): patch nullables to use Option type #18343

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
Closed
shenlong-tanwen wants to merge 6 commits into from
Closed

fix(mobile): patch nullables to use Option type #18343

shenlong-tanwen wants to merge 6 commits into from

Conversation

@shenlong-tanwen
Copy link
Member

@shenlong-tanwen shenlong-tanwen commented May 17, 2025

Description

  • Patches the generator to wrap nullable values from the openapi spec with Option<T>
  • Skips wrapping optionals which aren't nullable

@shenlong-tanwen
Copy link
Member Author

shenlong-tanwen commented Jul 9, 2025

Will revisit this later after the Sqlite migration

@shenlong-tanwen shenlong-tanwen deleted the fix/mobile-generator-madness branch July 10, 2025 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jrasm91 jrasm91 Awaiting requested review from jrasm91

@alextran1502 alextran1502 Awaiting requested review from alextran1502

Assignees

No one assigned

Labels

changelog:bugfix 📱mobile

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@shenlong-tanwen