-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[Text Analytics] Allow up to one action only per type #16114
[Text Analytics] Allow up to one action only per type #16114
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.
might be worth including this behavior change in changelog as well
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.
Could you add tests for it?
…xtanalytics/disable-multiple-tasks
@kristapratico and @maririos done, thanks for the feedback! |
Hello @deyaaeldeen! Because this pull request has the p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (
|
[servicebus]Update readme.python.md (Azure#16114) * Update readme.python.md * Update readme.python.md
[servicebus]Update readme.python.md (Azure#16114) * Update readme.python.md * Update readme.python.md
[servicebus]Update readme.python.md (Azure#16114) * Update readme.python.md * Update readme.python.md
[servicebus]Update readme.python.md (Azure#16114) * Update readme.python.md * Update readme.python.md
Because the service currently support one action only per type except for PII actions. However they plan to disable multiple actions for PII as well within a month. This PR does client-side checking for this across types and throw a meaningful error message. This limitation will be relaxed in the next service preview version.