search

DrewWeth / HackMidwest

HackMidwest
2 stars 0 forks source link

Auto-detect host instead of hard coding it in #17

Closed chrisdopuch closed 10 years ago

chrisdopuch commented 10 years ago

Right now we either have "localhost:3000" or 'http://hack-midwest-app.herokuapp.com/" hard coded in for URLS. This makes debugging locally difficult, and is bad practice overall. Should be changed to detect the host and fill in dynamically.

DrewWeth commented 10 years ago

I've been using root_url for all the controllers that send texts. That should avoid problems

chrisdopuch commented 10 years ago

Good call. I'm refactoring Georgi's JS to do the same. I'll go ahead and close the issue, we should just keep this in mind moving forward.