Group owners cannot see group if "Group is visible to all users" is off

When working to add custom styling via CSS, we had to make a custom group for the users. To help manage the group going forward, all moderators were added to the group as owners. However, the moderators aren’t able to see the group on the /groups page, despite being owners.

3 Likes

Any ideas here @tgxworld?

1 Like

A lot of the group features are tied to the group being visible to the public.

I have the opposite problem from @jomaxro in that I have groups available to the public (e.g., for directories of users) but I don’t want the groups to be visible directly on the /groups page.

One solution for both issues would be to have a setting on each individual group for “show on Groups page”.

This feature could still respect the permissions for accessing non-public groups. Users could see that were was a group on the /group page, but if they select the group, it could show the permission/error screen unless they are a member of the group.

How would that solve my issue? I don’t want it visible to the general community - I only want there to be an easy way for owners to see their own groups.

Ok. So your requirement would be:

  • show non-public groups on the group page (but only to group members)

To cover the scenario of not showing public groups, how about also:

  • add setting to individual group page for hide on group page

Groups can be used for a lot of different reasons (permissions, events, directories, badges, etc.) so suppressing a group from the /groups page would be a nice option to have.

No. I only want it visible to group owners - the people who need the ability to edit the group.

Ok. My bad. I read “owners” in your original post but then thought it would be useful for other group members to find their non-public groups too. So not technically not part of “your requirement.”

How about restating as follows:

  • list public and non-public groups on the /groups page
  • the label for the non-public group could be “Restricted” unless the user is a member in which case the label would include the existing membership label
  • add setting to individual group page for hide on groups page (will remain visible for group owners) (default to yes for non-public)
  • setting would apply for public and non-public groups

I realize we are solving for slightly different issues, so forgive my hijacking your requirements.

However, if we are going to address the logic of what shows up on the /groups page, I’d like to consider the use case of suppressing public groups from that page. The above get us there with the exception for always showing group owners their groups.

I have to disagree that your request is a bug - seems to me like a feature request. All I’m looking to see is for owners to be able to access and edit groups they own - which is currently only possible by knowing the exact URL, or being a part of said group. Your features seem to expand the group page more (which I’m not against), but doesn’t seem to be a bug (“A bug report means something is broken, preventing normal/typical use of Discourse.”)

I believe the /groups page was initially spec’d out for public groups only. It was not a “my groups” page (which would be nice by the way).

I fully support your request since it would make group administration easier for group owners (non-admins). I will leave the bug vs. feature decision for others.

Technically group owners have to be members of the group. I wish that were not true, but I believe that is how it works now. The group slugs are listed on the owner/member’s profile page. Not a convenient way to find a group at all but technically there.

Happy to split my feature request off, but if we are looking at the logic of the page, I would like to add some flexibilty.

My additional request also let’s non-owners find their groups (and the display of those groups can be turned off if needed).

1 Like

This bug is valid, we show all groups to admin so it makes sense to show groups that the user is an owner of as well.

4 Likes

Ok I have a PR up at

https://github.com/discourse/discourse/pull/4686

5 Likes