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

"PropertyName" Naming Strategy #49

Closed
weaverryan opened this issue Nov 27, 2011 · 11 comments
Closed

"PropertyName" Naming Strategy #49

weaverryan opened this issue Nov 27, 2011 · 11 comments

Comments

@weaverryan
Copy link

Hi guys!

I was wondering if there was a specific reason for not having a naming strategy that simply uses the property name as the end field name. The use case is serializing an object for consumption by a BackboneJS model, but then a server-side Sf2 form to handle the BackboneJS model save (e.g. deserialize the posted data then proxy through a form).

Unless anyone can see a reason against this, I'm happy to make a PR for this.

Thanks!

@schmittjoh
Copy link
Owner

Sounds good to me.

On Sun, Nov 27, 2011 at 7:56 PM, Ryan Weaver <
reply@reply.github.com

wrote:

Hi guys!

I was wondering if there was a specific reason for not having a naming
strategy that simply uses the property name as the end field name. The use
case is serializing an object for consumption by a BackboneJS model, but
then a server-side Sf2 form to handle the BackboneJS model save (e.g.
deserialize the posted data then proxy through a form).

Unless anyone can see a reason against this, I'm happy to make a PR for
this.

Thanks!


Reply to this email directly or view it on GitHub:
#49

@lsmith77
Copy link
Contributor

+1

2 similar comments
@jgornick
Copy link

👍

@srosato
Copy link

srosato commented Apr 26, 2012

+1

@ianunruh
Copy link

+1

@wheelsandcogs
Copy link
Contributor

This would remove the need for #139 for me (although there's no reason both shouldn't be available).

@ruimarinho
Copy link
Contributor

+1 this would be really nice!

@marco-santamaria
Copy link

+1

@madhatter42
Copy link

@weaverryan just wondering if you've been able to make any progress on this (or are planning to)?

@fredjiles
Copy link

+1

@schmittjoh
Copy link
Owner

This is super simple to implement. If you really need it, please someone send a PR instead of adding +1s on this issue.

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

No branches or pull requests