Azure-Samples / summarization-openai-python-promptflow

This solution converts speech to text and then processes and summarizes the text based on the prompt scenario.
MIT License
27 stars 22 forks source link

use user managed identity to auth to speech instead of key #3

Closed yungezz closed 6 months ago

yungezz commented 6 months ago

Purpose

Does this introduce a breaking change?

[ ] Yes
[ ] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

What to Check

Verify that the following are valid

Other Information

marlenezw commented 6 months ago

Thanks for this PR @yungezz! I'll start testing locally and make changes directly to this PR and merge once changes are good. I've marked the PR ready for review but let me know if you'd prefer that I make a seperate PR? Thanks

marlenezw commented 6 months ago

Actually, there are a bunch of updates that have been made in main that would need to be changed to merge this in. Let me create a sepearte PR for the moment