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

Optimize Self Hosted & Other Improvements #29

Closed
5 tasks done
nprail opened this issue Apr 1, 2017 · 4 comments · Fixed by #33
Closed
5 tasks done

Optimize Self Hosted & Other Improvements #29

nprail opened this issue Apr 1, 2017 · 4 comments · Fixed by #33

Comments

@nprail
Copy link

nprail commented Apr 1, 2017

Simply put, this does not work out of the box when self hosted. It is VERY specific to Rallly.co and some changes need to be made. Here are some of the things I found:

Self Hosting Optimization

  • The from email address is hard coded to noreply@rallly.co (should be set in config/main.js)
  • The from name is hard coded to Rallly (also should be set it config/main.js)
  • Docker/docker-compose support: Containers are all the rage these days. This would be nice for a quick setup of Rallly.
  • Better installation instructions. I had to jump through quite a few hoops to get it running in the first place. There is no docs on how MongoDB should be configured or how to keep Rallly running (e.g. with Forever.
  • While there is the ability to set your own SendGrid API key, it doesn't actually work because it references a SendGrid email template which is SendGrid account based. The template ID should also not be hard coded. The creation of a SendGrid template should be documented.

General Improvements/Feature Ideas

Rallly.co Cloud Hosted Service

I can definitely help with some of this. I've got a decent amount of MEAN (minus mongo) experience. Be expecting some PRs.

I love the idea of this software and I would love to use it but it needs a little work as all software does. 😉

@nprail nprail changed the title Optimize Self Hosted Optimize Self Hosted & Other Improvements Apr 1, 2017
@lukevella
Copy link
Owner

Hey @nprail, thanks for writing this up. You're right, Rallly in it's current state is not optimized to be self hosted. Unfortunately my availability is very limited, which means I'm not really able to sufficiently offer support for this project and I don't know if and when this will change. Of course you're free to use and modify the source as you wish, and feel free to ask if you have any questions.

nprail added a commit to nprail/Rallly that referenced this issue Apr 1, 2017
@nprail
Copy link
Author

nprail commented Apr 1, 2017

@lukevella Awesome!

I'm going to move some of these tasks to their own issues to keep discussion from being cluttered with quite a few different things. As you can see, I already added some new config options. I'm unable to test them, though, because I don't know how to configure the SendGrid template.

@nprail
Copy link
Author

nprail commented Apr 2, 2017

@lukevella What do you think about using nodemailer instead of only SendGrid?

@nprail
Copy link
Author

nprail commented Apr 20, 2017

@lukevella Be expecting a PR soon with self hosting improvements. I've done almost everything in this issue.

nprail added a commit to nprail/Rallly that referenced this issue Apr 20, 2017
nprail added a commit to nprail/Rallly that referenced this issue Apr 20, 2017
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 a pull request may close this issue.

2 participants