Skip to content

Fix TestClient for extra unquote in query parameters (#1952)#1953

Merged
Kludex merged 3 commits intoKludex:masterfrom
azogue:fix/testclient-query-unquote
Nov 16, 2022
Merged

Fix TestClient for extra unquote in query parameters (#1952)#1953
Kludex merged 3 commits intoKludex:masterfrom
azogue:fix/testclient-query-unquote

Conversation

@azogue
Copy link
Copy Markdown
Contributor

@azogue azogue commented Nov 16, 2022

The starting point for contributions should usually be a discussion

Simple documentation typos may be raised as stand-alone pull requests, but otherwise please ensure you've discussed your proposal prior to issuing a pull request.

This will help us direct work appropriately, and ensure that any suggested changes have been okayed by the maintainers.

@gaborbernat
Copy link
Copy Markdown

Can confirm that this fixes it. Any plans for merging and releasing this soon?

@Kludex
Copy link
Copy Markdown
Owner

Kludex commented Nov 16, 2022

Gracias @azogue por el PR :)

I was thinking: "tonight I'll just watch Netflix", and then I saw @gaborbernat's comment... Since I'm a fan of your work, I got up, and opened my IDE hahaha

Thanks for all your open source work. 😁

I'll cook a release in the next minutes.

@Kludex Kludex merged commit 24887d6 into Kludex:master Nov 16, 2022
@azogue azogue deleted the fix/testclient-query-unquote branch November 16, 2022 19:07
@gaborbernat
Copy link
Copy Markdown

Much appreciated ❣️

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.

3 participants