Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notification spam for related actions #53

Open
dgw opened this issue Jan 2, 2020 · 1 comment
Open

Notification spam for related actions #53

dgw opened this issue Jan 2, 2020 · 1 comment
Milestone

Comments

@dgw
Copy link
Member

dgw commented Jan 2, 2020

In an ideal world, this plugin would output only one message for situations like these examples:

  • new issue/PR opened with labels/milestone/assignee (others?) already set
    • target output: a single unified presentation of all details pertaining to the new issue/PR, on one line (though it may use max_messages to have Sopel split a long line)
  • milestone removed, then another added
    • target output: a single "updated the milestone" message
  • multiple labels added/removed in quick succession
    • target output: a single message listing the labels added and removed

I think these fairly common occurrences can be built with future expandability in mind. Coalescing notifications like this should happen only for events on the same issue/PR from the same user.

@dgw dgw added the enhancement label Jan 2, 2020
@dgw dgw added this to the 0.4.0 milestone Jan 2, 2020
@dgw dgw modified the milestones: 0.4.0, 0.5.0 Jun 27, 2020
@dgw
Copy link
Member Author

dgw commented Jun 27, 2020

New thought: This seems like it'll almost deserve its own release due to how much of the webhook stuff will need to be rewritten.

@dgw dgw modified the milestones: 0.5.0, 0.6.0 Jun 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant