Skip to content
Snippets Groups Projects

Add desktop notifications support to the cli client

Add desktop notifications support to the cli client
Anne/Notifications into project/channels
1 open thread
Open
Sydney Anne Ericksonrequested to merge
Anne/Notifications into project/channels
1 open thread

Fun little change to make the cli client more usable

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
client/channels.go
170 171 message := usernameField + " " + timestampField + "\n" + messageField
171 172
172 173 c.ReceivedMsgChan <- message
174
175 err := beeep.Notify(senderUsername, strings.TrimSpace(text), "icon.png")
  • Jono Wenger
    Jono Wenger @jonow
    Maintainer

    Put it in a go func and leave a comment about what it is and why its there

    Also, does icon.png need to be included with the binary? If so, you need to look how you can embed it.

    Edited by Jono Wenger
  • Please register or sign in to reply
Please register or sign in to reply