1900582 - Update Translations ModelState to include ERROR_DELETION when a failure Occurs

Status: RESOLVED FIXED

(Firefox for Android :: Translations, task, P2)

Description

[Olivia Hall [:olivia]]

A download or deletion error may occur on the translations download screen. This state should be reflected in the UI. ERROR_DELETION and likely ERROR_DOWNLOAD should be listed in the ModelState. The reducer should receive this state when it fails in TranslationsMiddleware.

Comment 1

[Olivia Hall [:olivia]]

This bug should additionally ensure translation state clears across page loads to support https://phabricator.services.mozilla.com/D210627. (If it makes sense to do here, I suspect this might need to be filed as a new bug, but I'll research it here.)

Comment 2

[Olivia Hall [:olivia]]

Attachment: Bug 1900582 - Update Android Translations ModelState to include ERROR States

Right now, when an error occurs when downloading or deleting a translations model,
there is nothing to indicate which model had an error.

This patch introduces ERROR_DELETION and ERROR_DOWNLOAD to indicate state.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cb7cbba6de88
Update Android Translations ModelState to include ERROR States r=android-reviewers,giorga

Comment 4

[Aron Cseh]

https://hg.mozilla.org/mozilla-central/rev/cb7cbba6de88