Closed shanewho closed 10 years ago
please fix the small style issues then it should be good to merge, assuming the unittests and regression test pass.
Ok, change made. Let me know if there is anything you'd like. Thanks for the feedback.
Hey shane, I merged in the other CL that was inspired by your work. You should now be able to revert your changes to the .java files. Can you resolve your changes, then we can merge.
Got my branch all messed up, will fix and recreate in a bit
This enables the server to be hosted on Heroku. You need to be able to set the postgres url to the Amazon RDS instance they setup, and use a specified port. This also replaces rsync with a cp command to make it work on very stripped down systems. Finally, you can't currently set the server url if you are not yet logged into an existing server (it redirects from preferences to login if you aren't logged in). This change allows preferences to be accessed if not logged in.
If these changes are pulled in, I am close to having a buildpack that will actually deploy the server to a Heroku instance. Hoping to have a 2 click heroku deploy to get a custom server instance deployed, then easy to set server url on the clients.