feat: add support for using premium apis#2885
feat: add support for using premium apis#2885gavinbarron wants to merge 6 commits intonext/mgt-chatfrom
Conversation
|
I think we should add the premium APIs query string parameter only on APIs that support it. This will limit the scope and reduce questions (and confusion). |
I was reading the document that said to put it on every teams call: https://microsoft.sharepoint.com/:w:/t/ExtensibilityandFundamentals/EfO9ynUvRptOrpUz5ydmqvEBTMfWipfRkYC73HhUbCClwA?e=UbKVeR I feel that the scope of this feature is unclear. |
|
🚀 New react-contoso sample application deployed here |
|
Marking as do not merge until we get more clarity on scope and impact |
|
🚀 New react-contoso sample application deployed here |
|
🚀 New react-contoso sample application deployed here |
musale
left a comment
There was a problem hiding this comment.
@gavinbarron should we have a generic error state for when the app is not monetized but a chat requested for premium apis?
Yes we should have an error state for that, good call. |
|
Wouldn't it make sense to allow dev to specify model=A in case they had a qualifying use case? |
Given that model=A is reserved for Security and Compliance products I can't see a scenario where an application using delegated permissions could fit in that role, I'm 99% sure that they are also restricted to app only scenarios https://learn.microsoft.com/en-us/graph/teams-licenses#modela-requirements |
|
🚀 New react-contoso sample application deployed here |
|
Closes #2722
PR Type
Description of the changes
Adds the ability for a developer to specify that a chat component should append
model=Bto graph calls thus enabling higher throughput and attracting cost. Needs to have Azure billing configured.PR checklist
yarn build) and changes have been tested in at least two supported browsers (Edge + non-Chromium based browser)yarn setLicense)Other information