Skip to content

Typo "Azure Blob storage service"→"Azure Blob Storage"#7164

Merged
ktoliver merged 2 commits intoMicrosoftDocs:livefrom
hyoshioka0128:patch-400
Jan 27, 2022
Merged

Typo "Azure Blob storage service"→"Azure Blob Storage"#7164
ktoliver merged 2 commits intoMicrosoftDocs:livefrom
hyoshioka0128:patch-400

Conversation

@PRMerger-2
Copy link
Copy Markdown
Contributor

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

@LitKnd
Copy link
Copy Markdown
Contributor

LitKnd commented Jan 6, 2022

Thanks @hyoshioka0128 ! This looks great. I added one tiny suggestion for you to review if that makes sense.

Kendra from the Database Docs team

@ktoliver ktoliver added the aq-pr-triaged tracking label for the PR review team label Jan 6, 2022
@LitKnd
Copy link
Copy Markdown
Contributor

LitKnd commented Jan 26, 2022

Hi,

Just checking back in on this. The suggestion I made wasn't incorporated-- is that something you can add to the commit? There is an extra "the" in the following sentence:

An Azure storage account and SQL credential are required prerequisites and for all backup and restore operations to the Azure Blob Storage.

I believe this should be:

An Azure storage account and SQL credential are required prerequisites and for all backup and restore operations to Azure Blob Storage.

Thanks,
Kendra

@hyoshioka0128
Copy link
Copy Markdown
Contributor Author

@LitKnd - Thanks! I overlooked it. I also accepted.

@LitKnd
Copy link
Copy Markdown
Contributor

LitKnd commented Jan 27, 2022

Hi @hyoshioka0128 ,

It still shows that there was not a second commit. I think the process with the suggestion is a bit confusing.

It is not a big deal and I can circle back and clean up that one word after :)

Thanks again for your submission and for making our docs better.

Kendra

#sign-off

@hyoshioka0128
Copy link
Copy Markdown
Contributor Author

@LitKnd - Thanks!

@LitKnd
Copy link
Copy Markdown
Contributor

LitKnd commented Jan 27, 2022

Closing and reopening as I think maybe some automation got stuck.
#please-close

@PRMerger14 PRMerger14 closed this Jan 27, 2022
@LitKnd
Copy link
Copy Markdown
Contributor

LitKnd commented Jan 27, 2022

#please-open

@LitKnd
Copy link
Copy Markdown
Contributor

LitKnd commented Jan 27, 2022

#sign-off

@PRMerger6 PRMerger6 requested a review from LitKnd January 27, 2022 17:25
@PRMerger6
Copy link
Copy Markdown
Contributor

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

@ktoliver ktoliver merged commit a527374 into MicrosoftDocs:live Jan 27, 2022
@ktoliver
Copy link
Copy Markdown
Contributor

I removed "the" where indicated. Thanks, all.

@hyoshioka0128
Copy link
Copy Markdown
Contributor Author

@ktoliver - Thanks!

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.