pick up PR for named routes cookbook again
This PR is supposed to pick up the great work started in https://github.com/haskell-servant/servant/pull/1542 and rebase on current changes as well as apply fixes suggested by the reviewers, potentially also addressing new change requests.
Thank you very much @MangoIV for this work! Reading the PR content, I feel like we ought to merge the NamedRoutes and Generics cookbooks together, under the NamedRoutes name.
that's a good idea, I will do that then :)
@MangoIV Are you still planning to do this?
I am. I think we’re blocked by servant-qc (release, I have updated it) but I don’t know, let me check; will get to it this week perhaps ;)
@MangoIV You may rebase your PR on top of master now, I've integrated servant-quickcheck in the monorepo.
Lovely thank you <3
Any news/blockers here regarding the merge? I also find this very valuable.
@tfc yes, Mango still has to merge the two cookbooks. :)
I have basically dropped the entire generic cookbook in favour of named routes, afaict I have transported all the information that didn't already occur in named routes but I would appreciate a thorough review to confirm.
@ysangkok i still have a couple of failures sorry, I should mark it as draft again
What's left to do here? The build errors have disappeared
@ysangkok reminder you wanted to click the button <3
Thanks everyone for the patience.
@tchoutri Do you think we could deploy the documentation from master? I still don't see this listed on https://docs.servant.dev/en/stable/cookbook/index.html
@ysangkok look at your URL, you're on stable. Check out the master branch: https://docs.servant.dev/en/master/cookbook/named-routes/NamedRoutes.html