| Issue category | Task because we should update all third party assests/libraries upto date. |
|---|---|
| Issue priority | Normal |
Problem/Motivation
Backbone Current version 1.1.2
New version 1.2.1
http://backbonejs.org/#changelog
https://github.com/jashkenas/backbone/
Proposed resolution
Patch attached
Remaining tasks
Needs Reviewers eyes.
User interface changes
API changes
Beta phase evaluation
| Comment | File | Size | Author |
|---|---|---|---|
| #22 | backbone-upgrade-2494063-22.patch | 177.41 KB | dcmul |
Comments
Comment #1
tohesi commentedComment #2
droplet commentedBig changes. Applying patch, top right edit button disappeared
Comment #3
nod_Yeah their changelog is massive it's been more than a year since their last realease: http://backbonejs.org/#changelog
Too bad jessebeach is not around anymore.
It's a novice issue in the sense that a backbone dev should find it relatively easy to fix.
Comment #4
droplet commentedthis issue: https://github.com/jashkenas/backbone/issues/3611
Only 2 places used this pattern, we can do code refactoring before upstream fixing. It's even more readability.
Comment #5
nod_Need to add the unminified backbone source too (for the map file).
Comment #6
nod_Comment #7
chananapeeyush commentedUpdated issue summary.
Comment #8
tohesi commentedVersion 1.2.1 already out. Updated files, issue title & summary accordingly.
Comment #10
tohesi commentedForgot to update the test.
Comment #11
tohesi commentedComment #14
tohesi commentedComment #15
nod_Contextual links are working, toolbar, quickedit, ckeditor and the tour module as well.
Comment #18
naveenvalechaSetting RTBC :) As already by @nod_, Also Tested patch locally,Ckeditor, toolbar is working fine :)
Comment #20
dcmul commented@nod_ should we not look into the reasons why the tests are failing before RTBC? or is this an exception.
Comment #21
dcmul commentedpatch failing
Comment #22
dcmul commentedI have done a re-roll.
Comment #23
daffie commented@dcmul: Can you tell me what you changed in the reroll. If you are able to please add an interdiff.txt.
We do not need this file. We have backbone-min.js and backbone-min.map for that.
Comment #24
nod_No we need it, the map file is useless without the original source. Needs to be in the patch.
Comment #25
daffie commentedOK I nod_ says we need the file then we need the file. To RTBC.
Comment #28
daffie commentedBack to RTBC.
Comment #30
webchickCommitted and pushed to 8.0.x. Thanks!
Thanks also for the manual testing. Always spooky when updating these JS libs. :)