- ๐จ๐ฆCanada joelpittet Vancouver
Weโve returned to this issue as per ๐ Move the work on drupal.org. Enable testing Fixed
The functionality discussed in https://github.com/Gizra/og/issues/254 was originally addressed in https://github.com/Gizra/og/pull/253, which was later closed in favor of https://github.com/Gizra/og/pull/565. However, the current implementation has some issues:
- The code fails when no role_ids are passed in, as
$this->ogMembership->getUserGroupIdsByRoleIds()
requires role_ids to function. - Thereโs a minor bug in the form where
type="text"
should betype="textfield"
for the two relevant options.
To provide a working solution, Iโll repost the code from https://github.com/Gizra/og/pull/253.
- The code fails when no role_ids are passed in, as
- Merge request !21Pull request https://github.com/Gizra/og/pull/253 by tombola โ (Merged) created by joelpittet
-
joelpittet โ
committed d2e668ef on 8.x-1.x
Issue #3037785 by joelpittet, meecect, ramil g, mpp, amitaibu: Port...
-
joelpittet โ
committed d2e668ef on 8.x-1.x
Automatically closed - issue fixed for 2 weeks with no activity.