Fix multiple QueryParams / QueryFlags

haskell-servant / servant-quickcheck

40 stars 21 forks source link

Closed declension closed 7 years ago

declension commented 7 years ago

Add test API taking multiple QueryParams
Add basic test using this API, generating an endpoint to ensure correct HTTP one=foo&two=bar query string generation is happening (that fails on master)
Fix (re)creation of query string to append & before the new parameter if there is already a built query string.

Fixes #23.

declension commented 7 years ago

@jkarni is this along the right lines?

declension commented 7 years ago

@jkarni - thanks. The last update should fix the QueryFlags too - hopefully still good to merge.

jkarni commented 7 years ago

Thanks again! I'll fix the CI issue you mention in #25 and if everything works make a release.