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

Email notifications for admins when new talks are submitted #27

Closed
phiggins opened this issue Mar 3, 2014 · 7 comments
Closed

Email notifications for admins when new talks are submitted #27

phiggins opened this issue Mar 3, 2014 · 7 comments

Comments

@phiggins
Copy link
Member

phiggins commented Mar 3, 2014

When someone submits a new talk through our talk submission form, admins should get an email about it. The email should contain most of the information in the submission.

This feature will be done when:

  • A talk submission queues an email to admins who have opted-in to notifications.
  • Bonus points: An admin should be able to opt-in or opt-out of these notifications. This should probably just be a boolean flag that is exposed in the admin panel.

The site is hosted on heroku, and there are a few free email options that can be investigated:

... and possibly more. The key things we're looking for are simplicity and no cost.

@audreycarlsen
Copy link

I'd like to take a stab at this!

@phiggins
Copy link
Member Author

phiggins commented Mar 7, 2014

Go for it! Let me know if you have questions.

@audreycarlsen
Copy link

Question: How is admin functionality set up for this website? I'm not seeing an Admin model or admin column in the User or Dude models. I'm also not seeing any sign of an admin controller or dashboard view. Is admin functionality buried in the code or in a gem, or should I go ahead and start implementing it from scratch?

@zenspider
Copy link
Member

we use the railsadmin gem (or something similarly named).

If you wanted to make me really happy, you'd obliterate the "Dude" model. Fucking horrible name. Don't blame me or pete for that one. :)

@zenspider
Copy link
Member

OK. I just double checked. The User model is currently only for the admin interface. The Dude model is just for the people we list and their projects.

We don't have a real notion of roles at this point, so we can't collapse the two. Until we have roles, leaving Dude as-is is fine. Adding roles is out of scope for this task. I'll make sure there is a ticket for it.

zenspider added a commit that referenced this issue Apr 7, 2014
Talk Email Notifications to Admins Issue #27
@phiggins
Copy link
Member Author

phiggins commented Apr 9, 2014

Closed by #60

@phiggins phiggins closed this as completed Apr 9, 2014
@zenspider
Copy link
Member

On Apr 8, 2014, at 19:29, pete higgins notifications@github.com wrote:

Closed by #60

oops! thanks!

@seattlerb seattlerb locked and limited conversation to collaborators Jan 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants