- Issue created by @catch
- 🇵🇱Poland Graber
Maybe I was a bit too fast with planning of moving the classes sub-module outside of the Drupal LMS repo as it can go out of sync at some point and will require splitting automated tests..
- 🇬🇧United Kingdom catch
Ah yes I was thinking just an optional module in the main repo - at least to start with.
- 🇵🇱Poland Graber
This will be more tricky as it'll break lms_membership_request → module. lms_membership_request → will now need 2 cases: lms_classes installed or not.
We should probably just do it without looking at lms_membership_request and update it when someone else decides to support the initiative and then update it?I can do the work here as it doesn't look very scary but the membership request part may be a different thing.
- 🇬🇧United Kingdom catch
Could lms_membership_request add a hard dependency on lms_classes?
I would think the non-class case would be pretty much raw https://www.drupal.org/project/grequest → or at least not far off? But if there's some extra things we want to add, that becomes a feature request against lms_membership_request rather than a bug then.