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

Issue 668 study group announcement #680

Merged
merged 14 commits into from Oct 21, 2017

Conversation

Projects
None yet
2 participants
@railsstudent
Collaborator

railsstudent commented Oct 20, 2017

Fixes #668.

When study group is created, the method called studyGroupNotification() in slack/method.js to send simple text message to channel.
Study group notification uses notebook emoji and 'Study Group Alerts' username to display alert in the channel.
Username is defined as a property in setting-development.json; therefore, a corresponding entry should exist in production config in production hosting location.

@lpatmo lpatmo added the In Review label Oct 20, 2017

@railsstudent

This comment has been minimized.

Show comment
Hide comment
@railsstudent

railsstudent Oct 20, 2017

Collaborator

slackalert

Collaborator

railsstudent commented Oct 20, 2017

slackalert

@lpatmo

This comment has been minimized.

Show comment
Hide comment
@lpatmo

lpatmo Oct 21, 2017

Member

So excellent. Thank you @railsstudent!

Member

lpatmo commented Oct 21, 2017

So excellent. Thank you @railsstudent!

@lpatmo

lpatmo approved these changes Oct 21, 2017

@lpatmo lpatmo merged commit ab1241c into codebuddies:staging Oct 21, 2017

@lpatmo lpatmo added closed and removed In Review labels Oct 21, 2017

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