1502805 - Integrate browser notifications

Status: NEW

(Tree Management :: Perfherder, enhancement, P3)

Description

[Ionuț Goldan [:igoldan]]

Browser notifications should be enabled, so that whenever a new perf alerts shows up, we can quickly investigate.

Ideally, this should be configurable, because we only investigate 3 alter categories out of 12. If we cannot filter the desired alert types, this could become annoying right from the start.

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

I don't view this as a priority- possibly there are other use cases I am not aware of.  I find that this could be a can of worms if we get a new alert vs an alert summary- and when we nudge we will get new alert summaries.  When I am actively sheriffing, I want to focus on my alerts, not the long list of new incoming alerts that I am getting notified about.

I see some value in this, assuming you are at your browser and not focused on other stuff.  It would be cool to have notifications for a specific test and a developer could sign up as well :)

Is this a lot of development time?  I don't know and I assume it would be a lot, but if it is little, then doing this is lower risk.  I see a lot of other higher priority features to add to perfherder though.

Comment 2

[xuan pan]

Hello, I am an outreachy 2020 intern candidate and I would like to work on this bug if possible. Thank you.

Comment 3

[xuan pan]

(In reply to xuan pan from comment #2)

Hello Kimberly,
I am Pan and I am one of the outreachy 2020 intern candidate. I would like to work on this bug if possible. Thank you.

Comment 4

[Kimberly Sereduck :kimberlythegeek]

(In reply to xuan pan from comment #2)

Hello, I am an outreachy 2020 intern candidate and I would like to work on this bug if possible. Thank you.

Hi there! Yes, I can assign this to you.

Comment 5

[xuan pan]

Hello,I am looking at making this enhancement. I have some doubts about it. Would this be best implemented with some kind of drop down in the header, or a new page? How much configuration should there be? Hope you guys can help me out.

Comment 6

[Kimberly Sereduck :kimberlythegeek]

(In reply to xuan pan from comment #5)

Hello,I am looking at making this enhancement. I have some doubts about it. Would this be best implemented with some kind of drop down in the header, or a new page? How much configuration should there be? Hope you guys can help me out.

I believe what Ionut is referring to is notifications to the browser itself. I've never worked with these notifications, but you see them on some websites, i.e., "Facebook.com would like permission to send you desktop notifications"

I could be mistaken, Ionut, is this what you had in mind?

Comment 7

[Ionuț Goldan [:igoldan]]

(In reply to Kimberly Sereduck :kimberlythegeek from comment #6)

(In reply to xuan pan from comment #5)

Hello,I am looking at making this enhancement. I have some doubts about it. Would this be best implemented with some kind of drop down in the header, or a new page? How much configuration should there be? Hope you guys can help me out.

I believe what Ionut is referring to is notifications to the browser itself. I've never worked with these notifications, but you see them on some websites, i.e., "Facebook.com would like permission to send you desktop notifications"

I could be mistaken, Ionut, is this what you had in mind?

Yep, that's right.

Comment 8

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/6253

Comment 9

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Beatrice, do you think this would be a useful feature? I believe the original intention was for sheriffs to be notified in the browser when there's a new alert summary. The (now closed) patch looks more like it allows users to watch specific alert summaries for changes.

Comment 10

[Acasandrei Beatrice (needinfo me)]

I believe this feature may not currently provide significant value for sheriffs. It could be a nice to have in the future, but I wouldn't prioritize it at this time.