tangled.org / core
Monorepo for Tangled tangled.org
fork atom

appview/notify/db: set hard-limit to @-mention notification #759

merged opened by boltless.me targeting master from feat/mentions
STACK 4
#765 draft: render backlinks
1
#11
#764 appview/pages: fix broken issue-comment links
0
#11
#761 appview,lexicons: atprotate the mentions & references
0
#12
#760 appview: parse reference links from markdown body
0
#12
Labels

None yet.

assignee

None yet.

Participants 1
AT URI
at://did:plc:xasnlahkri4ewmbuzly2rlc5/sh.tangled.repo.pull/3m4vklvknsb22
+7
Diff #6
unified split
+7
appview/notify/db/db.go
··· 13 13 "tangled.org/core/idresolver" 14 14 ) 15 15 16 + const ( 17 + maxMentions = 5 18 + ) 19 + 16 20 type databaseNotifier struct { 17 21 db *db.DB 18 22 res *idresolver.Resolver ··· 421 425 issueId *int64, 422 426 pullId *int64, 423 427 ) { 428 + if eventType == models.NotificationTypeUserMentioned && len(recipients) > maxMentions { 429 + recipients = recipients[:maxMentions] 430 + } 424 431 recipientSet := make(map[syntax.DID]struct{}) 425 432 for _, did := range recipients { 426 433 // everybody except actor themselves

History

7 rounds 0 comments
sign up or login to add to the discussion
boltless.me submitted #6
interdiff
1 commit
expand
40c30be9
appview/notify/db: set hard-limit to @-mention notification
expand 0 comments
pull request successfully merged
boltless.me submitted #5
diff interdiff
1 commit
expand
9fd26505
appview/notify/db: set hard-limit to @-mention notification
expand 0 comments
boltless.me submitted #4
diff interdiff
1 commit
expand
2414e4b3
appview/notify/db: set hard-limit to @-mention notification
expand 0 comments
boltless.me submitted #3
diff interdiff
1 commit
expand
353070e9
appview/notify/db: set hard-limit to @-mention notification
expand 0 comments
boltless.me submitted #2
diff interdiff
1 commit
expand
c587dd3d
appview/notify/db: set hard-limit to @-mention notification
expand 0 comments
boltless.me submitted #1
diff interdiff
1 commit
expand
d7eab231
appview/notify/db: set hard-limit to @-mention notification
expand 0 comments
boltless.me submitted #0
diff
1 commit
expand
3fd36c3b
appview/notify/db: set hard-limit to @-mention notification
expand 0 comments