-
Notifications
You must be signed in to change notification settings - Fork 196
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: api me with aad auth template #11092
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #11092 +/- ##
==========================================
+ Coverage 87.00% 87.16% +0.15%
==========================================
Files 451 457 +6
Lines 26783 27623 +840
Branches 5376 5560 +184
==========================================
+ Hits 23303 24077 +774
- Misses 1758 1778 +20
- Partials 1722 1768 +46 |
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.
Is there any source code difference from no-auth api me template? Could we consider merge them into one template?
The manifest, README, YML, and Bicep files differ from templates without auth. Do you think it might be complicated to generate different templates through variables? Should we merge them into one template? |
https://msazure.visualstudio.com/Microsoft%20Teams%20Extensibility/_workitems/edit/25150693
Add a new API message extension template to support AAD auth.