azure-docs icon indicating copy to clipboard operation
azure-docs copied to clipboard

Doc updates for de id overview

Open LeaKass opened this issue 9 months ago • 11 comments

added 2 images, updates text, added a section

LeaKass avatar Apr 16 '25 19:04 LeaKass

@LeaKass : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. @kimiamavon

prmerger-automator[bot] avatar Apr 16 '25 19:04 prmerger-automator[bot]

@LeaKass : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. @kimiamavon

prmerger-automator[bot] avatar Apr 16 '25 19:04 prmerger-automator[bot]

Learn Build status updates of commit f1ccceb:

:warning: Validation status: warnings

File Status Preview URL Details
articles/healthcare-apis/deidentification/overview.md :warning:Warning Details
articles/healthcare-apis/deidentification/Surrogated consistently accross batches.png :white_check_mark:Succeeded
articles/healthcare-apis/deidentification/Tag Redact Surrogate.png :white_check_mark:Succeeded

articles/healthcare-apis/deidentification/overview.md

  • Line 48, Column 206: [Warning: hard-coded-locale - See documentation] Link 'https://learn.microsoft.com/en-us/rest/api/health-dataplane/deidentify-text/deidentify-text?view=rest-health-dataplane-2024-11-15&tabs=HTTP#phicategory' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
  • Line 76, Column 112: [Warning: hard-coded-locale - See documentation] Link 'https://azure.microsoft.com/en-us/pricing/calculator/' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
  • Line 48, Column 206: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/en-us/rest/api/health-dataplane/deidentify-text/deidentify-text?view=rest-health-dataplane-2024-11-15&tabs=HTTP#phicategory' will be broken in isolated environments. Replace with a relative link.
  • Line 48, Column 206: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: https://learn.microsoft.com/en-us/rest/api/health-dataplane/deidentify-text/deidentify-text?view=rest-health-dataplane-2024-11-15&tabs=HTTP#phicategory

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Hi @leakass - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. We can keep this PR open for review and merge, but would you make future content updates in the private repo? Thank you!


@kimiamavon-msft

Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged" @MicrosoftDocs/public-repo-pr-review-team

v-dirichards avatar Apr 16 '25 20:04 v-dirichards

Learn Build status updates of commit 31b0c4b:

:x: Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
:x:Error Details

  • [Error: CannotMergeCommit] Cannot merge commit 31b0c4bbd42155fbf701979e4351c9e577d7981f in branch doc-updates-for-de-id-overview of repository https://github.com/LeaKass/azure-docs into branch main (commit ca4ae1c078cb72a05af8d0e611b3e83565a3515a). Please follow this documentation: https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line/ to use git.exe to resolve you content conflicts locally and then push to remote.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Learn Build status updates of commit e88f208:

:warning: Validation status: warnings

File Status Preview URL Details
articles/healthcare-apis/deidentification/overview.md :warning:Warning Details
[articles/healthcare-apis/deidentification/Surrogated consistently accross batches.png](https://github.com/LeaKass/azure-docs/blob/doc-updates-for-de-id-overview/articles/healthcare-apis/deidentification/Surrogated consistently accross batches.png) :white_check_mark:Succeeded
[articles/healthcare-apis/deidentification/Tag Redact Surrogate.png](https://github.com/LeaKass/azure-docs/blob/doc-updates-for-de-id-overview/articles/healthcare-apis/deidentification/Tag Redact Surrogate.png) :white_check_mark:Succeeded

articles/healthcare-apis/deidentification/overview.md

  • Line 49, Column 206: [Warning: hard-coded-locale - See documentation] Link 'https://learn.microsoft.com/en-us/rest/api/health-dataplane/deidentify-text/deidentify-text?view=rest-health-dataplane-2024-11-15&tabs=HTTP#phicategory' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
  • Line 91, Column 112: [Warning: hard-coded-locale - See documentation] Link 'https://azure.microsoft.com/en-us/pricing/calculator/' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
  • Line 49, Column 206: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/en-us/rest/api/health-dataplane/deidentify-text/deidentify-text?view=rest-health-dataplane-2024-11-15&tabs=HTTP#phicategory' will be broken in isolated environments. Replace with a relative link.
  • Line 49, Column 206: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: https://learn.microsoft.com/en-us/rest/api/health-dataplane/deidentify-text/deidentify-text?view=rest-health-dataplane-2024-11-15&tabs=HTTP#phicategory

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@kimiamavon-msft - Could you review this proposed update to your article and enter #sign-off in a comment if it's ready to merge? Note that the image names need to be updated to reflect the Microsoft style (all lower case, no spaces, words separated by hyphens).

Thanks!

v-dirichards avatar May 05 '25 21:05 v-dirichards

Learn Build status updates of commit 1972d36:

:bulb: Validation status: suggestions

File Status Preview URL Details
articles/healthcare-apis/deidentification/overview.md :bulb:Suggestion Details
[articles/healthcare-apis/deidentification/Surrogated consistently accross batches.png](https://github.com/LeaKass/azure-docs/blob/doc-updates-for-de-id-overview/articles/healthcare-apis/deidentification/Surrogated consistently accross batches.png) :white_check_mark:Succeeded
[articles/healthcare-apis/deidentification/Tag Redact Surrogate.png](https://github.com/LeaKass/azure-docs/blob/doc-updates-for-de-id-overview/articles/healthcare-apis/deidentification/Tag Redact Surrogate.png) :white_check_mark:Succeeded

articles/healthcare-apis/deidentification/overview.md

  • Line 49, Column 206: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/rest/api/health-dataplane/deidentify-text/deidentify-text?view=rest-health-dataplane-2024-11-15&tabs=HTTP#phicategory' will be broken in isolated environments. Replace with a relative link.
  • Line 49, Column 206: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: https://learn.microsoft.com/rest/api/health-dataplane/deidentify-text/deidentify-text?view=rest-health-dataplane-2024-11-15&tabs=HTTP#phicategory

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
File Change Percent This PR contains file(s) with more than 30% file change.
GDPR Content(s) PR references GDPR and must be human reviewed to ensure appropriate guidelines are followed.
Image File This PR added or updated an image file(s).

prmerger-automator[bot] avatar May 05 '25 21:05 prmerger-automator[bot]

@kimiamavon-msft

Can you review this old PR and determine whether it needs to be closed or merged?

Note that the image names need to be updated to reflect the Microsoft style (all lower case, no spaces, words separated by hyphens).

@MicrosoftDocs/public-repo-pr-review-team

v-dirichards avatar May 27 '25 21:05 v-dirichards

I sent an email to the content owner today.

@MicrosoftDocs/public-repo-pr-review-team

v-ccolin avatar Jun 20 '25 07:06 v-ccolin

@LeaKass

It looks like you made these changes in the private repo PR 300303. If you'd like to update the article in its current state, please open a new PR in the private repo. Thanks.

#please-close

@MicrosoftDocs/public-repo-pr-review-team

ttorble avatar Jul 30 '25 09:07 ttorble