back round 5 view raw

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

merged

opened by

boltless.me 1 month ago targeting master from feat/mentions

Signed-off-by: Seongmin Lee git@boltless.me

options

unified split

Changed files

appview

notify

db.go

appview/notify/db/db.go

···

       13
        
       	"tangled.org/core/idresolver"

     

       14
        
       )

     

       15
        
       

     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       16
        
       type databaseNotifier struct {

     

       17
        
       	db  *db.DB

     

       18
        
       	res *idresolver.Resolver

     
···

       421
        
       	issueId *int64,

     

       422
        
       	pullId *int64,

     

       423
        
       ) {

     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       424
        
       	recipientSet := make(map[syntax.DID]struct{})

     

       425
        
       	for _, did := range recipients {

     

       426
        
       		// everybody except actor themselves

···

       13
        
       	"tangled.org/core/idresolver"

     

       14
        
       )

     

       15
        
       

     

       16
       +
       const (

     

       17
       +
       	maxMentions = 5

     

       18
       +
       )

     

       19
       +
       

     

       20
        
       type databaseNotifier struct {

     

       21
        
       	db  *db.DB

     

       22
        
       	res *idresolver.Resolver

     
···

       425
        
       	issueId *int64,

     

       426
        
       	pullId *int64,

     

       427
        
       ) {

     

       428
       +
       	if eventType == models.NotificationTypeUserMentioned && len(recipients) > maxMentions {

     

       429
       +
       		recipients = recipients[:maxMentions]

     

       430
       +
       	}

     

       431
        
       	recipientSet := make(map[syntax.DID]struct{})

     

       432
        
       	for _, did := range recipients {

     

       433
        
       		// everybody except actor themselves