Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Sample requests should also support basic GET requests with standard que... #189

Closed
wants to merge 1 commit into from

Conversation

cauld
Copy link

@cauld cauld commented Jan 12, 2015

...ry parameters

Right now when doing sample requests with basic GET endpoints JSON stringified params are being appended to the end of the url rather than added as standard query params. For example, when sending a sample request with sample data like var1 = 123 & var2 = 456 you get something like example.com/?{%22var1%22:%22123%22,%var2%22:%2245%22} instead of example.com/?var1=123&var2=456.

@2naive
Copy link

2naive commented Jan 19, 2015

+1

1 similar comment
@kimar
Copy link

kimar commented Jan 20, 2015

+1

@nazrhyn
Copy link

nazrhyn commented Jan 21, 2015

👍
I was about to make this change myself, but it looks like @cauld already did!

@vherilier
Copy link

BTW +1

@digitalkaoz
Copy link

+1

@nazrhyn
Copy link

nazrhyn commented May 21, 2015

I'll add that we integrated this pull request into our local copy of this repo and it has mostly worked with a few modifications.

@gaving
Copy link

gaving commented May 28, 2015

👍 Hoping this feature makes it in.

@IcanDivideBy0
Copy link

@rottmann is there any chance to see this one merged please ?
There is actually 4 PR about this issue...

@iXioN
Copy link

iXioN commented Sep 3, 2015

+1

@rottmann
Copy link
Member

Thanks, added in 0.14.0

@rottmann rottmann closed this Dec 15, 2015
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.

None yet

10 participants