Closed
Bug 1273067
Opened 9 years ago
Closed 9 years ago
Upgrade React to 15.0.2 for Loop
Categories
(Hello (Loop) :: Client, defect, P2)
Hello (Loop)
Client
Tracking
(Not tracked)
RESOLVED
FIXED
Iteration:
49.2 - May 23
People
(Reporter: standard8, Assigned: standard8)
References
Details
Attachments
(1 file)
15.0.2 was released a while ago. It turns out it is relatively simple to upgrade.
The React.addons.LinkedStateMixin is now "deprecated due to very low popularity", although there's another component that could replace it, the new one isn't being maintained, and the react docs say to just use onChange handlers.
Also, they've changed the way that the do mixins with the "require" usages. This affects the standalone production build system - we now include the add-on that's required.
I'd have liked to be able to limit the non-production build to just react plus the single add-on, but unfortunately that's not possible at the moment.
|
Assignee | |
Comment 1•9 years ago
|
||
Attachment #8752741 -
Flags: review?(fernando.campo)
Attachment #8752741 -
Flags: review?(edilee)
Attachment #8752741 -
Flags: review?(dcritchley)
|
||
Updated•9 years ago
|
Attachment #8752741 -
Flags: review?(fernando.campo)
Attachment #8752741 -
Flags: review?(edilee)
Attachment #8752741 -
Flags: review?(dcritchley)
Attachment #8752741 -
Flags: review+
|
|
Assignee | |
Comment 2•9 years ago
|
||
https://github.com/mozilla/loop/commit/4a15eeb2447cd3d5cfc0551d85a1b1e25a657f06
https://github.com/mozilla/loop/commit/70e35abdaaceafb7ba225c4ae6f7f93a6052cb4e
Status: NEW → RESOLVED
Iteration: --- → 49.2 - May 23
Closed: 9 years ago
Rank: 23
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•