carrot / ship

⛔️ currently unmaintained ⛔️
Other
151 stars 14 forks source link

Fix 404 in netlify deployer for new sites #71

Closed biilmann closed 9 years ago

biilmann commented 9 years ago

The lookup function would reject the promise if the site didn't already exist, and people would get a 404 error when trying to deploy to a new netlify site.

This change makes sure lookup returns null instead of rejecting the promise and makes ship path/ -to netlify work for new sites...

jescalan commented 9 years ago

Suppressing errors seems like a risky fix to me, but you are the one who builds this API so I've gotta trust you!