1046738 - How should we notify the user when feedback data sending fails?

Status: RESOLVED WONTFIX

(Hello (Loop) :: Client, defect)

Description

[Nicolas Perriault (:NiKo`) — needinfo me if you need my attention]

This is related to bug 972992; we don't have UX spec for when sending the feedback data possibly fails. 

- Should we display a notification? Where?
- Should we rather update the "Thank you" screen?
- Should we just log the error to the browser console and keep going?

Comment 1

[Nicolas Perriault (:NiKo`) — needinfo me if you need my attention]

NI :darrin here.

Comment 2

[Romain Testard [:RT]]

Personal opinion I think we should just log the error to the browser console and keep going.
Users know we won't get back to them when they submit feedback and they should not be bothered by error messages if they are kind enough to provide feedback so I think we may want to implement a retry in the background (probably not FF34 though) but we don't want to display an error message to the user.

Comment 3

[Darrin Henein [:darrin]]

Not a bad option to fail silently as RT proposes. If we want to allow a retry we could do something like the attached flow.

Comment 4

[Darrin Henein [:darrin]]

Attachment: feedback-retry.png

Comment 5

[Nicolas Perriault (:NiKo`) — needinfo me if you need my attention]

(In reply to Darrin Henein [:darrin] from comment #4)
> Created attachment 8465600 [details]
> feedback-retry.png

This <3 (personal opinion too).

Comment 6

[Romain Testard [:RT]]

OK, let's implement a silent failure for now as I am not convinced a notification is needed from a user standpoint.
This also seems to have the benefit of involving less work at a time where we should try restricting the scope.
We can revisit later if we find that this is a useful feature.

I am therefore marking this as a client backlog dependency.

Comment 7

[:shell escalante]

changing way we collect feedback to NPS - no longer relevant