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

When a post goes live, notify users who touched it #14

Open
peterkinnaird opened this issue May 29, 2013 · 3 comments
Open

When a post goes live, notify users who touched it #14

peterkinnaird opened this issue May 29, 2013 · 3 comments

Comments

@peterkinnaird
Copy link
Contributor

Encourage them to share/retweet/reshare it.

'touch' is any operation including just clicked through to read, voted, wrote headline (even one that didn't 'win'), etc

@artfuldodger
Copy link
Member

Just trying to think this through a bit here:

Somewhat dependent on #12

As an additional prerequisite, we'll need the concept of a notification and a delivery mechanism for users.

To make this easier, we can introduce the concept of an Interaction which has a polymorphic association to an item (content or link) and a polymorphic association to an action (click, submission, like). This would store some redundant data, so I'm not completely sold on the idea. Alternatively, we could have a class that's just responsible for identifying what people have interacted with the content that was posted either directly or indirectly via it's associated link or its other contents and creating appropriate notifications.

@peterkinnaird
Copy link
Contributor Author

Right. My inclination would be to do it the second way but it's not clear that this is the 'right' way to do it. I'm going to ping Nathan on this who might have some ideas about the scalability of each approach.

@peterkinnaird
Copy link
Contributor Author

Right. The second way seems better to me since it's a separable process, that could be scheduled independently. I'm not really sure though. Pinging Nathan to see if he has ideas about the scalability of either method since they both seem about the same in terms of coding difficulty.

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

No branches or pull requests

2 participants