Created
February 11, 2015 22:07
-
-
Save commuterjoy/50c2d041464fd90eadc9 to your computer and use it in GitHub Desktop.
notes on app per branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Heroku apps aren't hosted on a ft.com domain so cookie reads fail - a pr | |
request router in fastly with an canary-next.ft.com CNAME? | |
- Some XHR requests fail (eg, __flags.json) as the app is only serving engels | |
routes. | |
- We shouldn't set NODE_ENV to production - adapt next-build-tools to do this. | |
- It takes 3 mins for the heroku box to be alive, which means it's hard to | |
point sauce labs at it in the same build - Talk to Heroku ?? | |
- Need to truncate project, branch, build, time etc. in to a 30 character | |
length string. | |
- Destroying apps on the fly feels v. risky. | |
- Move the CODESHIP ENV stuff out of build tools and pass as an argument. | |
- Update github with a message saying 'ready' here - THINKABOUT privacy |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment