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

Fix delete_json request helper to accept a query hash that gets seri… #33

Merged
merged 1 commit into from Sep 14, 2016

Conversation

kjellm
Copy link
Contributor

@kjellm kjellm commented Sep 13, 2016

…alized to query_string

@kjellm kjellm changed the title Fix delete_json request helper to accept a query hatch that gets seri… Fix delete_json request helper to accept a query hash that gets seri… Sep 13, 2016
@thhermansen
Copy link
Contributor

thhermansen commented Sep 13, 2016

Egentlig litt rart, da det ser ut for meg som at delete metoden vi kaller på burde kunne tatt i mot params ala #get metoden. ..eller er det noe jeg tenker feil her?

I #env_for ser det ut som det er noe custom håndtering om methoden er get.

Vi kunne jo akseptert hash som params, og brukt denne om vi fikk inn en hash og putta query stringen vi bygger på URL-en slik du gjør. Så hadde vi støttet begge ting, og spesielt hadde det vært fint om det er meninga at rack test #delete burde funka med hash til params slik som get gjør.


edit: Glem det siste jeg sa. Hadde jeg sett litt nærmere på encode_www_form, som jeg i farta bare passa på å url encode string-en, hadde jeg sett at endringen gjør det jeg tenkte vi burde gjøre :-)


Mao: 👍

@kjellm
Copy link
Contributor Author

kjellm commented Sep 14, 2016

Egentlig litt rart, da det ser ut for meg som at delete metoden vi kaller på burde kunne tatt i mot params ala #get metoden. ..eller er det noe jeg tenker feil her?

Nei, enig i at det er rart.

@kjellm kjellm merged commit 22b21ea into master Sep 14, 2016
@kjellm kjellm deleted the fix_delete_json branch September 14, 2016 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants