Skip to content

[New] Oath2 With RC Apps Scheduler and WebHooks Integration #7

Merged
RonLek merged 32 commits into
RocketChat:mainfrom
samad-yar-khan:webhooks
Jul 17, 2022
Merged

[New] Oath2 With RC Apps Scheduler and WebHooks Integration #7
RonLek merged 32 commits into
RocketChat:mainfrom
samad-yar-khan:webhooks

Conversation

@samad-yar-khan
Copy link
Copy Markdown
Contributor

Issues

Proposed changes (including videos or screenshots)

OAuth2 and RC Scheduler API

2022-07-06.04-41-37.mp4

Repository Subscriptions / WebHooks Integration

2022-07-06.15-31-10.mp4
  • Unsubscribe to all repository events ( issues, pull_request, push, deployment_status, star) using /github USERNAME/REPONAME unsubscribe
2022-07-06.15-20-55.mp4
  • New Subscription Modal and add Subscription Feature
2022-07-06.15-34-42.mp4
  • Delete Subscriptions
2022-07-06.15-40-28.mp4
  • Updating and Using Same Hook for different rooms (Edge Case Handling)
2022-07-06.15-43-16.mp4
  • Deleting Non Required events from the same hook ( Edge Case Handling)
2022-07-06.15-49-32.mp4

@RonLek
Copy link
Copy Markdown
Contributor

RonLek commented Jul 6, 2022

@samad-yar-khan Thanks for your PR. For authentication can we show a private message to the user who triggered the /github command within the same channel instead of the github-bot posting it? Would be a good improvement on the UX.

@samad-yar-khan
Copy link
Copy Markdown
Contributor Author

@samad-yar-khan Thanks for your PR. For authentication can we show a private message to the user who triggered the /github command within the same channel instead of the github-bot posting it? Would be a good improvement on the UX.

@RonLek Yes, we can do that. Thankyou for the suggestion. I will make the changes.

@samad-yar-khan
Copy link
Copy Markdown
Contributor Author

@RonLek I have made the changes, the login notifications will now be dispalayed to the user in the same room as private messages.

2022-07-06.23-02-34.mp4

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.

LGTM! Features look amazing. Thanks. 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants