Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Storage] STG 88 Archboard Review Feedback #30129

Conversation

vincenttran-msft
Copy link
Member

This PR addresses the archboard feedback for STG88. This includes the following feedback:

  • Updating return types docstring (ItemPaged -> ItemPaged[Handle])
  • Making access_rights Optional.

@github-actions github-actions bot added the Storage Storage Service (Queues, Blobs, Files) label Apr 28, 2023
@azure-sdk
Copy link
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

azure-storage-file-share

@vincenttran-msft vincenttran-msft merged commit d3df623 into Azure:feature/storage-stg88 Apr 28, 2023
@vincenttran-msft vincenttran-msft deleted the vincenttran/stg88_archboard_fb branch April 28, 2023 19:24
vincenttran-msft added a commit that referenced this pull request May 3, 2023
* Initial STG87 feature branch setup

* Revert "Initial STG87 feature branch setup"

This reverts commit 8b43aa2.

* [Storage] Initial STG88 feature branch setup (#29401)

* Initial STG88 feature branch setup

* Add version

* [Storage] [STG88] High throughput append blob (#29442)

* Added comments + sync&async test

* Remove comments

* [Storage] [STG88] Owner, group, permissions in `DirectoryProperties` and `FileProperties` (#29491)

* Added testcases

* Added owner,group,permissions to DirectoryProperties and FileProperties

* PR feedback, update properties docstrnig

* [Storage] [STG88] List Handles Access Rights (#29581)

* Done sync&async

* Remove preprod from base_client

* [Storage] Add `encryption_context` kwarg to `upload_data` API  (#29893)

* Upload + async/sync tests

* Version drop

* Datalake-blob dependency, bring back version

* Use param instead of kwarg (#29925)

* paramtype not param (#29927)

* [Storage] STG 88 Archboard Review Feedback (#30129)

* Changelog
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Storage Storage Service (Queues, Blobs, Files)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants