Hello!
When are you going to change the *.info.yml for Drupal 9?
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | sliderwidget-Drupal_9-10_support-3157814-6-8.x.patch | 685 bytes | dunebl |
| #4 | sliderwidget-Drupal_9_support-3157814-4-8.x.patch | 679 bytes | dunebl |
| #3 | 3157814-3.patch | 307 bytes | ankithashetty |
Issue fork sliderwidget-3157814
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
andypostComment #3
ankithashettyUpdated
sliderwidget.info.ymlwithcore_version_requirement. Please review.Thank you!
Comment #4
duneblI have updated this patch with an added dependency to the
jquery_ui_slidermodule because the jquery is deprecated in D9.=>I have also changed the use of the
core/jquery.ui.sliderintojquery_ui_slider/slidersee #3168263: Deprecated library: The "core/jquery.ui.slider" asset library is deprecated in drupal:8.8.0 and is removed from drupal:9.0.0
Maybe we could create a special version for D9 due to this new dependency.
Comment #5
Zenovii_Kholyavka commentedReviewed and tested this patch, it works correctly.
Comment #6
eliechoufani commentedIf you're using Composer with Drupal 9 and want to apply this patch via Composer :
Add this to the repositories entry in your composer.json file :
Before this :
Then add this line in the require entry in composer.json also : "drupal/sliderwidget": "2.0"
After that, run composer update drupal/sliderwidget and composer install
Composer will download the module and apply the patch directly.
Comment #8
batonac commentedCan confirm that the a combo of #4 and #5 works. Hope to see this pushed out soon!
Comment #9
maxilein commentedIs this part of the dev?
Comment #10
codechefmarc#6 was perfect for me, but I also had to add in the patch itself via the composer patches. Here is what else I added under the "extra" section of my composer.json:
Comment #11
maxilein commentedshould #6 and #10 not be part of a patch that gets invoked by "composer require 'drupal/sliderwidget:^2.0'" ?
Seems to be a lot of manual extras..
Comment #12
wouters_f commentedThis may be merged.
The flow as mentioned above:
Require the dependency:
composer require drupal/jquery_ui_sliderAdd repository
Add Patch
and then
composer require drupal/sliderwidgetTested and works nice!
Happy 2022 to all! 🎇
Comment #13
duneblHere is the patch for D10
Comment #14
duneblComment #16
afagiolihttps://www.drupal.org/project/sliderwidget/releases/8.x-3.0 supports Drupal 10
Thanks!
Comment #17
afagioliComment #18
afagioliComment #19
afagioli