Skip to content
This repository has been archived by the owner on Nov 19, 2018. It is now read-only.

Add support to the input form as an array name #48

Closed

Conversation

jonataa
Copy link

@jonataa jonataa commented Jan 27, 2015

I tried using the code below, but it only send the last checkbox (not checked).

<input type="checkbox" name="checkbox1[]" value="foo" checked>
<input type="checkbox" name="checkbox1[]" value="bar" checked>
<input type="checkbox" name="checkbox1[]" value="fizz">

Also, It isn't working for the input field as array format names. This link can help to understand: http://stackoverflow.com/questions/4709926/how-can-i-use-the-array-format-input-field-names-in-my-html-form-that-posts-to-p

I done some changes in the ajax-form.js to support it, but be will need update the current tests. I have not sure if this is better way. Make sense?

@rnicholus
Copy link
Owner

Thanks for this. It might be a while before I can look at this closer. I'm a bit overcommitted at the moment.

@rnicholus rnicholus added bug and removed bug labels Jan 29, 2015
@rnicholus
Copy link
Owner

I'm starting to look into this now as the last task to complete before releasing ajax-form 2.0.

@rnicholus rnicholus added this to the 2.0.0 milestone May 12, 2015
@rnicholus
Copy link
Owner

Completed, and about to be part of ajax-form 2.0.

@jonataa
Copy link
Author

jonataa commented May 16, 2015

Awesome!! Thanks, @rnicholus!

@rnicholus
Copy link
Owner

I threw this together in about an hour earlier today, but it's covered by the unit tests. Let me know if anything isn't quite right.

On Sat, May 16, 2015 at 6:21 PM, Jonata Weber notifications@github.com
wrote:

Awesome!! Thanks, @rnicholus!

Reply to this email directly or view it on GitHub:
#48 (comment)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants