Skip to content
This repository was archived by the owner on Jun 11, 2026. It is now read-only.

Upgrade Storage Blob sdk from T1 to T2 - #14

Open
Rudy (MSFT) (AFengKK) wants to merge 4 commits into
Azure-Samples:mainfrom
AFengKK:upgrade_sdk
Open

Upgrade Storage Blob sdk from T1 to T2#14
Rudy (MSFT) (AFengKK) wants to merge 4 commits into
Azure-Samples:mainfrom
AFengKK:upgrade_sdk

Conversation

@AFengKK

@AFengKK Rudy (MSFT) (AFengKK) commented May 16, 2022 •
edited
Loading

Copy link
Copy Markdown

1.Upgrade Storage Blob sdk from T1 to T2
2.Upgrade files:

  • CleanTrigger1/clean.py
  • CleanTrigger2/clean.py
  • README.md
  • Reconcile/clean.py
  • Reconcile/fetch_blob.py
  • azure-deploy-linux-app-plan.json
  • blob_to_smart_contract/clean.py
  • blob_to_smart_contract/fetch_blob.py
  • requirements.txt

Laurent Mazuel (@lmazuel), Johan Stenberg (ex-msft) (@johanste), Kyle P (@kyle-patterson), Scott K (@kurtzeborn), Anna Tisch (@annatisch) for notification.

@AFengKK
Rudy (MSFT) (AFengKK) marked this pull request as ready for review May 16, 2022 09:44
@v-xuto

Tong Xu (MSFT) (v-xuto) commented May 20, 2022

Copy link
Copy Markdown

Laurent Mazuel (@lmazuel), Johan Stenberg (ex-msft) (@johanste), Kyle P (@kyle-patterson), Scott K (@kurtzeborn), Anna Tisch (@annatisch) Could you help review this PR? Thanks a lot.

2 similar comments
@v-xuto

Tong Xu (MSFT) (v-xuto) commented May 26, 2022

Copy link
Copy Markdown

Laurent Mazuel (@lmazuel), Johan Stenberg (ex-msft) (@johanste), Kyle P (@kyle-patterson), Scott K (@kurtzeborn), Anna Tisch (@annatisch) Could you help review this PR? Thanks a lot.

@v-xuto

Tong Xu (MSFT) (v-xuto) commented Jun 9, 2022

Copy link
Copy Markdown

Laurent Mazuel (@lmazuel), Johan Stenberg (ex-msft) (@johanste), Kyle P (@kyle-patterson), Scott K (@kurtzeborn), Anna Tisch (@annatisch) Could you help review this PR? Thanks a lot.

johanste
Johan Stenberg (ex-msft) (johanste) reviewed Jun 10, 2022
View reviewed changes
Comment thread requirements.txt Outdated
@v-xuto

Tong Xu (MSFT) (v-xuto) commented Jun 17, 2022

Copy link
Copy Markdown

Johan Stenberg (ex-msft) (@johanste) We have updated by your comments, please re-review this PR. Thanks a lot.

2 similar comments
@v-xuto

Tong Xu (MSFT) (v-xuto) commented Jun 24, 2022

Copy link
Copy Markdown

Johan Stenberg (ex-msft) (@johanste) We have updated by your comments, please re-review this PR. Thanks a lot.

@v-xuto

Tong Xu (MSFT) (v-xuto) commented Jul 13, 2022

Copy link
Copy Markdown

Johan Stenberg (ex-msft) (@johanste) We have updated by your comments, please re-review this PR. Thanks a lot.

@v-xuto

Tong Xu (MSFT) (v-xuto) commented Aug 30, 2022

Copy link
Copy Markdown

Anna Tisch (@annatisch) We have updated by your comments, please re-review this PR.

3 similar comments
@v-xuto

Tong Xu (MSFT) (v-xuto) commented Sep 9, 2022

Copy link
Copy Markdown

Anna Tisch (@annatisch) We have updated by your comments, please re-review this PR.

@v-xuto

Tong Xu (MSFT) (v-xuto) commented Nov 4, 2022

Copy link
Copy Markdown

Anna Tisch (@annatisch) We have updated by your comments, please re-review this PR.

@v-xuto

Tong Xu (MSFT) (v-xuto) commented Mar 10, 2023

Copy link
Copy Markdown

Anna Tisch (@annatisch) We have updated by your comments, please re-review this PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer
@johanste Johan Stenberg (ex-msft) (johanste) johanste left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@AFengKK @v-xuto @johanste