search

dillonkearns / elm-pages

Hybrid Elm framework with full-stack and static routes.
https://elm-pages.com
BSD 3-Clause "New" or "Revised" License
658 stars 97 forks source link

Fix missing response headers for ApiRoutes #448

Closed adamdicarlo0 closed 10 months ago

adamdicarlo0 commented 10 months ago

I put in some debug code where I made this change and discovered the headers are only present in the body object.

The statusCode is present in the body object as well, but it is also in the main object, so I left that intact.

netlify[bot] commented 10 months ago

Deploy request for elm-pages-todos pending review.

Visit the deploys page to approve it

Name Link
Latest commit 96f46f6025e36264fbea26b0ef66f40a2dd5e9e8
adamdicarlo0 commented 10 months ago

Whoops, https://github.com/dillonkearns/elm-pages/pull/428 beat me to it. I knew I'd seen something about this before.