- ๐บ๐ธUnited States dobe
Add a pull request at github for this issue: https://github.com/Gizra/og/pull/755
Attaching diff so I can use in composer.
- ๐บ๐ธUnited States loze Los Angeles
This will only work if your groups are nodes. Ive been using the code here which will work for any entity that is enabled as a group
https://github.com/Gizra/og/issues/752 - ๐บ๐ธUnited States dobe
@loze create a patch and post it here. Or a pull request at the github, either way it doesn't help if it just a comment it basically got lost when this should be a part of the module.
- Status changed to Needs review
about 2 years ago 10:59pm 12 March 2023 - ๐บ๐ธUnited States loze Los Angeles
Added a pull request here
https://github.com/Gizra/og/pull/756 - Merge request !12Issue #3306494 Add views relationship to og_membership from all group enabled entities. โ (Open) created by loze
- ๐บ๐ธUnited States loze Los Angeles
MR12 adds views relationships to og_memberships from any group enabled entity types.
This allows you to create a view of all groups a user is a member of or show the membership counts for a list of groups.
- Status changed to Needs work
9 months ago 7:23am 12 September 2024 - ๐ท๐ดRomania claudiu.cristea Arad ๐ท๐ด
This is a nice addition. Could you, please, add a test to prove it works? Setting to "Needs work" because needs test coverage.
- ๐บ๐ธUnited States loze Los Angeles
Thanks @claudiu.cristea. Unfortunately, I have been unable to wrap my head around how to write tests. I've tried on several projects but can't seem to grasp it.
Hopefully someone else interested in this feature can help out with that.
- First commit to issue fork.
- ๐จ๐ฆCanada joelpittet Vancouver
I did a related test over here, have a look maybe it can be piggybacked on โจ Add views relationship to group entities from og_membership entities Needs work
- ๐จ๐ฆCanada joelpittet Vancouver
Added a kernel test that exercises the new logic using
node
entities. I picked nodes because theyโre the most common OG group type and we already haveentity_test
coverage elsewhere, so this gives us a bit of variety.
Happy to swap it toentity_test
if youโd prefer uniformityโjust let me know.
Test bot is green ๐คโ ! - ๐จ๐ฆCanada joelpittet Vancouver
Need a second pair of eyes to confirm it does what it needs to before committing. So setting to Needs Review.