Skip to content

Conversation

@zhiyuanliang-ms
Copy link
Contributor

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

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@zhiyuanliang-ms
Copy link
Contributor Author

@mrm9084 @amerjusupovic @rossgrambo

I have updated this PR to use system assigned identity and token credential (instead of user assigned identity and connection string). This could also be applied to quote of the day example in other languages.

@zhiyuanliang-ms
Copy link
Contributor Author

@jimmyca15 @zhenlan Do you think we should use targeting context accessor pattern in this example?

@zhenlan
Copy link
Collaborator

zhenlan commented Feb 24, 2025

@jimmyca15 @zhenlan Do you think we should use targeting context accessor pattern in this example?

Let's not worry about it for now.

@zhiyuanliang-ms zhiyuanliang-ms merged commit 5739d56 into main Mar 4, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants