Closed (fixed)
Project:
Flex Slider
Version:
8.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
23 Mar 2019 at 23:08 UTC
Updated:
30 Jun 2020 at 22:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
daiwik.addweb commented@mcdwayne, Kindly review the attached patch on the same hope this will helps you.
Thanks!
Comment #3
daiwik.addweb commentedComment #5
amateichuk commentedComment #6
jenlamptonI believe that if you add drupal 9 as an option to the
composer.jsonfile in this project you can get a nice little Compatible with Drupal 9 badge in the Project information section on the module page.Below is a code sample from a module that has the badge.
It also looks like it may possible to get the badge by adding the 'core_version_requirement' key in the modules info.yml file, which, in turn, will add the version to the require section of
composer.json. Example follows.Do you want to include the change in this Patch or open a separate issue?
Comment #7
amateichuk commentedComment #8
andralex commentedFigured out that the last commit is now changed to 3daf2116a523a7a6816bb0f22ce495dae0e4fb61 and there are some extra deprecations exists now.
Here is a new patch with fixes of extra deprecations. Skipped libraries_detect(), libraries_get_path() and tests. Going to move them into separate issues to not overweight here.
Comment #9
Steven Brown commentedRan drupal-check against
3daf211. Here are some new errors that are not in the list above.Comment #10
amateichuk commentedComment #11
Sudeepthi Peteti commentedHi,
After applying patch#10, there is still some deprecated code. Changed them.
Also added core_version_requirement as well in info.yml file.
Comment #12
Sudeepthi Peteti commentedComment #13
amaria commentedI will review these patches and change accordingly since some of the changes are taken care of in other tickets.
Comment #15
amaria commentedThis has been committed. I made a few minor changes to fix the messenger calls and removed the library api and test code changes that was done in other tickets. Thanks to all who participated. Note that it will still complain about a couple of library api calls that I left in for backward compatibility until 8.9 becomes the minimum supported version of Drupal.