Skip to content
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

[text analytics] add redaction text property to PII entities #14090

Closed
iscai-msft opened this issue Aug 10, 2020 · 3 comments · Fixed by #14794
Closed

[text analytics] add redaction text property to PII entities #14090

iscai-msft opened this issue Aug 10, 2020 · 3 comments · Fixed by #14794
Assignees
@maririos
Labels
blocking-release Blocks release Client This issue points to a problem in the data-plane of the library. Cognitive - Text Analytics
Milestone
[2020] September

Comments

@iscai-msft
Copy link

No description provided.

@iscai-msft iscai-msft added Client This issue points to a problem in the data-plane of the library. Cognitive - Text Analytics labels Aug 10, 2020
@iscai-msft iscai-msft added this to the [2020] September milestone Aug 10, 2020
@iscai-msft
Copy link
Author

Blocked until service adds property to swagger

@maririos
Copy link
Member

how should we surface this in our libraries? will it be a problem to log it? or what else?

@maririos maririos added the blocking-release Blocks release label Aug 25, 2020
@maririos
Copy link
Member

Confirmed and it is ok to have it in PII and log it.

@maririos maririos mentioned this issue Sep 2, 2020
Merged
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@maririos maririos

Labels
blocking-release Blocks release Client This issue points to a problem in the data-plane of the library. Cognitive - Text Analytics
Projects
None yet
Milestone
[2020] September
Development

Successfully merging a pull request may close this issue.

[TA] Add PII back maririos/azure-sdk-for-net
2 participants
@maririos @iscai-msft