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

Allow notifications for new closures / events #144

Closed
charlesbrandt opened this issue Apr 24, 2018 · 4 comments
Closed

Allow notifications for new closures / events #144

charlesbrandt opened this issue Apr 24, 2018 · 4 comments
Assignees
Milestone

Comments

@charlesbrandt
Copy link
Contributor

When new events are added to the system, a subset of users would like to receive a notification email about the new events.

This feature should allow configuration of the group of users, along with sending the notification when new events are added to the system.

Initially, Andrew, Adam and Terri have all requested to receive these alerts.

@inghamn
Copy link
Member

inghamn commented Apr 27, 2018

inRoads will need a way to identify users who want these notifications. Users should probably be able to sign themselves up, which implies some sort of "My Account" edit screen, which we'll need to build.

@rbdietz
Copy link
Member

rbdietz commented Apr 27, 2018 via email

@inghamn
Copy link
Member

inghamn commented Apr 27, 2018

We're looking at this type of notification, as well as #151. Plus I suspect we'll have even more types of notifications to send out. It doesn't seem like we want to keep creating more and more google groups (one for each type of notification).

It seems like inRoads should send emails directly to individual users.

@inghamn
Copy link
Member

inghamn commented May 1, 2018

I need to make notifications something that can be turned on and off. I don't want to be sending out emails for every test event we do in test and staging.

Because of this and #151 we probably need an interface for maintaining the notifications and email lists.

inghamn added a commit that referenced this issue May 3, 2018
We're going to need a seperate flag for each type of
notification we want to send out.

Updates #144
Updates #151
@inghamn inghamn closed this as completed in 83eb271 May 3, 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

No branches or pull requests

3 participants