Hello project maintainers,

Creating this issue to help make this module compatible with Drupal 10.

There is already an issue available for this Automated Drupal 10 compatibility fixes which is in fixed state but not resolving the D10 non compatibility issue.

CommentFileSizeAuthor
#3 Afterpatch-Functionality.png31.59 KBchandra gowsalya kannan
#3 Functionality Course enrollments.jpg78.3 KBchandra gowsalya kannan
#3 CourseConfig.jpg71.38 KBchandra gowsalya kannan
#3 Drupal 10 upgrade status.jpg13.88 KBchandra gowsalya kannan
#2 course-drupal-check.txt186.21 KBrahul17
#2 course_d10.3.x-dev-3355135-1.patch16.16 KBrahul17

Comments

rahul1707 created an issue. See original summary.

rahul17’s picture

rahul17 commented
Assigned: rahul17 » Unassigned
Status: Active » Needs review
StatusFileSize
new course_d10.3.x-dev-3355135-1.patch16.16 KB
new course-drupal-check.txt186.21 KB

I have created this patch which makes module Drupal 10 compatible. Upgrade status is showing compatible with D10.

Please review this patch.

chandra gowsalya kannan’s picture

chandra gowsalya kannan commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Drupal 10 upgrade status.jpg13.88 KB
new CourseConfig.jpg71.38 KB
new Functionality Course enrollments.jpg78.3 KB
new Afterpatch-Functionality.png31.59 KB

Applied patch #2 which makes the module D10 compatible. Attached upgrade status screenshot. Before applying the patch, needs to install all dependency modules mentioned in info.yml, then applying patch is working good.
Moving to RTBC !!

djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Status: Reviewed & tested by the community » Needs review

I scheduled some D10 automated tests. If those pass we can commit the changes.

glynster’s picture

glynster commented

@djdevin can we commit and release and update? RTBC!

  • 79ce4330 committed on 3.x 
    Issue #3355135 by rahul17, Chandra Gowsalya Kannan, glynster: Drupal 10...
djdevin’s picture

djdevin
he/him
Primary language English
Location Philadelphia
commented
Status: Needs review » Fixed

Looks good. Thanks!

Status: Fixed » Closed (fixed)

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