Allow groups to be invited to topics

Rate limiting is blocking group invitations to topics.

This seems counter-intuitive. Ideally this should be fixed but if not, is there a work-around?

2 Likes

Is this something @techapj could look at? I am not clear exactly what is being reported, can you give repro steps?

1 Like

Yep yep.

Create a group (my example group contains 22 members).
Click Invite on a topic. Invite the group to the topic.

You can’t because of the rate limiting error above.

This is Discourse 1.8.10

2 Likes

I’m assuming that is for an admin account? If so, that should be a bug since rate limits do not apply to admins.

3 Likes

Correct – it’s an admin account.

1 Like

Note, removed #starter-task from here, anything that involves fiddling with rate limiting code can get very hairy.

4 Likes

Interestingly enough, this is not due to rate limiting at all: the invites are rate limited message is hardcoded and will be shown regardless.


The real cause - groups can only be invited to private messages:

https://github.com/discourse/discourse/blob/dc9723904045a5cf1a3d5f55e34fb582abe73214/app/controllers/topics_controller.rb#L469-L475

Do we want to either:

  1. Disable being able to select a group altogether
  2. Allow groups to be invited to topics
5 Likes

This. Inviting groups to topics is a core workflow for many community managers.

4 Likes

Hey @xrav3nz, are you working on this or would still like to? Just want to make sure because I was thinking of taking this up.

4 Likes

:wave: have been a bit inactive, but yeah I’m working on this. Thanks for checking with me.

5 Likes

Here is the PR :wink:

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

I hope I didn’t miss anything…


(sorry for the poor video quality, shouldn’t have saved as 480p…)

7 Likes

Updated the title to reflect that this is a feature request as we’ve never allowed groups to be invited into public topics before. In fact, being able to type a group name in the invite modal of a public topic is actually a bug.

@HAWK Would mentioning a group work for your use case? I’m asking because inviting group in the context of a private message is basically to give the group members access to the private message. The only access control methods we have for public topics are at the category level and I’m not sure if it would be easy to bypass that with invites at the moment.

7 Likes

It’s not elegant and creates noise but yeah, technically it works so not a show stopper.

2 Likes

It would be elegant to be able to invite groups to Topics.

1 Like