Skip to content
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(teams): teams v1 #336

Merged
merged 16 commits into from
Feb 3, 2022
Merged

feat(teams): teams v1 #336

merged 16 commits into from
Feb 3, 2022

Conversation

samuelmasse
Copy link
Contributor

@samuelmasse samuelmasse commented Feb 2, 2022

Upgrades teams to be version 1.0.0. Added more content types and fixed some things. As for the API version nothing really changed for this channel. We were using botbuilder 4.14.0 before and now we use 4.15.0

Closes DEV-2277

@linear
Copy link

linear bot commented Feb 2, 2022

DEV-2277 Teams

packages/channels/package.json Show resolved Hide resolved
packages/channels/src/teams/README.md Show resolved Hide resolved
Comment on lines +21 to +23
| Quick Reply | ✅ |
| Postback | ✅ |
| Say Something | ✅ |
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I sorted these out. Turns out this channel only received text, even if it was quick reply, postback and say something. Now all three work.

Comment on lines +4 to +7
export interface TeamsConfig extends ChannelConfig {
appId: string
appPassword: string
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed the tenantId here because I don't know what it's used for. We can add it back if we need it.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think it was for some validation but I'm not sure either.

@samuelmasse samuelmasse marked this pull request as ready for review February 2, 2022 23:08
Comment on lines +4 to +7
export interface TeamsConfig extends ChannelConfig {
appId: string
appPassword: string
}
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it was for some validation but I'm not sure either.

@samuelmasse samuelmasse merged commit 3720261 into master Feb 3, 2022
@samuelmasse samuelmasse deleted the sm-teams-v1 branch February 3, 2022 00:00
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