Closed
Bug 1897785
Opened 11 months ago
Closed 11 months ago
Update API on `TranslationsAction.OperationRequestedAction` To Accept an Optional TabID
Categories
(Firefox for Android :: Translations, task, P2)
Tracking
()
RESOLVED
FIXED
128 Branch
Tracking | Status | |
---|---|---|
firefox128 | --- | fixed |
People
(Reporter: olivia, Assigned: olivia)
References
Details
(Whiteboard: [fxdroid][foundation][translations:128])
Attachments
(1 file)
For the series TranslationsAction.OperationRequestedAction
, requires a tab ID. This is only a requirement for the page settings API. This change is to make tabId fully optional and for page settings, as a sensible default, to use the current tab and issue a warning.
Assignee | ||
Updated•11 months ago
|
Severity: -- → N/A
Type: defect → task
Component: Accounts and Sync → Translations
Priority: -- → P2
Updated•11 months ago
|
Assignee | ||
Comment 1•11 months ago
|
||
Most calls under OperationRequestedAction
do not require a tabId because they change
information on the global store and not the session store.
- This patch makes
tabId
an optional forOperationRequestedAction
- For the one call,
FETCH_PAGE_LANGUAGES
, that does require a tabId a
sensible default of the current tab will be used and a warning will be logged.
Depends on D210562
Updated•11 months ago
|
Assignee: nobody → ohall
Status: NEW → ASSIGNED
Pushed by ohall@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0c561b26d0e8
Update Translations `OperationRequestedAction` to Not Require a Tab ID r=android-reviewers,giorga
Comment 3•11 months ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 11 months ago
status-firefox128:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 128 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•