Skip to content
This repository has been archived by the owner on Jul 11, 2022. It is now read-only.

Repository Mute Filters #21

Merged
merged 31 commits into from
Jan 9, 2017
Merged

Repository Mute Filters #21

merged 31 commits into from
Jan 9, 2017

Conversation

andrewhathaway
Copy link
Member

@andrewhathaway andrewhathaway commented Jan 8, 2017

This PR implements per-account per-repository mute filters (#10). This enables users to mute a repositories notifications by Subject Type and Reason. To create or edit a mute filter for a repository, you right click on the repo in the filter list and hit Edit Mute Filter.

image

Clicking Edit Mute Filter will take you to the following window where you can switch off, or on, certain Reasons or Subject Types. You can also delete the mute filter entirely using the X on the top right. Keep will show you notifications for that Reason or Subject Type, whereas Ignore will remove them from your view.

image

This PR also adds the following section to an accounts settings. Each button will take you to the page above, per repository.

image

Notifications that are muted are still ingested by Hawk Eye, but never shown to the user. This is so that if you decide to un-mute certain notifications they will become instantly available. The same goes for altering your mute filters, they will disappear when you edit your mute filter. Sounds will not be raised for notifications that have been muted.

The Settings design for this isn't the best it could be, by a long shot, but I think we can put in a separate PR to alter the structure of that.

…o allowedReasons as allowReasons was a typo!
@bterlson
Copy link

bterlson commented Jan 9, 2017

Looks like a great start, would definitely go through my repos and toggle a few of these!

Might be good to translate the reasons into more easily understandable scenarios. I don't know off hand what manual means, at least.

@andrewhathaway andrewhathaway self-assigned this Jan 9, 2017
@andrewhathaway
Copy link
Member Author

@bterlson Great point. I have changed Subscribed across the whole UI to "Watching", and Manual to "Watching Thread". I used https://developer.github.com/v3/activity/notifications/ to work out any changes here. I think the rest make sense?

I'm happy for this to go ahead and possibly release an initial version of this?

@bterlson
Copy link

bterlson commented Jan 9, 2017

I guess "Authored" could be more clear as "My issue" or something similar? But Manual was definitely the most confusing one.

You either have to ship it or send me a build ;)

@andrewhathaway
Copy link
Member Author

@bterlson The issue there is that we'd need to separate it out to My Issue, My Pull Request and My Commit for that to make sense. It also wouldn't fit the wording of the others? I'm sure this is something we can look at in the future, but I'm going to merge this PR and prep a build!

@andrewhathaway andrewhathaway merged commit 546e202 into master Jan 9, 2017
@andrewhathaway andrewhathaway deleted the mute-filter branch January 9, 2017 19:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants