Closed
Bug 1560689
Opened 5 years ago
Closed 5 years ago
Remove react-intl dependencies and script loading
Categories
(Firefox :: New Tab Page, task, P1)
Firefox
New Tab Page
Tracking
()
RESOLVED
FIXED
Firefox 69
| Tracking | Status | |
|---|---|---|
| firefox69 | --- | fixed |
People
(Reporter: Mardak, Assigned: Mardak)
References
Details
Attachments
(2 files)
Once we don't import react-intl, we can remove:
- the vendored dependency and license files
- package.json dependency entry
- script loading from aboutNewTabService and html prerendering
|
||
Updated•5 years ago
|
Priority: -- → P1
|
||
Comment 1•5 years ago
|
||
It can also be removed from bin/vendor.js
|
Assignee | |
Comment 2•5 years ago
|
||
The actual package.json dependency can't be removed until we remove all uses, e.g., bug 1556888 for tests and whatever lingering unnecessary injectIntl related calls.
|
|
Assignee | |
Updated•5 years ago
|
Assignee: nobody → edilee
|
||
Updated•5 years ago
|
Component: Activity Streams: Newtab → New Tab Page
|
||
Comment 3•5 years ago
|
||
|
|
Assignee | |
Comment 4•5 years ago
|
||
Depends on D36764
Pushed by elee@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c61360a73028 Remove react-intl dependencies and script loading r=dmose
|
||
Comment 6•5 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox69:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 69
You need to log in
before you can comment on or make changes to this bug.
Description
•