-
Notifications
You must be signed in to change notification settings - Fork 10
Conversation
"wikimedia/composer-merge-plugin": "~1.3" | ||
"wikimedia/composer-merge-plugin": "~1.3", | ||
"php-amqplib/php-amqplib": "^2.6", | ||
"solarium/solarium": "^3.7" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Saphyel is this being added as a patch as well? hard to find this since so many files changed :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pvhee for avoid that crappiness, I split this into 3 commits, check the last one in order to see that patch! :D
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, had missed this!
@gusliedke @AndyEPhipps @Heleen-cr this core update also updates the followign JS libs :
Can you verify that our JS code will work correctly with these updated libs? Not worried about CKEditor but we should definitely verify jQuery and Twig |
@pvhee will check this out locally, I can't see the jQuery update borking anything really |
@AndyEPhipps you can also use the platform.sh url to just review functionality that relies on jquery |
@pvhee : tested on plat.sh env, all seems well; custom ESU and nav JS all fine, Panelizer stuff fine, Twig seems to be fine (no broken styles, so all functions/logic working as before). Have done the same on the RND17 test env will the same outcome |
Fixes https://jira.comicrelief.com/browse/PLAT-536 See changelog for 8.2.x
Manual deployment steps
update-core
in RND17