Skip to content

Fix typo in FHIR audit header name#58507

Merged
ktoliver merged 1 commit into
MicrosoftDocs:masterfrom
c-w:patch-1
Jul 7, 2020
Merged

Fix typo in FHIR audit header name#58507
ktoliver merged 1 commit into
MicrosoftDocs:masterfrom
c-w:patch-1

Conversation

@c-w

@c-w c-w commented Jul 6, 2020

Copy link
Copy Markdown
Contributor

The documentation currently states that custom FHIR audit log headers must be named like X-MS-AZUREFHIR-AUDIT-AUDIT-<name>.

However, all the examples of custom FHIR audit log headers in the documentation only contain a single AUDIT followed by a name, i.e. all examples follow the template X-MS-AZUREFHIR-AUDIT-<name>.

To prevent reader confusion, this change makes the documented header name template consistent with the examples.

The documentation currently states that custom FHIR audit log headers must be named like `X-MS-AZUREFHIR-AUDIT-AUDIT-<name>`.

However, all the examples of custom FHIR audit log headers in the documentation only contain a single `AUDIT` followed by a name.

To prevent reader confusion, this change makes the documented header name template consistent with the examples.
@ktoliver ktoliver requested a review from matjazl July 6, 2020 21:43
@ktoliver ktoliver added aq-pr-triaged tracking label for the PR review team healthcare-apis/svc labels Jul 6, 2020
@ktoliver

ktoliver commented Jul 6, 2020

Copy link
Copy Markdown
Contributor

@matjazl - Would you review the proposed update to your article? Thanks.

@PRMerger8

Copy link
Copy Markdown
Contributor

@c-w : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@matjazl matjazl left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes I approve the chan

@ktoliver ktoliver merged commit 02d36e4 into MicrosoftDocs:master Jul 7, 2020
@c-w c-w deleted the patch-1 branch July 7, 2020 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants