Signed-off-by: Seongmin Lee git@boltless.me
+7
-7
appview/db/db.go
+7
-7
appview/db/db.go
···
1113
1113
return err
1114
1114
})
1115
1115
1116
-
runMigration(conn, logger, "add-usermentioned-preference", func(tx *sql.Tx) error {
1117
-
_, err := tx.Exec(`
1118
-
alter table notification_preferences add column user_mentioned integer not null default 1;
1119
-
`)
1120
-
return err
1121
-
})
1122
-
1123
1116
return &DB{
1124
1117
db,
1125
1118
logger,
1126
1119
1127
1120
1121
+
return err
1122
+
})
1123
+
1124
+
runMigration(conn, logger, "add-usermentioned-preference", func(tx *sql.Tx) error {
1125
+
_, err := tx.Exec(`
1126
+
alter table notification_preferences add column user_mentioned integer not null default 1;
1127
+
`)
1128
1128
return err
1129
1129
})
1130
1130
appview/db/notifications.go
appview/db/notifications.go
This patch was likely rebased, as context lines do not match.
appview/models/notifications.go
appview/models/notifications.go
This file has not been changed.
appview/pages/templates/user/settings/notifications.html
appview/pages/templates/user/settings/notifications.html
This file has not been changed.
appview/settings/settings.go
appview/settings/settings.go
This file has not been changed.
History
9 rounds
1 comment
boltless.me
submitted
#8
1 commit
expand
collapse
appview: add
user_mentioned notification preference option
Signed-off-by: Seongmin Lee <git@boltless.me>
3/3 success
expand
collapse
expand 0 comments
pull request successfully merged
boltless.me
submitted
#7
1 commit
expand
collapse
appview: add
user_mentioned notification preference option
Signed-off-by: Seongmin Lee <git@boltless.me>
3/3 success
expand
collapse
expand 0 comments
boltless.me
submitted
#6
1 commit
expand
collapse
appview: add
user_mentioned notification preference option
Signed-off-by: Seongmin Lee <git@boltless.me>
2/3 failed, 1/3 success
expand
collapse
expand 0 comments
boltless.me
submitted
#5
1 commit
expand
collapse
appview: add
user_mentioned notification preference option
Signed-off-by: Seongmin Lee <git@boltless.me>
2/3 failed, 1/3 running
expand
collapse
expand 0 comments
boltless.me
submitted
#4
1 commit
expand
collapse
appview: add
user_mentioned notification preference option
Signed-off-by: Seongmin Lee <git@boltless.me>
1/3 failed, 2/3 timeout
expand
collapse
expand 0 comments
boltless.me
submitted
#3
1 commit
expand
collapse
appview: add
user_mentioned notification preference option
Signed-off-by: Seongmin Lee <git@boltless.me>
3/3 success
expand
collapse
expand 0 comments
boltless.me
submitted
#2
1 commit
expand
collapse
appview: add
user_mentioned notification preference option
Signed-off-by: Seongmin Lee <git@boltless.me>
3/3 success
expand
collapse
expand 0 comments
boltless.me
submitted
#1
1 commit
expand
collapse
appview: add
user_mentioned notification preference option
Signed-off-by: Seongmin Lee <git@boltless.me>
3/3 success
expand
collapse
expand 1 comment
boltless.me
submitted
#0
1 commit
expand
collapse
appview: add
user_mentioned notification preference option
Signed-off-by: Seongmin Lee <git@boltless.me>
stack lgtm, can we make the modification to notify only upto 5 users per comment? the rest should be ignored. could be useful as a spam prevention technique.