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

Use $.find, not $.children, to parse nested inputs within a specific form #110

Merged
merged 1 commit into from
May 3, 2015

Conversation

david-davidson
Copy link
Contributor

Reopening against minor

This updates getForm to rely on $.find, not $.children, when searching for inputs within a specific form (not a view). That's consistent with how it handles inputs within a view (View.$, which is the equivalent of $.find), and it lets Syphon pull data from inputs nested more than one level down, like <div><input></input></div>. Fixes #104 .

/cc @rhubarbselleven

Marionette: 2.4.1 / Syphon: 0.6.0

@rhubarbselleven
Copy link
Contributor

👍 Fantastic; great work @david-davidson

rhubarbselleven added a commit that referenced this pull request May 3, 2015
Use `$.find`, not `$.children`, to parse nested inputs within a specific form
@rhubarbselleven rhubarbselleven merged commit 815c355 into marionettejs:minor May 3, 2015
@happylinks
Copy link

When is this going to be released in the master version?

@joews
Copy link

joews commented Jul 2, 2015

Ping - any chance of getting this into a release?

@samccone
Copy link
Member

samccone commented Jul 2, 2015

yes we can cut a release today :) @happylinks @jwhitfieldseed do either of you want to help maintain this lib with us :)

@joews
Copy link

joews commented Jul 2, 2015

@samccone I am pretty short on open source time at the moment but I will help out where I can :)

@happylinks
Copy link

Sure, sounds fun. I'll try to help when needed :)

@happylinks
Copy link

Bump - can somebody please release this? If there's anything I can do to help, let me know.

@happylinks
Copy link

I build it myself and updated the version, not sure if I did it 100% correct though. Anybody care to check? happylinks@a525b2e
I can make a pull request afterwards.

@samccone
Copy link
Member

@happylinks @jwhitfieldseed released as 0.6.1

@happylinks
Copy link

Shouldn't you also run grunt build? Currently the lib/backbone.syphon.js in minor is still old. Or am I mistaken?

@samccone
Copy link
Member

yea I messed up 😢 sorry all fixed in 0.6.2

@happylinks
Copy link

Jee, thanks a lot! :)

@samccone
Copy link
Member

thank you!

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.

None yet

5 participants