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

Inconsistent docs

Open kasimrehman opened this issue 3 years ago • 6 comments

The page starts off with stating that "There is no plan to use Azure File Sync.". Then Azure File Sync is mentioned several times. It seems that the text for the hybrid scenario was used as a base for the cloud-only scenario.


Document Details

Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

kasimrehman avatar Dec 05 '22 22:12 kasimrehman

@kasimrehman Thanks for your feedback! We will investigate and update as appropriate.

AjayBathini-MSFT avatar Dec 06 '22 04:12 AjayBathini-MSFT

This line is the problem: [!INCLUDE storage-files-migration-namespace-mapping].

You cannot include the same section for both the cloud-only and the hybrid scenario for Phase 1. At least not fully. Some part of this include will be the same for both scenarios and some part will be different. Please also check other places in the docs that are affected by assuming that this include applied to both the cloud-only as well as the hybrid scenario.

kasimrehman avatar Dec 06 '22 08:12 kasimrehman

@kasimrehman Thanks for bringing this to our attention. I'm going to assign this to the document author so they can take a look at it accordingly

ManoharLakkoju-MSFT avatar Dec 06 '22 10:12 ManoharLakkoju-MSFT

@khdownie Can you please check and add your comments on this doc update request as applicable.

ManoharLakkoju-MSFT avatar Dec 06 '22 10:12 ManoharLakkoju-MSFT

@kasimrehman you're right, but this will take a bit of time to fix. I'll get someone to review the docs and verify the steps. Thank you!

khdownie avatar Dec 08 '22 00:12 khdownie

@kasimrehman I have updated the article in question: https://learn.microsoft.com/en-us/azure/storage/files/storage-files-migration-nas-cloud-databox

However it seems that the other cloud-only article has this same issue. So I will update that as well.

khdownie avatar Dec 16 '22 16:12 khdownie

@khdownie Looking good now.

kasimrehman avatar Dec 16 '22 20:12 kasimrehman

Thanks @kasimrehman - I fixed the other cloud only migration article too:

https://learn.microsoft.com/en-us/azure/storage/files/storage-files-migration-robocopy

khdownie avatar Dec 16 '22 23:12 khdownie

#please-close

khdownie avatar Dec 16 '22 23:12 khdownie