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

New option: Sent messages only about open/new/close change status #2

Open
psaavedra opened this issue Sep 5, 2016 · 3 comments
Open
Assignees
Milestone

Comments

@psaavedra
Copy link

No description provided.

@aperezdc aperezdc self-assigned this Sep 5, 2016
@aperezdc aperezdc modified the milestone: v0.3 Sep 5, 2016
@psaavedra
Copy link
Author

Rethinking about this. The plugin could take in consideration the Priority field of the ticket.

For example:

  • Send updates for issues = boolean
  • Send updates for issues minimum priority = high|medium| ...
  • Send messages for new issue = boolean
  • Send messages for closed issue = boolean

@aperezdc
Copy link
Owner

aperezdc commented Sep 6, 2016

@psaavedra: I'm going to add the following settings:

notify = new,closed,changed
notify_updated_if = severity:high:highest, priority:blocker

The idea is to allow specifying any field in notify_updated_if. The above example would send notifications for updated tickets if severity is one of high or highest or the priority field is blocker. For example, the following would be also possible:

notify_updated_if = component:super-important-thing

How does this sound?

@psaavedra
Copy link
Author

Sounds really good

El 6/9/2016 6:52 p. m., "Adrian Perez" notifications@github.com escribió:

@psaavedra https://github.com/psaavedra: I'm going to add the following
settings:

notify = new,closed,updatednotify_updated_if = severity:high:highest, priority:blocker

The idea is to allow specifying any field in notify_updated_if. The above
example would send notifications for updated tickets if severity is one
of high or highest or the priority field is blocker. For example, the
following would be also possible:

notify_updated_if = component:super-important-thing

How dows this sound?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#2 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXABs282aUoyKns7WtlszWy-bz82macks5qnZpMgaJpZM4J09-M
.

aperezdc added a commit that referenced this issue Sep 6, 2016
This implements the a new setting which allows specifying a list of ticket
events which cause notifications to be sent:

  [webhook]
  ...
  notify = created,closed

When this setting is empty, all state changes are notified.

This implements part of the changes suggested in issue #2.
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

2 participants