Problem/Motivation

Drupal 10 compatibility Issues

Steps to reproduce

1. Install the component module on a Drupal website.
2. Run the update scan with the Upgrade Status Drupal module.
3. Notice all the issues mentioned by Upgrade Status to make the theme compatible.
(Attached is the text file of the issues reported)

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#8 Screenshot 2022-10-19 at 2.57.38 PM.png145.71 KBankitv18
#8 Screenshot 2022-10-19 at 2.57.28 PM.png71.32 KBankitv18
single-export-component-2022-10-13T08_14_40+0000.txt524 bytespanshulk

Issue fork component-3315088

Command icon 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:

About issue forks

Comments

panshulk created an issue. See original summary.

panshulk opened merge request !8

panshulk’s picture

panshulk commented

With this the module is now ready for Drupal 10.
I'm now able to see that the module is working as expected with Drupal 10 latest release. (10.0.0-beta2)

Before merging this & getting it ready for Drupal 10, the roadmap to release 1.0.0 issues needs to be looked into first - https://www.drupal.org/project/component/issues/3285072

panshulk’s picture

panshulk commented
Version: 1.0.0-rc3 » 1.0.x-dev
Assigned: panshulk » Unassigned
panshulk’s picture

panshulk commented
Status: Active » Needs review

vipin.mittal18 made their first commit to this issue’s fork.

ankitv18’s picture

ankitv18 commented
Assigned: Unassigned » ankitv18

I'll review MR!8

ankitv18’s picture

ankitv18 commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Screenshot 2022-10-19 at 2.57.28 PM.png71.32 KB
new Screenshot 2022-10-19 at 2.57.38 PM.png145.71 KB

Hi,
MR!8 patch is applying successfully without any failure, also run the upgrade status tool to check the compatibility of component module.
Attaching the screenshots for your reference.

ankitv18’s picture

ankitv18 commented
Assigned: ankitv18 » Unassigned

vishalkhode’s picture

vishalkhode commented
Status: Reviewed & tested by the community » Needs work

Reopening this ticket as we've ended support for Drupal 8. We can continue provide support for Drupal 8 by adding alternative as mentioned here: https://www.drupal.org/node/3080612

vishalkhode opened merge request !12

vishalkhode’s picture

vishalkhode commented
Status: Needs work » Needs review
deepakkm’s picture

deepakkm commented
Status: Needs review » Reviewed & tested by the community

@vishalkhode MR looks good and works fine on 8,9 and 10 version of Drupal.

vishalkhode opened merge request !13

  • vishalkhode committed c19be9c on 1.0.x 
    Issue #3315088: Added fix to support back Drupal 8.
vishalkhode’s picture

vishalkhode commented
Status: Reviewed & tested by the community » Fixed
vipin.mittal18’s picture

vipin.mittal18
He/Him
Primary language English
Location Greater Noida, India
commented
Status: Fixed » Closed (fixed)