Skip to content
This repository was archived by the owner on Oct 30, 2025. It is now read-only.

[FIX] Migrate Github oauth to Github apps#99

Merged
murtaza98 merged 6 commits intoRocketChat:developfrom
dhruvjain99:fine-grained-access
Mar 15, 2021
Merged

[FIX] Migrate Github oauth to Github apps#99
murtaza98 merged 6 commits intoRocketChat:developfrom
dhruvjain99:fine-grained-access

Conversation

@dhruvjain99
Copy link
Copy Markdown
Contributor

Fixes #94

@dhruvjain99 dhruvjain99 force-pushed the fine-grained-access branch 2 times, most recently from 3f28906 to 7786521 Compare March 15, 2021 15:35
- Add warning note on public room creation for private repo
- Fix two room items displayed for same room in sidebar on creating a new room
- Add Github app slug constant
- Fix check users for private room on fetching webhook id
- Move roomMembers and create room to one controller
@dhruvjain99 dhruvjain99 force-pushed the fine-grained-access branch from 7786521 to 93f59cb Compare March 15, 2021 15:55
@RonLek RonLek force-pushed the fine-grained-access branch from 93f59cb to 9f001c3 Compare March 15, 2021 16:19
@dhruvjain99 dhruvjain99 force-pushed the fine-grained-access branch 2 times, most recently from 14875e2 to 93f59cb Compare March 15, 2021 16:31
Copy link
Copy Markdown
Contributor

@RonLek RonLek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Sing-Li @murtaza98 we can merge this PR in.

@murtaza98 murtaza98 merged commit 2fa9cda into RocketChat:develop Mar 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[IMPROVE] Replace GitHub OAuth Apps with GitHub Apps

3 participants