Closed
Bug 1592209
Opened 5 years ago
Closed 5 years ago
Investigate either removing or renaming engineName in the modern configuration for Search
Categories
(Firefox :: Search, task, P2)
Firefox
Search
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox72 | --- | fixed |
People
(Reporter: standard8, Assigned: standard8)
References
Details
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
The modern configuration for search currently calls for engineName. However, this is more just a random identifier that makes it easy for readers to identify the engine.
In discussions with Mike (de Boer), we realised that this might not actually be necessary. For example, on the Remote Settings server, we could just use webExtension.id as the identifier.
If we do need a separate identifier field, then we should call it engineId to be clearer as to what it is relating to.
|
Assignee | |
Updated•5 years ago
|
Assignee: nobody → standard8
Status: NEW → ASSIGNED
Iteration: --- → 72.2 - Nov 4 - 17
|
|
Assignee | |
Comment 1•5 years ago
|
||
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/acacc410970f
Remove engineName from the modern configuration. r=mikedeboer
|
||
Comment 3•5 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox72:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 72
You need to log in
before you can comment on or make changes to this bug.
Description
•