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

fix(codegen): protoc to keep snake_case for swagger and ts out #2149

Merged
merged 6 commits into from
Mar 16, 2022

Conversation

clockworkgr
Copy link
Collaborator

generate_javascript and generate_openapi were generating types incompatible with the json models of the API

Added:
json_names_for_fields=false option to the openapi protoc plugin
--ts_proto_opt=snakeToCamel=false to the ts-proto protoc plugin

Copy link
Collaborator Author

@clockworkgr clockworkgr left a comment

Choose a reason for hiding this comment

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

LGTM

@marinhoarthur
Copy link
Contributor

Removed SpVuexError usage because @starport/vuex does not export that type anymore.

marinhoarthur
marinhoarthur previously approved these changes Mar 15, 2022
@ilgooz ilgooz changed the base branch from develop to release/v0.19.5 March 16, 2022 07:46
@ilgooz ilgooz changed the base branch from release/v0.19.5 to develop March 16, 2022 07:46
@ilgooz ilgooz dismissed marinhoarthur’s stale review March 16, 2022 07:46

The base branch was changed.

@ilgooz ilgooz merged commit 1e3eee7 into develop Mar 16, 2022
@ilgooz ilgooz deleted the fix/snake-case-for-swagger-ts-proto branch March 16, 2022 07:47
ilgooz added a commit that referenced this pull request Mar 16, 2022
* fix: protoc to keep snake_case for swagger and ts out

* refactor

* removed SpVuexError

Co-authored-by: İlker G. Öztürk <ilkergoktugozturk@gmail.com>
Co-authored-by: marinhoarthur <arthurtutuca@gmail.com>
Jchicode pushed a commit to Jchicode/cli that referenced this pull request Aug 9, 2023
…e#2149)

* fix: protoc to keep snake_case for swagger and ts out

* refactor

* removed SpVuexError

Co-authored-by: İlker G. Öztürk <ilkergoktugozturk@gmail.com>
Co-authored-by: marinhoarthur <arthurtutuca@gmail.com>
Jchicode pushed a commit to Jchicode/cli that referenced this pull request Aug 9, 2023
…e#2149)

* fix: protoc to keep snake_case for swagger and ts out

* refactor

* removed SpVuexError

Co-authored-by: İlker G. Öztürk <ilkergoktugozturk@gmail.com>
Co-authored-by: marinhoarthur <arthurtutuca@gmail.com>
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.

3 participants