-
Notifications
You must be signed in to change notification settings - Fork 31
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
Add a way to sign up for Tuesday emails only #87
Comments
Hi, is #47 solved? I can take up on this task. |
Ok, so here is going to be the outline.
Please advise if you want anything changed. |
I have only one question @aekazitt. Would it be better to create a User record for each email address that signs up for the emails only? |
Hi, after looking into the schema, I advise against the above approach. While the casual subscriber, though can be assigned as a Role does not need the complexity of Authentication implemented nor being Featured. My approach so far has been to create a model named the TuesdayReader which can either register new email address or pull email address from registered User with appropriate validations implemented. The solution is aimed at creating as little side-effects as possible, while also permitting existing users to Opt-in to the Tuesday Newsletters and not automatically added to the Mailing List. (Community Member / Contributing Member Mailing List can also be implemented.) Here are some follow-up questions that I would like to ask:
And for the Casual Member -> Community Member Transition
|
@aekazitt some answers:
|
What's done since our last chat:
Remaining Tasks:
|
In order to use ActiveJobs (as part of ActiveSupport) I need to upgrade Ruby version fro 2.2.0 to 2.2.2+ Is that ok? |
As long as it doesn't break anything else, I'm fine with it. |
In issue #47 we send an email to users, the public should have a way to just opt in to these emails. Maybe through a sign up to be a user.
BUT these people wouldn't be show on the people list.
The text was updated successfully, but these errors were encountered: