Skip to content

Upgrade Storage Blob sdk from T1 to T2 #14

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
Open
AFengKK wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Upgrade Storage Blob sdk from T1 to T2 #14

AFengKK wants to merge 4 commits into from

Conversation

@AFengKK
Copy link

@AFengKK AFengKK commented May 16, 2022 •
edited
Loading

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

@lmazuel, @johanste, @kyle-patterson, @kurtzeborn, @annatisch for notification.

@AFengKK AFengKK marked this pull request as ready for review May 16, 2022 09:44
@v-xuto
Copy link

v-xuto commented May 20, 2022

@lmazuel, @johanste, @kyle-patterson, @kurtzeborn, @annatisch Could you help review this PR? Thanks a lot.

2 similar comments
@v-xuto
Copy link

v-xuto commented May 26, 2022

@lmazuel, @johanste, @kyle-patterson, @kurtzeborn, @annatisch Could you help review this PR? Thanks a lot.

@v-xuto
Copy link

v-xuto commented Jun 9, 2022

@lmazuel, @johanste, @kyle-patterson, @kurtzeborn, @annatisch Could you help review this PR? Thanks a lot.

johanste
johanste reviewed Jun 10, 2022
View reviewed changes
@v-xuto
Copy link

v-xuto commented Jun 17, 2022

@johanste We have updated by your comments, please re-review this PR. Thanks a lot.

2 similar comments
@v-xuto
Copy link

v-xuto commented Jun 24, 2022

@johanste We have updated by your comments, please re-review this PR. Thanks a lot.

@v-xuto
Copy link

v-xuto commented Jul 13, 2022

@johanste We have updated by your comments, please re-review this PR. Thanks a lot.

@v-xuto
Copy link

v-xuto commented Aug 30, 2022

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

3 similar comments
@v-xuto
Copy link

v-xuto commented Sep 9, 2022

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

@v-xuto
Copy link

v-xuto commented Nov 4, 2022

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

@v-xuto
Copy link

v-xuto commented Mar 10, 2023

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

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

Reviewers

1 more reviewer

@johanste 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