Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

Switched to using new MongoDB service in Bluemix #1039

Merged
merged 1 commit into from
Nov 4, 2015

Conversation

ryanjbaxter
Copy link
Contributor

Fixes #1029
The MongoLabs service in Bluemix is being removed so we need to use a different Mongo service.

@@ -91,4 +93,4 @@ module.exports = {
}
}
}
};
};
Copy link
Member

Choose a reason for hiding this comment

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

newline

Copy link
Member

Choose a reason for hiding this comment

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

newline

@ryanjbaxter ryanjbaxter force-pushed the bluemix-mongo-changes branch from f262ff4 to d948e79 Compare October 30, 2015 20:30
[IBM Bluemix](https://bluemix.net). Both provide free trials and support pay-as-you-go models
for hosting applications in the cloud. After you have an account follow the below steps to
for hosting applications in the cloud. After you have an account follow the below steps to
Copy link
Member

Choose a reason for hiding this comment

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

Why newline here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I dont think i added a new line I just deleted a space at the end of the lines

Copy link
Member

Choose a reason for hiding this comment

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

Okay, can you remove it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i'm confused you want me to add the trailing spaces back in? Why?

Copy link
Member

Choose a reason for hiding this comment

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

Make lines 201 and 202 one line.

@ryanjbaxter ryanjbaxter force-pushed the bluemix-mongo-changes branch 3 times, most recently from 3abebdd to 28bad2f Compare November 2, 2015 14:12
@ryanjbaxter ryanjbaxter force-pushed the bluemix-mongo-changes branch from 28bad2f to 7340e68 Compare November 2, 2015 16:51
@mleanos
Copy link
Member

mleanos commented Nov 2, 2015

LGTM

@ilanbiala
Copy link
Member

@lirantal LGTM.

@codydaig
Copy link
Member

codydaig commented Nov 3, 2015

LGTM

lirantal added a commit that referenced this pull request Nov 4, 2015
Switched to using new MongoDB service in Bluemix
@lirantal lirantal merged commit 61ea25c into meanjs:master Nov 4, 2015
@rhutchison rhutchison mentioned this pull request Nov 4, 2015
3 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants