tangled.org / core
Monorepo for Tangled — https://tangled.org
fork atom

appview: introduce email notifications for @ mentions on issue/pr comments #393

closed
opened by boltless.me targeting master from boltless.me/core: feat/mentions

Stacked on top of #392

Yes, I know we have stacked PRs, but I want to explicitly separate two sets of commits and review both on different places

This is MVC implementation of email notification.

Still lot of parts are missing, but this is a PR with most basic features.

0
by boltless.me 2 comments
expand 7 commits
90679d26
appview: reporesolver: pass entire Repo object through context
3fbb4f0b
apview: replace all use of db.Repo.AtUri with db.Repo.RepoAt()
1b6628ab
appview/pages: markup: add @ user-mention parsing in markdown
e1287c95
appview: add NewIssueComment event
18b248e5
appview: email: send email notification on mention
9eebc110
appview: email: add source to email notification
dfa71348
appview: email: notify mentioned users on pull-comments
oppi.li

i'd prefer if this was stacked! it would make it slightly easier to review:

  • in 1b6628ab: can we use the handle regex from the userutil module and customize it a bit for atRegex?
  • 18b248e5 and 9eebc110 seem to be modifying the same file, it would be nice to merge the two changes (running jj absorb when editing the second change should automatically solve it!).
  • similarly, dfa71348 seems to update the logic for issue-comment emails. would be nice to collapse or clean up the diffs.

this was only a first pass of reviews, but ill go through and have a closer look at the logic in the code, and give this a go locally. thanks again for working on this, its looking pretty cool already!

boltless.me

@oppi.li Thank you for the review!

and yeah I agree that I should have made this PR stacked.

Tell me if you think complete resubmission in form of stacked PR will be better.

sign up or login to add to the discussion
1
by boltless.me 1 comment
expand 6 commits
90679d26
appview: reporesolver: pass entire Repo object through context
3fbb4f0b
apview: replace all use of db.Repo.AtUri with db.Repo.RepoAt()
1b6628ab
appview/pages: markup: add @ user-mention parsing in markdown
34398af1
appview: add NewIssueComment event
8f874517
appview: email: send email notification on mention
5f2288c9
appview: email: notify mentioned users on pull-comments
boltless.me

@oppi.li I addressed your last two points. Thank you again for the detailed review.

for regex part, I'm not sure if we can share part of regex and modify by our needs.

sign up or login to add to the discussion
2
by boltless.me 0 comments
expand 7 commits
5a3ea5b6
appview: reporesolver: pass entire Repo object through context
1654667a
apview: replace all use of db.Repo.AtUri with db.Repo.RepoAt()
55002a1e
appview: db/repos: remove AtUri from Repo
7c74733d
appview/pages: markup: add @ user-mention parsing in markdown
47ad1a3b
appview: add NewIssueComment event
886b401c
appview: email: send email notification on mention
efb0f1b3
appview: email: notify mentioned users on pull-comments
sign up or login to add to the discussion
3
by boltless.me 0 comments
expand 7 commits
e33a63e7
appview: reporesolver: pass entire Repo object through context
26adbe09
apview: replace all use of db.Repo.AtUri with db.Repo.RepoAt()
0a65a3f9
appview: db/repos: remove AtUri from Repo
c247aea0
appview/pages: markup: add @ user-mention parsing in markdown
869804c1
appview: add NewIssueComment event
1b57d9e6
appview: email: send email notification on mention
c154972c
appview: email: notify mentioned users on pull-comments
sign up or login to add to the discussion
4
by boltless.me 0 comments
expand 7 commits
05e23075
appview: reporesolver: pass entire Repo object through context
df874fc8
apview: replace all use of db.Repo.AtUri with db.Repo.RepoAt()
3413925b
appview: db/repos: remove AtUri from Repo
38014523
appview/pages: markup: add @ user-mention parsing in markdown
c3f41d5a
appview: add NewIssueComment event
0748a062
appview: email: send email notification on mention
e0000cba
appview: email: notify mentioned users on pull-comments
sign up or login to add to the discussion
5
by boltless.me 0 comments
expand 7 commits
639df66d
appview/pages: markup: add @ user-mention parsing in markdown
79fbdb0c
appview: add NewIssueComment event
d82b5d26
appview: db/issues: set IssueId on GetIssue
0f21c183
appview: email: support multiple recipients on SendEmail
92817524
appview: email: send email notification on mention
85b487ca
appview: settings: add email preference setting
e4deb0f4
appview: email: notify mentioned users on pull-comments
sign up or login to add to the discussion
6
by boltless.me 1 comment
expand 7 commits
28409862
appview/pages: markup: add @ user-mention parsing in markdown
171488e2
appview: add NewIssueComment event
d3b193bb
appview: db/issues: set IssueId on GetIssue
1b5db515
appview: email: support multiple recipients on SendEmail
cad6e912
appview: email: send email notification on mention
4a8e0e3c
appview: settings: add email preference setting
732b33e8
appview: email: notify mentioned users on pull-comments
closed without merging
sign up or login to add to the discussion
Labels

None yet.

assignee

None yet.

Participants 2
AT URI
at://did:plc:xasnlahkri4ewmbuzly2rlc5/sh.tangled.repo.pull/3lvdu42ftpn22