- Issue created by @sneha_surve
- Open on Drupal.org →Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7last update
10 months ago Waiting for branch to pass - last update
10 months ago Build Successful - 🇬🇧United Kingdom james.williams
This assumes the jQuery UI module(s) are installed; I don't think this project currently requires them, or brings them in as dependencies at all. So these library definitions would either need to become dynamic, so that they're only available when those dependencies are in place, or enforce that those dependencies are installed (which would be even if these libraries aren't used).
- Status changed to Needs review
10 months ago 11:32am 9 February 2024 - Status changed to Needs work
10 months ago 11:36am 9 February 2024 - 🇬🇧United Kingdom james.williams
I have reviewed; and the answer is that more work is needed.
- 🇦🇺Australia VladimirAus Brisbane, Australia
Thanks for the patch.
I agree with @james.williams.Needs at least to add dependencies (see this issue 📌 Update jQuery UI Draggable to 2.x Fixed as an example) or replace functionality with core functionality (see this issue 🌱 [Plan] Remove jQuery UI components used by Drupal core and replace with a set of supported solutions Fixed ).
- 🇳🇿New Zealand amanp Poneke
An update for this was included in 3.31.0. See #3427365 → .
There's an open MR-42 to handle ensuring that the dependencies are enabled.