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

Mongodb hosting deployment #609

Merged
merged 3 commits into from Feb 8, 2017

Conversation

Projects
None yet
3 participants
@cfnelson
Contributor

cfnelson commented Feb 7, 2017

Added more details to the MongoDB hosting services to use with Galaxy section of the guide.

Changes made:

  • Added recommendation for compose.io and MongoDB Atlas in addition to mLab
  • Added more details on features to look out for when selecting a hosting service
  • Added hyperlink to a step-by-step guide to setting up MongoDB Atlas for your Meteor app

I attempted to keep changes to a minimum, let me know if this is a significant change and I can update the CHANGELOG.md.

@meteor-bot

This comment has been minimized.

meteor-bot commented Feb 7, 2017

@cfnelson: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Meteor Contributor Agreement here: https://contribute.meteor.com/

@stubailo

This comment has been minimized.

Contributor

stubailo commented Feb 7, 2017

Yeah this should definitely be in the changelog. A big reason we have that is so that people who haven't looked in a while can catch up on what has been added. This is great new information, so it should get a spot there.

@cfnelson

This comment has been minimized.

Contributor

cfnelson commented Feb 8, 2017

@stubailo Added a short description to the Changelog. Let me know what else I need to do to get this merged. 😄

@stubailo

This comment has been minimized.

Contributor

stubailo commented Feb 8, 2017

LGTM. I think it's good that the linked article makes a comparison for people.

@stubailo stubailo merged commit 77ce3d6 into meteor:master Feb 8, 2017

1 check passed

CLA Author has signed the Meteor CLA.
Details
@cfnelson

This comment has been minimized.

Contributor

cfnelson commented Feb 8, 2017

@stubailo Thanks for merging! 😄 Let me know if you would like something like the comparison in the Meteor guides. Will be happy to make further contributions and/or create a MongoDB deployment section in the guide.

@stubailo

This comment has been minimized.

Contributor

stubailo commented Feb 9, 2017

Personally I'd rather not have that directly inside because it's very easy for it to get outdated. I think the list of things to look for is perfect.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment