-
Notifications
You must be signed in to change notification settings - Fork 357
feat: remove the Teams, TeamsMsal2 and the Msal providers in v3.x.x #2231
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
Conversation
Signed-off-by: Martin Musale <martinmusale@microsoft.com>
|
Thank you for creating a Pull Request @musale. This is a checklist for the PR reviewer(s) to complete before approving and merging this PR:
|
sebastienlevert
left a comment
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.
Should we reference the npmjs package or a branch for the v2 features?
Use npm for now, we should have a post go-live step to update these links to point at the branch IMO |
Closes #2157
Closes #2138
PR Type
Description of the changes
Removes the
TeamsProviderandTeamsMsal2Providerin favour of theTeamsFxProvider.Removes the
MsalProviderin favor of theMsal2Provider.PR checklist
yarn build) and changes have been tested in at least two supported browsers (Edge + non-Chromium based browser)yarn setLicense)Other information