-
Notifications
You must be signed in to change notification settings - Fork 61
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
feat: remote for google chat #223
Conversation
Thanks a ton for the PR(s). Love the doc PR too. This looks great at a glance. Will do a deeper dive including set up and running this sometime this week. Thanks for taking the time! |
Pull Request Test Coverage Report for Build 2087638035
💛 - Coveralls |
Has there been any progress on this PR? My company is interested in using Flottbot and remote would be good for our Google implementation Thanks |
Sorry for the delay. Had a few personal things come up. I did one attempt at setting this up some time ago with the docs and didn't make it all the way. I don't think it was an issue with the docs though. Likely, user error. I will give it another shot. In the meantime, if you don't mind getting the branch up-to-date, i will pause on the other PRs until this is resolved. |
ah yes. i got stuck on "Google Chat API is only available to Google Workspace users" - will try and see how to get that resolved |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i'm not able to test without a workspace account, but the implementation looks fine - thanks for the work.
@wass3r - this is up to date now. |
Ran into an issue with unknown message types, but resolved it with 543ca20. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks again, also for your patience 🙇🏼
Proposed change
Hi team! This PR adds initial support for Google Chat (#221). There's is an accompanying pull request for documentation here: target/flottbot-docs#56
Any feedback is appreciated!
Types of changes
What types of changes is this pull request introducing to flottbot? Put an
x
in the boxes that applyChecklist
You can fill this out after creating your PR. Put an
x
in the boxes that apply