eskimor / servant-purescript

Translate servant API to purescript code, with the help of purescript-bridge.
BSD 3-Clause "New" or "Revised" License
105 stars 44 forks source link

fix: use the correct function and arguments for query args #10

Closed lthms closed 8 years ago

lthms commented 8 years ago

When using the Capture combinator, the generated output was using a non existing function. With this patch, the output compiles, but I do not have an easy way to test it yet.

What do you think?

eskimor commented 8 years ago

Ouch! I am guessing you mean QueryParam combinator? Ok - I never tested queries yet, but this still hurts. Thanks a lot for the fix!

lthms commented 8 years ago

Yes, sorry, my mistakes. I will be able to test it soon, I am currently working on a project with several API, including one with such combinators. I’ll tell you if it works.

Thanks for the merge!

eskimor commented 8 years ago

Awesome! Please! Also if you encounter problems - I am happy to help out! Thanks for testing!