Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add PropertyNamingStrategy #37

Merged
merged 2 commits into from
Feb 14, 2013
Merged

Conversation

passkey1510
Copy link
Contributor

As discussed in #33

@passkey1510
Copy link
Contributor Author

When the PR is merged, I could make another one on SerializerBundle to adapt the changes. Thanks.

@schmittjoh
Copy link
Owner

Could you rename it to IdenticalPropertyNamingStrategy and also add a small test case?

@passkey1510
Copy link
Contributor Author

Renamed and added test cases

schmittjoh added a commit that referenced this pull request Feb 14, 2013
Add PropertyNamingStrategy
@schmittjoh schmittjoh merged commit f95ba24 into schmittjoh:master Feb 14, 2013
@schmittjoh
Copy link
Owner

Thanks, merged!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants