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.

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

Update heroku docs in getting started #139

Merged
merged 1 commit into from
Aug 7, 2020
Merged

Update heroku docs in getting started #139

merged 1 commit into from
Aug 7, 2020

Conversation

shaftoe
Copy link
Contributor

@shaftoe shaftoe commented Aug 5, 2020

I was not that familiar with Heroku add-ons so I had to go through some documentation, I hope this will help new Ackee users willing to deploy on Heroku

@coveralls
Copy link

coveralls commented Aug 5, 2020

Pull Request Test Coverage Report for Build 507

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 46.981%

Totals Coverage Status
Change from base Build 505: 0.0%
Covered Lines: 454
Relevant Lines: 776

💛 - Coveralls

@@ -170,7 +170,9 @@ Simply deploy to Heroku by clicking this button:

### 2. Configure Ackee

Ensure that you're using the correct CORS headers by setting [`ACKEE_ALLOW_ORIGIN`](CORS%20headers.md#heroku-or-platforms-as-a-service-configuration).
- You need to have a MongoDB instance running, for example using [mLab MongoDB](https://devcenter.heroku.com/articles/mongolab) add-on which is currently available for free. This is as simple as typing `heroku addons:create mongolab --app <YOUR_APP_NAME>` using the CLI, or using the web dashboard; more details at the [official documentation](https://devcenter.heroku.com/articles/managing-add-ons). The add-on will inject `MONGODB_URI` variable into the Ackee application so you just need to restart Heroku app _dyno_ to make Ackee process connect to MongoDB. NOTE: you need to provide your credit card details in order to use add-ons in Heroku.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should suggest a different add-on provider as mLab is being deprecated soon.

Reference: https://devcenter.heroku.com/changelog-items/1823

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aleccool213 can you recommend any?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not yet 😟 I have yet to switch off of mLab.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aleccool213 yeah, ObjectRocket offering is too expensive for most basic needs, I wonder if it's possible/makes sense to create an Heroku add-on to run it on a free tier dyno

@electerious electerious merged commit c9cc5a4 into electerious:develop Aug 7, 2020
@electerious
Copy link
Owner

@shaftoe Thanks for the PR and thanks @aleccool213 for approving the changes! 👍

@shaftoe shaftoe deleted the fix-heroku-docs branch August 7, 2020 09:52
@shaftoe
Copy link
Contributor Author

shaftoe commented Aug 7, 2020

@aleccool213 @electerious FYI, I'm testing the free MongoDB Atlas offer and it looks like a decent replacement for mLab's one for basic usage (512MB free storage, shared host), even though it doesn't come with any native Heroku integration.

HTH

@electerious
Copy link
Owner

@shaftoe Sounds good! Let us know how it performances and if 512mb are enough. Might be worth to reduce the data use in Ackee if not.

@shaftoe
Copy link
Contributor Author

shaftoe commented Aug 8, 2020

@electerious will do!
Also, I wonder if there's an easy way to prune old records from MongoDB, personally I'm only interested in having the last month or so available from the web interface. Probably some DANGER ZONE section in the web client with a "remove older than X days" feature would be best.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants