-
Notifications
You must be signed in to change notification settings - Fork 0
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.
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
All changes missing Kevin's sink changes #1
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few minor fixes required, will need to backport to Share repo as well.
...ics.Purview.Share/preview/2022-06-30-preview/examples/ReceivedShares_AdlsGen2AccountGet.json
Outdated
Show resolved
Hide resolved
...are/preview/2022-06-30-preview/examples/ReceivedShares_Activate_TenantEmailRegistration.json
Outdated
Show resolved
Hide resolved
...ne/Azure.Analytics.Purview.Share/preview/2022-06-30-preview/examples/ReceivedShares_Get.json
Outdated
Show resolved
Hide resolved
....Share/preview/2022-06-30-preview/examples/ReceivedShares_GetAll_AttachedReceivedShares.json
Show resolved
Hide resolved
...are/preview/2022-06-30-preview/examples/ReceivedShares_Register_TenantEmailRegistration.json
Outdated
Show resolved
Hide resolved
...urview.Share/preview/2022-06-30-preview/examples/SentShares_GetAll_SentShareInvitations.json
Outdated
Show resolved
Hide resolved
....Purview.Share/preview/2022-06-30-preview/examples/ReceivedShares_AdlsGen2AccountUpdate.json
Outdated
Show resolved
Hide resolved
This is just a starting point, for any other improvements feel free to create a PR into the main branch. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove /v2 from all routes in spec.
Choose a PR Template
Switch to "Preview" on this description then select one of the choices below.
Click here to open a PR for a Data Plane API.
Click here to open a PR for a Control Plane (ARM) API.