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

Using field's Json name instead of internal conversion from snake cas… #54

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Using field's Json name instead of internal conversion from snake cas… #54

wants to merge 4 commits into from

Commits on Sep 28, 2018

  1. Using field's Json name instead of internal conversion from snake cas… 

    …e to camel case
    @dsborets
    dsborets committed Sep 28, 2018
    Configuration menu
    Copy the full SHA
    2ecdaa2 View commit details
    Browse the repository at this point in the history

  2. Adding more test

    @dsborets
    dsborets committed Sep 28, 2018
    Configuration menu
    Copy the full SHA
    2c2ab6a View commit details
    Browse the repository at this point in the history

  3. Adding more test to increase a code coverage

    @dsborets
    dsborets committed Sep 28, 2018
    Configuration menu
    Copy the full SHA
    e6a069b View commit details
    Browse the repository at this point in the history

Commits on Oct 24, 2018

  1. Converting originl gRPC field name to JSON name using custom conversi… 

    …on logic to cover the case of using

`[json_name="some_name"]` in .proto file so we can still call a getter by name. Plus some performance
enhancements - convert fields name during application startup time only.
    @dsborets
    dsborets committed Oct 24, 2018
    Configuration menu
    Copy the full SHA
    df5c2d5 View commit details
    Browse the repository at this point in the history