-
Notifications
You must be signed in to change notification settings - Fork 289
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Discourse Integration #363
Comments
I don't think adding users IDs of everyone in the team repo is the right approach: while that's needed occasionally (like Discord and maybe Zulip -- even though I have an idea on how to remove the need for setting a Zulip ID) is a huge churn, and we should avoid that at all costs. While it's more expensive regarding API calls, we can get each user's ID from their email address by calling |
Well what's expensive in this context? Like I don't know how what limits we have on the discourse API. If it's just sub-optimal then I think that approach is fine. |
I'm not aware of any rate limits, and we can tweak them since we're the owners of the instance. |
Neat idea. Teams don't use discourse that much (and when they do their team status isn't as important) so this hasn't been needed as much. But it doesn't hurt! |
I'd like to see this as well. |
is this going on? |
Discourse has the notion of groups which can be given different privacy and permissions to different forums, it would be nice if we could sync our discourse forums to have groups that match the teams in the team repository. The implementation should be a straightforward addition as discourse's API is relatively simple. The most amount of work would probably be getting everyone to add their discourse usernames to the team repo.
The text was updated successfully, but these errors were encountered: