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

Handle arrays in parameters #1

Closed
tmeasday opened this issue Jul 17, 2014 · 2 comments
Closed

Handle arrays in parameters #1

tmeasday opened this issue Jul 17, 2014 · 2 comments

Comments

@tmeasday
Copy link
Contributor

The plan is to use rail's style, and have URLs of the form

/path?foo[]=a&foo[]=b

To end up with this.params.foo === ['a', 'b'].

The alternative is /path?foo=a&foo=b. But the downside there is you can't do a single element array, which is sucky. Either way you can't do a zero-element array. Not sure if there's anything principled that can be done about this.

Let me know if you want me to go ahead with this.

@tmeasday tmeasday self-assigned this Jul 17, 2014
@cmather
Copy link
Contributor

cmather commented Jul 17, 2014

Yep sounds good

On Jul 17, 2014, at 2:21 AM, Tom Coleman notifications@github.com wrote:

The plan is to use rail's style, and have URLs of the form

/path?foo[]=a&foo[]=b
To end up with this.params.foo === ['a', 'b'].

The alternative is /path?foo=a&foo=b. But the downside there is you can't do a single element array, which is sucky. Either way you can't do a zero-element array. Not sure if there's anything principled that can be done about this.

Let me know if you want me to go ahead with this.


Reply to this email directly or view it on GitHub.

@tmeasday
Copy link
Contributor Author

This appears to be on master.

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

No branches or pull requests

2 participants