Closed
Bug 1872703
Opened 1 year ago
Closed 1 year ago
AC Add Error Support Connection to API Calls
Categories
(Firefox for Android :: Translations, enhancement, P2)
Tracking
()
RESOLVED
FIXED
123 Branch
| Tracking | Status | |
|---|---|---|
| firefox123 | --- | fixed |
People
(Reporter: olivia, Assigned: olivia)
References
Details
(Whiteboard: [fxdroid][foundation])
Attachments
(1 file)
This bug is to build on bug 1862263 to intercept the error as it exits GeckoView and ensure a TranslationError throwable is called. For example, a translation API call such as this one, should be intercepted to map to a TranslationError to ensure we do not cross the Gecko abstraction layer elsewhere when handling errors.
More background in this conversation.
| Assignee | ||
Updated•1 year ago
|
Severity: -- → N/A
Priority: -- → P2
| Assignee | ||
Updated•1 year ago
|
Blocks: fx-translation-android
Whiteboard: [fxdroid][foundation]
Updated•1 year ago
|
| Assignee | ||
Comment 1•1 year ago
|
||
Should also use this bug to adjust the utility function (fromGeckoErrorToTranslationError) for mapping to something to fit in with the Kotlin paradigm:
fun TranslationsException.intoTranslationError(): TranslationError {
return when (code) {
// ...
}
}
See this discussion for more details on what to adjust.
Comment 2•1 year ago
|
||
| Assignee | ||
Updated•1 year ago
|
Assignee: nobody → ohall
Comment 3•1 year ago
|
||
Authored by https://github.com/ohall-m
https://github.com/mozilla-mobile/firefox-android/commit/48c24fc63985ef1edb467b46f8617f815d08a452
[main] Bug 1872703 - Named Translation Errors
Status: NEW → RESOLVED
Closed: 1 year ago
status-firefox123:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 123 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•