Skip to content
GitLab
Explore
Sign in
Primary navigation
Search or go to…
Project
notifications-bot
Manage
Activity
Members
Labels
Code
Merge requests
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Deploy
Releases
Package registry
Model registry
Operate
Terraform modules
Analyze
Contributor analytics
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
GitLab community forum
Contribute to GitLab
Provide feedback
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
archives
notifications-bot
Merge requests
!27
Gofunc for notify, set toSend properly, fix unsent handling
Code
Review changes
Check out branch
Download
Patches
Plain diff
Expand sidebar
Merged
Gofunc for notify, set toSend properly, fix unsent handling
hotfix/notification-list
into
release
Overview
0
Commits
2
Pipelines
0
Changes
1
Merged
Gofunc for notify, set toSend properly, fix unsent handling
Jonah Husson
requested to merge
hotfix/notification-list
into
release
Jan 19, 2022
Overview
0
Commits
2
Pipelines
0
Changes
1
0
0
Merge request reports
Viewing commit
7bb3878f
Prev
Next
Show latest version
1 file
+
4
−
5
Inline
Compare changes
Side-by-side
Inline
Show whitespace changes
Show one file at a time
7bb3878f
Fix go func to be pretty
· 7bb3878f
Jonah Husson
authored
Jan 19, 2022
notifications/notifications.go
+
4
−
5
View file @ 7bb3878f
Edit in single-file editor
Open in Web IDE
Show full file
@@ -206,11 +206,10 @@ func (nb *Impl) SendBatch(data map[int64][]*notifications.Data) ([]*notification
if
err
!=
nil
{
return
nil
,
errors
.
WithMessage
(
err
,
"Failed to get list of tokens to notify"
)
}
for
_
,
n
:=
range
toNotify
{
tn
:=
n
go
func
()
{
nb
.
notify
(
csvs
[
tn
.
EphemeralId
],
tn
)
}()
for
i
:=
range
toNotify
{
go
func
(
res
storage
.
GTNResult
)
{
nb
.
notify
(
csvs
[
res
.
EphemeralId
],
res
)
}(
toNotify
[
i
])
}
return
unsent
,
nil
}
Loading