Skip to content

Feature/discourse migration #28

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

Merged
merged 33 commits into from
Apr 23, 2018
Merged

Feature/discourse migration #28

merged 33 commits into from
Apr 23, 2018

Conversation

vikasrohit
Copy link

Merging discourse migration changes into dev. Major change is email notifications for messaging related events. This feature can be controlled via env variables, so we would deploy the release to production with this feature being disabled and then will enable it after validating that changes are working fine in dev mode (which is also controlled via env variable).

fyi @mtwomey @cwdcwd @sachin-maheshwari @gondzo

gondzo and others added 30 commits March 23, 2018 00:55
deploying discourse migration branch to dev env
# Conflicts:
#	config/default.js
#	deploy.sh
… address

Updated message API url to point to beta URL so that it fetches the topic from new database
Used latest naming convention for email topics i.e. domain being originating application
@vikasrohit vikasrohit merged commit 80b6257 into dev Apr 23, 2018
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.

2 participants