Skip to content

Commit 0fa54f4

Browse files
committed
Added todo list.
1 parent 9d87726 commit 0fa54f4

1 file changed

Lines changed: 36 additions & 0 deletions

File tree

todo.md

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
# Why
2+
3+
Handling comment subscriptions in a single place, rather than trying to work
4+
out who we should and should not notify, will improve both our code and the user
5+
experience.
6+
7+
# How
8+
9+
We will store subscriptions in a new model, and automatically create subscriptions when certain events happen.
10+
11+
# What
12+
13+
We will create a new model for subscriptions, and listeners for comments, attachments and changesets. Then we will refactor the comment notification code to use the subscriptions instead of determining recipients.
14+
15+
# Caveats
16+
17+
No UI will be in this branch.
18+
19+
Need to figure out how to handle blanket subscriptions.
20+
21+
# Tasks
22+
23+
## Subscriptions
24+
25+
- [ ] model
26+
- [ ] db
27+
- [ ] api - create, update, delete, query
28+
29+
## Listeners
30+
31+
- [ ] comments_created - create a subscription for the comment author
32+
- [ ] attachment_added - create a subscription for the attachment author
33+
- [ ] attachment_deleted - remove all subscriptions for the attachment (should we also remove all comments?)
34+
- [ ] attachment_reparented - update all subscriptions for the attachment
35+
- [ ] changeset_added - create a subscription for the changeset author
36+
- [ ] changeset_modified - update the subscription (author?)

0 commit comments

Comments
 (0)