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.

Sign up for GitHub

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

Jump to bottom

@alvesfabi OBO Support #803

Merged
merged 76 commits into from
May 15, 2024
Merged

@alvesfabi OBO Support #803

merged 76 commits into from
May 15, 2024

Conversation

teresaqhoang
Copy link
Contributor

Motivation and Context

@alvesfabi's work to implement the OBO flow for OpenAI Plugin Specs

Description

Contribution Checklist

alvesfabi and others added 17 commits January 29, 2024 19:45
@alvesfabi
initial version
30b55c0
@alvesfabi
in progress with OBO token
bbb1446
@alvesfabi
working version native plugin hardcoded
d096fe3
@alvesfabi
working version odata query
05d6888
@alvesfabi
working version persona
529ed41
@alvesfabi
working version with app settings
72fb304
@alvesfabi
Merge pull request #1 from alvesfabi/clean-for-pr
e17ab92 
Clean for pr
@alvesfabi
cleaned up version
5960d97
@alvesfabi
Merge pull request #2 from alvesfabi/clean-for-pr
7e9bc5a 
cleaned up version
@alvesfabi
cleaned up version 3
bde84da
@alvesfabi
Merge pull request #3 from alvesfabi/clean-for-pr
f02cb0f 
cleaned up version 3
@alvesfabi
improved readme
0b348f3
@alvesfabi
Merge pull request #4 from alvesfabi/clean-for-pr
2616c56 
improved readme
@alvesfabi
fixed obo readme
39f2611
@alvesfabi
Merge pull request #5 from alvesfabi/fixed-readme-obo
683d7a9 
fixed obo readme
@alvesfabi
Merge remote-tracking branch 'upstream/main'
f5a2f2d
@alvesfabi
fixed plugin description
da81269
@teresaqhoang teresaqhoang self-assigned this Feb 15, 2024
@github-actions github-actions bot added documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file webapp Pull requests that update Typescript code webapi Pull requests that update .net code labels Feb 15, 2024
teresaqhoang
teresaqhoang commented Feb 15, 2024
View reviewed changes
.vscode/launch.json Outdated Show resolved Hide resolved
chat azure open ai dev settings.txt Outdated Show resolved Hide resolved
webapi/Controllers/ChatController.cs Outdated Show resolved Hide resolved
webapi/Plugins/Chat/ApiConnectorPlugin.cs Outdated Show resolved Hide resolved
webapi/Plugins/Chat/ApiConnectorPlugin.cs Outdated Show resolved Hide resolved
webapi/Plugins/Chat/ApiConnectorPlugin.cs Outdated Show resolved Hide resolved
webapp/src/Constants.ts Outdated Show resolved Hide resolved
webapp/tsconfig.json Outdated Show resolved Hide resolved
@teresaqhoang teresaqhoang marked this pull request as ready for review May 9, 2024 18:27
alliscode
alliscode previously approved these changes May 10, 2024
View reviewed changes
TaoChenOSU
TaoChenOSU reviewed May 10, 2024
View reviewed changes
plugins/OBO/README.md Outdated Show resolved Hide resolved
webapi/Controllers/ChatController.cs Outdated Show resolved Hide resolved
webapi/Plugins/Chat/ChatPlugin.cs Outdated Show resolved Hide resolved
webapi/Plugins/Chat/MsGraphOboPlugin.cs Show resolved Hide resolved
webapi/Plugins/Utils/JsonUtils.cs Outdated Show resolved Hide resolved
@alvesfabi @TaoChenOSU
Update plugins/OBO/README.md
02f2708 
Co-authored-by: Tao Chen <taochen@microsoft.com>
@alvesfabi @TaoChenOSU
Update webapi/Controllers/ChatController.cs
849f88e 
Co-authored-by: Tao Chen <taochen@microsoft.com>
alvesfabi
alvesfabi reviewed May 11, 2024
View reviewed changes
Copy link
Contributor

@alvesfabi alvesfabi left a comment

Choose a reason for hiding this comment

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

fixed merge error - added comment for msal setup change in index.ts

@teresaqhoang teresaqhoang added this pull request to the merge queue May 15, 2024
Merged via the queue into microsoft:main with commit d79f076 May 15, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alliscode alliscode alliscode approved these changes

@TaoChenOSU TaoChenOSU Awaiting requested review from TaoChenOSU

@alvesfabi alvesfabi Awaiting requested review from alvesfabi

Assignees

@teresaqhoang teresaqhoang

Labels
dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation PR: paused For PRs that have been converted to draft, are facing blockers or have no active development planned webapi Pull requests that update .net code webapp Pull requests that update Typescript code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@teresaqhoang @TaoChenOSU @alliscode @alvesfabi