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

Adds docs about query string decoding #131

Merged
merged 1 commit into from Sep 16, 2015
Merged

Conversation

@conradkleinespel
Copy link
Contributor

conradkleinespel commented Sep 16, 2015

Hello,

I was looking for a library to decode a query string from a URL, when I hadn't realised url::form_urlencoded::parse did just that.

I hope this little doc addition makes things a bit more clear.

Best regards,
Conrad

Review on Reviewable

@SimonSapin
Copy link
Member

SimonSapin commented Sep 16, 2015

Looks good.

@bors-servo r+

@bors-servo
Copy link
Contributor

bors-servo commented Sep 16, 2015

📌 Commit ff02193 has been approved by SimonSapin

@bors-servo
Copy link
Contributor

bors-servo commented Sep 16, 2015

Testing commit ff02193 with merge f8a1344...

bors-servo pushed a commit that referenced this pull request Sep 16, 2015
Adds docs about query string decoding

Hello,

I was [looking](https://twitter.com/kleinespel/status/643462183783923712) for a library to decode a query string from a URL, when I hadn't realised `url::form_urlencoded::parse` did just that.

I hope this little doc addition makes things a bit more clear.

Best regards,
Conrad

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/rust-url/131)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 16, 2015

☀️ Test successful - travis

@bors-servo bors-servo merged commit ff02193 into servo:master Sep 16, 2015
3 checks passed
3 checks passed
continuous-integration/appveyor AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.