search

inhabitedtype / ocaml-webmachine

A REST toolkit for OCaml
Other
221 stars 31 forks source link

Implement missing POST functionality #50

Closed tmcgilchrist closed 4 years ago

tmcgilchrist commented 8 years ago

Just a heads up I'm going part way through implementing this.

Basically I'm going to follow how erlang web machine does it.

tmcgilchrist commented 8 years ago

FYI https://github.com/inhabitedtype/ocaml-webmachine/commit/6ff58d817e065eff5eefd3980880c1a26c41a17c

seliopou commented 8 years ago

Cool! Send a pull request when you think it's ready. I'm releasing 0.3.1 right now and will then merge your cache headers PR.

seliopou commented 4 years ago

@tmcgilchrist Do you mind if I use some or all of the patch you referenced above?

tmcgilchrist commented 4 years ago

Not at all. Go for it.

On Mon, 20 Apr 2020 at 4:34 am, Spiros Eliopoulos notifications@github.com wrote:

@tmcgilchrist https://github.com/tmcgilchrist Do you mind if I use some or all of the patch you referenced above?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/inhabitedtype/ocaml-webmachine/issues/50#issuecomment-616203366, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABJXONKOEVKNQK55JDWLSTRNM74ZANCNFSM4B3F4B4A .

hannesm commented 4 years ago

this has been merged as part of #96 and part of the 0.7.0 release. I think this issue can be safely closed.