[Cognitive Services ] Content Safety API Review, Added project related API

[azure-sdk]

New API Review meeting has been requested.

Service Name: Cognitive Services Review Created By: Shakira Sun @suxi-ms Review Date: 06/22/2023 04:00 PM PT Onboarding Record: PR: Hero Scenarios Link: Not Provided Core Concepts Doc Link: Not Provided

Description:

Detailed meeting information and documents provided can be accessed here For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

[azure-sdk]

Meeting updated by Shakira Sun

Service Name: Cognitive Services Review Created By: Shakira Sun Review Date: 06/22/2023 04:00 PM PT Onboarding Record: PR: Hero Scenarios Link: Not Provided Core Concepts Doc Link: Not Provided

Description:

Detailed meeting information and documents provided can be accessed here For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

[mikekistler]

@suxi-ms Please add a link to the PR to be reviewed here, or if there is no PR any other materials we can use to prep for the meeting.

[suxi-ms]

Hi Mike,

I think I have updated the issue link and PR link in the app(Azure SDK Team Scheduler Rest Review - PowerAppshttps://apps.powerapps.com/play/e/ed2ffefd-774d-40dd-ab23-7fff01aeec9f/a/e41c69de-ef96-44ba-b287-0e012e7c13ae) several days ago, is there any way to sync the new links to calendar? @.*** Thanks, Shakira

From: Mike @.> Sent: Monday, June 19, 2023 11:13 AM To: @.> Cc: Shakira @.>; @.> Subject: Re: [Azure/azure-rest-api-specs] [Cognitive Services ] API Review (Issue #24271)

@suxi-mshttps://github.com/suxi-ms Please add a link to the PR to be reviewed here, or if there is no PR any other materials we can use to prep for the meeting.

— Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-rest-api-specs/issues/24271#issuecomment-1596429133, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOTX3BFJ2QK6Q6GS3LPOPJTXL67TRANCNFSM6AAAAAAYYBTZG4. You are receiving this because you were mentioned.Message ID: @.***>

[suxi-ms]

Here is the link for the PR: https://github.com/Azure/azure-rest-api-specs-pr/pull/13081

[mikekistler]

Notes from API Review meeting 6/22/23

• How is the policy to be deployed identified?
• The policy within a project is really just the deployed policy. should probably be named deployedPolicy
• Policy management APIs will be reviewed in a future review
• Policy should be a first-class resource that can be created, updated, deleted, with a friendly name
• Consider removing "maxPageSize" if you want to avoid pagination
• Recommend keeping pagination because it is breaking to add it later
• Update the action name to deployPolicy
• Need to clean up the changes to previous versions of the OpenAPI -- minimize as much as possible

[mengaims]

Hi @mikekistler and @markweitzel, since we've canceled 0630 release, we are merging the changes discussed in https://github.com/Azure/azure-rest-api-specs-pr/issues/12812 here in 0730 release. Here are the updates in our PR:

1. Add OAuth2Auth to all versions.
2. Defined ImageWithTextImageData as: model ImageWithTextImageData { ...ImageData; } Since we will not expose ImageWithText analyze in 0730, it's removed and only exist in 0530 version currently.
3. For isRegex field in TextBlocklist, we will not expose it now, and will have more customer investigation before add it again.
4. For batch APIs, change API name addBlockItems to addOrUpdateBlockItems.
5. Return 200 for addOrUpdateBlockItems and update existing ones.
6. Return 204 for removeBlockItems and ignore non-existing ones.