-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Make msteams use common recipient with filled data field #48908
base: master
Are you sure you want to change the base?
Conversation
Change msteams to using the provided data field in the common recipient type to store extra required data.
This pull request is automatically being deployed by Amplify Hosting (learn more). |
Can we remove teleport/integrations/access/common/recipient.go Lines 153 to 160 in c9d3ce3
|
e2d54f6
to
7b318dd
Compare
@hugoShaka Can you take a quick look to confirm this change makes sense as it was made since you approved it. |
@hugoShaka Is this still good to go with that latest change? |
Change msteams to using the provided data field in the common recipient type to store extra required data.