kamleshbp → created an issue.
Patch #5 is working fine for me. The issue was in RTBC rightly. Moving it back to RTBC again.
kamleshbp → made their first commit to this issue’s fork.
kamleshbp → created an issue.
kamleshbp → made their first commit to this issue’s fork.
kamleshbp → created an issue.
kamleshbp → created an issue.
kamleshbp → created an issue.
kamleshbp → created an issue.
Patch #2 created by bot is getting applied successfully. It makes the module D10 compatible. So moving the issue to RTBC.
I am adding a patch as well for making the module Drupal 10 compatible. It includes changes in .info.yml file and some deprecation fixes as per upgrade status module. Kindly review the same.
kamleshbp → created an issue.
The patch #2 created by bot is getting applied successfully. It makes the module D10 compatible and also the functionality of the module is working fine as well. Moving the issue to RTBC.
I tried applying the patch #2 created by bot. It got applied successfully, but when I tried to import submissions from csv file, I got one TypeError (added the screenshot). I think the issue is with the PHP version (On PHP 8.1 it is giving error, but on the lower versions it was giving a warning only.). Anyway, PHP 8.1 is the minimum requirement for Drupal 10, so this issue has to be fixed. I have added the patch for the same. Kindly review.
I tried the patch on comment #8. But it is giving 2 warnings in upgrade_status scan results (Added the image). So to fix it, I am adding the fixes. Access check is very common so that doesn't require any explanation (I guess). But 2nd warning is that the core/underscore will be removed in Drupal 10. There is a throttle function from underscore.js which is being used in frontend.js file. So I have added the alternative for the same as per this. Kindly review the patch.
The patch #2 is getting applied successfully and makes the module D10 compatible. So moving the issue to RTBC.
Patch #15 is getting applied successfully and it is working for me. So moving it to RTBC.
Patch present in comment #5 📌 Automated Drupal 10 compatibility fixes Fixed is getting applied cleanly and it makes module Drupal 10 compatible. So moving the issue to RTBC.
The patch created by bot is getting applied and it is working fine as expected. It makes the module Drupal 10 compatible (checked using Upgrade status module), so moving the issue to RTBC.
Patch #1 is working fine for me and it makes the module Drupal 10 compatible. Moving the issue to RTBC.
I am adding a patch as well for making the module Drupal 10 compatible. It includes changes in .info.yml file and drupal-check fixes.
kamleshbp → created an issue.
Patch #4 is working fine for me and it makes the module drupal 10 compatible. Moving the issue to RTBC.
kamleshbp → created an issue.
I tested patch #4 📌 Drupal 10 compatibility issues Fixed . It has one issue, so I have fixed it and added fixes for some drupal-check issues.