Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Deploy MVP to Heroku #64

Closed
nelsonic opened this issue Mar 10, 2015 · 8 comments
Closed

Deploy MVP to Heroku #64

nelsonic opened this issue Mar 10, 2015 · 8 comments
Assignees
@nelsonic
Labels
API enhancement New feature or enhancement of existing functionality help wanted If you can help make progress with this issue, please comment! MVP
Milestone
MVP

Comments

@nelsonic
Copy link
Member

https://addons.heroku.com/searchbox#micro should do it as discussed in #2
@nelsonic nelsonic added enhancement New feature or enhancement of existing functionality help wanted If you can help make progress with this issue, please comment! MVP API labels Mar 10, 2015
@nelsonic nelsonic self-assigned this Mar 10, 2015
@nelsonic nelsonic added this to the MVP milestone Mar 10, 2015
@nelsonic
Copy link
Member Author

What do we think of adding the button to the readme:
Deploy
see: https://github.com/peers/peerjs-server

@iteles iteles added the [alpha] label Apr 10, 2015
@nelsonic
Copy link
Member Author

Before we can deploy to heroku we need to loop through ALL our handlers and update the elasticsearch index to be an Environment Variable.

@nelsonic nelsonic mentioned this issue Apr 13, 2015
Closed
nelsonic added a commit that referenced this issue Apr 13, 2015
@nelsonic
uses process.env.ES_INDEX for ES index. #129 & #64
b9e598b
nelsonic added a commit that referenced this issue Apr 13, 2015
@nelsonic
adds Procfile for deploy to heroku. #64
f580e3c
nelsonic added a commit that referenced this issue Apr 13, 2015
@nelsonic
tidy index.html and alpha.js to resolve heroku warnings. #64
3a69c77
@nelsonic
Copy link
Member Author

dwylio-heroku-errors

@nelsonic
Copy link
Member Author

https://dwylio.herokuapp.com
but...
dwyl-heroku-console-error

@nelsonic
Copy link
Member Author

heroku (searchbox) not cooperating... thinking of switching to found but their entry plan is $40/month

@nelsonic
Copy link
Member Author

@iteles in case you are wondering what I'm up to #_WhatsTakingSoLong_ ?
We are having issues with heroku. 😢
So I'm taking time out to write _two_ example apps for

  1. hapi-auth-jwt2: Create "real-world" (Tested!) example using Redis? hapi-auth-jwt2#9
  2. esta: SearchBox (herku addon) provides a url (without port) we need to support that in options.js esta#70

Both will be code we will use in Time, its not wasted effort.
The more effort we put into our modules, the more adoption we will get.
The more adoption, the more "eyes" we have for rooting out any bugs (or security flaws!)
Really wish I could be timing the hours I have to invest in this... #soon! 😉

@nelsonic nelsonic mentioned this issue Apr 15, 2015
Merged
@nelsonic
Copy link
Member Author

@iteles please try: https://dwylio.herokuapp.com/ 😉

@iteles
Copy link
Member

iteles commented Apr 15, 2015

😯😯
Awesome! Officially ready to test for bugs 🎉

@iteles iteles closed this as completed Apr 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nelsonic nelsonic

Labels
API enhancement New feature or enhancement of existing functionality help wanted If you can help make progress with this issue, please comment! MVP
Projects
None yet
Milestone
MVP
Development

No branches or pull requests
2 participants
@nelsonic @iteles