1658845 - Allow users to see their own queue for their webhooks

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Extensions, enhancement)

Description

[Lisset Cuevas]

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:79.0) Gecko/20100101 Firefox/79.0

Expected results:

Allow users to see their own queue for their webhooks so they can see any errors that are occurring and possibly figure out why.

Comment 1

[David Lawrence [:dkl]]

On the preferences tab for webhooks, we can add this as a button or link to the back log page if the user has any webhooks with errors. If there are no errors, then do not display the link/button.

Comment 2

[David Lawrence [:dkl]]

(In reply to David Lawrence [:dkl] from comment #1)

On the preferences tab for webhooks, we can add this as a button or link to the back log page if the user has any webhooks with errors. If there are no errors, then do not display the link/button.

Could even be a new column in the current webhooks table that has a link for the specific webhook to the error log, otherwise nothing. Extra points if we can show the number of current errors as part of the link.

Comment 3

[Lisset Cuevas]

(In reply to David Lawrence [:dkl] from comment #2)

Could even be a new column in the current webhooks table that has a link for the specific webhook to the error log, otherwise nothing. Extra points if we can show the number of current errors as part of the link.

I'm working on this in this way, and about the queues page, I think it should only show the messages queued and the delete button, because if we use the original push queue view, it shows the message payload but I think it could confuse the user, because the payload sent it's a bit different of the original push message.

Comment 4

[Lisset Cuevas]

Attachment: image.png

I mean like this, what do you think?

Comment 5

[David Lawrence [:dkl]]

(In reply to Lisset Cuevas from comment #4)

Created attachment 9171006 [details]
image.png

I mean like this, what do you think?

Looks fine to me.

Comment 6

[Lisset Cuevas]

Attachment: GitHub Pull Request

Comment 7

[David Lawrence [:dkl]]

Merged to master.