Skip to content

routing.build: Allow multiple values for the same field#660

Merged
untitaker merged 1 commit into
pallets:masterfrom
houseofsuns:url-multiple-value
Jan 29, 2015
Merged

routing.build: Allow multiple values for the same field#660
untitaker merged 1 commit into
pallets:masterfrom
houseofsuns:url-multiple-value

Conversation

@houseofsuns
Copy link
Copy Markdown

This allows to construct urls like
'http://example.com/?multi=1&multi=2&multi=3' which have multiple
values for the same field. This is common practice for most web
frameworks. The reverse operation is allready supported, i.e. multiple
values get aggregated into a MultiDict.

@untitaker
Copy link
Copy Markdown
Contributor

Thanks, please add a testcase and a changelog entry!

This allows to construct urls like
'http://example.com/?multi=1&multi=2&multi=3' which have multiple
values for the same field. This is common practice for most web
frameworks. The reverse operation is allready supported, i.e. multiple
values get aggregated into a MultiDict.
@houseofsuns
Copy link
Copy Markdown
Author

I updated the commit in place (I hope github doesn't barf on this).

untitaker added a commit that referenced this pull request Jan 29, 2015
routing.build: Allow multiple values for the same field
@untitaker untitaker merged commit 9b161a6 into pallets:master Jan 29, 2015
@untitaker
Copy link
Copy Markdown
Contributor

Thanks!

untitaker added a commit that referenced this pull request Jan 29, 2015
joshfriend pushed a commit to joshfriend/werkzeug that referenced this pull request Feb 1, 2015
untitaker pushed a commit that referenced this pull request Feb 1, 2015
untitaker added a commit to untitaker/werkzeug that referenced this pull request Mar 18, 2015
untitaker added a commit to untitaker/werkzeug that referenced this pull request Mar 18, 2015
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Nov 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants