Problem/Motivation

This module is incompatible with D10

Steps to reproduce

composer require drupal/core:^10 drupal/faqfield -W

Proposed resolution

Make this module compatible with D10

Remaining tasks

Update constraint on core in `info.yml` file
Update constraint on jquery_ui_accordion to `^2.0.0`
Update any deprecated / regressed code

CommentFileSizeAuthor
#2 3334987-2.patch3.22 KBnoorulshameera

Issue fork faqfield-3334987

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

percoction created an issue. See original summary.

noorulshameera’s picture

noorulshameera commented
Status: Active » Needs review
StatusFileSize
new 3334987-2.patch3.22 KB

Adding patch with D10 compatibility issues fixed.

juanramonperez made their first commit to this issue’s fork.

juanramonperez’s picture

juanramonperez commented

I sent a commit to the issue fork merging the content of the patch #2. In the patch is missing the update to the composer.json file

Update constraint on jquery_ui_accordion to `^2.0.0`

I tested in a fresh installation of Drupal 10 and is working as expected.

elkaro’s picture

elkaro commented
Assigned: percoction » Unassigned
Status: Needs review » Reviewed & tested by the community

I tested this branch and it works.

If others want to use the branch here is a commit with the changes to composer.json:
https://gitlab.com/drutopia/drutopia_dev_template/-/commit/7e46e0d743dfb...

percoction opened merge request !3

  • heddn committed cd44e524 on 2.0.x authored by noorulshameera 
    Issue #3334987 by juanramonperez, percoction, noorulshameera, heddn: D10...
heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Reviewed & tested by the community » Fixed

Thanks everyone for your contributions.

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented

7.1 release tagged as well.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

heddn closed merge request !3