Hi

I see that there was an 8.1 version of this module. will this work with Recurly 5.0.1 to allow users to be granted roles based on Recurly plan?

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:

Comments

noopal created an issue. See original summary.

eojthebrave’s picture

This module hasn't been updated for Drupal 9/10 and off hand I'm not sure what that would require. However, probably not much. The Recurly 5.x module still invokes the same hook that this module uses to do it's job, so I would think that if you update this modules code for Drupal 9/10 it would work fine with the current version of the Recurly module. And I would be happy to give over maintainership of this module to anyone else who wants to actively work on updating it. It's not something I'm using anymore so haven't had the time to work on it. I do still use (and maintain) the main Recurly module though so I'm available to help answer questions etc. Just don't have the space to update this one myself.

aaron.ferris’s picture

I'll happily look to get the 8.x branch updated for D10 and general maintenance @eojthebrave if you want to make me co-maintainer?

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

shivam_tiwari’s picture

eojthebrave’s picture

Hi @aaron.ferris. I've added you as a maintainer. If you've got questions about the project at all feel free to ping me in Slack. Thank you!

aaron.ferris’s picture

Will do, thanks @eojthebrave

aaron.ferris’s picture

 [notice] Processing /var/www/html/web/modules/contrib/recurly_roles-3436806.

================================================================================
Recurly Roles, --
Scanned on Tue, 04/09/2024 - 15:32

FILE: web/modules/contrib/recurly_roles-3436806/recurly_roles.module

STATUS         LINE                           MESSAGE                           
--------------------------------------------------------------------------------
Fix later      99   Call to deprecated function user_role_names(). Deprecated in
                    drupal:10.2.0 and is removed from drupal:11.0.0. Use        
                    Drupal\user\Entity\Role::loadMultiple() and, if necessary,  
                    an inline implementation instead.                           
--------------------------------------------------------------------------------

FILE: web/modules/contrib/recurly_roles-3436806/recurly_roles.info.yml

STATUS         LINE                           MESSAGE                           
--------------------------------------------------------------------------------
Check manually 0    Value of core_version_requirement: ^9 || ^10 is not         
                    compatible with the next major version of Drupal core. See  
                    https://drupal.org/node/3070687.                            
--------------------------------------------------------------------------------

I think this is OK as is, ill raise a separate task to handle the D11 changes that can be handled now/later.

Thanks @shivam_tiwari

aaron.ferris’s picture

aaron.ferris’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

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