-
Notifications
You must be signed in to change notification settings - Fork 187
Create Find-SPOCrossTenantLongFilePathsForSiteRename.md #950
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.
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
Conversation
|
Learn Build status updates of commit 035b75a:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| sharepoint/sharepoint-ps/Microsoft.Online.SharePoint.PowerShell/Microsoft.Online.SharePoint.PowerShell.md | View (sharepoint-ps) | Details | |
| sharepoint/sharepoint-ps/Microsoft.Online.SharePoint.PowerShell/Find-SPOCrossTenantLongFilePathsForSiteRename.md | ✅Succeeded | View (sharepoint-ps) |
sharepoint/sharepoint-ps/Microsoft.Online.SharePoint.PowerShell/Microsoft.Online.SharePoint.PowerShell.md
- Line 0, Column 0: [Warning: PSMD2Yaml_InconsistentCmdletsInModule]
Inconsistent cmdlets found in module: Microsoft.Online.SharePoint.PowerShell. 1 cmdlets in the module folder but not listed in the module file: Find-SPOCrossTenantLongFilePathsForSiteRename.
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.
|
Learn Build status updates of commit a1ee2c9: ✅ Validation status: passed
For more details, please refer to the build report. |
| ``` | ||
|
|
||
| ## DESCRIPTION | ||
| SharePoint Cross Tenant User Migration has a restriction where the source tenant site's file path should not exceed 400 characters in length after merging to the target side. This cmdlet finds sites that exceed the 400 character limit. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestion from Rewrite agent:
During SharePoint cross-tenant user migration, file paths from the source tenant must be 400 characters or fewer after merging with the target tenant. This cmdlet helps you identify sites that exceed this limit. #Closed
|
|
||
| 1) - sites/CommTest1/Shared Documents/Level1LongFolderNameFor400CharLimitValidationTesting/Level2LongFolderNameFor400CharLimitValidationTesting/Level3LongFolderNameFor400CharLimitValidationTesting/Level4LongFolderNameFor400CharLimitValidationTesting/Level5LongFolderNameFor400CharLimitValidationTesting/Level6LongFolderNameFor400CharLimitValidationTesting/Level7LongFolderNameFor400CharLimit/a.txt | ||
| 2) - sites/CommTest1/Shared Documents/Level1LongFolderNameFor400CharLimitValidationTesting/Level2LongFolderNameFor400CharLimitValidationTesting/Level3LongFolderNameFor400CharLimitValidationTesting/Level4LongFolderNameFor400CharLimitValidationTesting/Level5LongFolderNameFor400CharLimitValidationTesting/Level6LongFolderNameFor400CharLimitValidationTesting/Level7LongFolderNameFor400CharLimit | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- remove this entire output thing as we don't do this for other cmdlets. only example usage is sufficient.
- also, use common public domains such as contoso and fabrikam -- prepspo is internal.
- remove the "PS C:..." prefix
Simply use this example:
Find-SPOCrossTenantLongFilePathsForSiteRename -OldSiteUrl [https://contoso.sharepoint.com/sites/site1](https://contoso.sharepoint.com/sites/site1) -NewSiteUrl [https://fabrikam.sharepoint.com/sites/site1](https://fabrikam.sharepoint.com/sites/site1)
#Closed
| List of internal components with too long paths for Cross-tenant migration of the source site [https://prepspo.spgrid.com/sites/CommTest1] | ||
| into the target relative url [/sites/CommTest1_longer] with the relative delta [7]: | ||
|
|
||
| No problem found |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this example is not needed and can be removed #Closed
| ## PARAMETERS | ||
|
|
||
| ### -NewSiteUrl | ||
| This parameter includes the source site URL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you rewrite this for clarity? "includes the source site URL" is not clear -- what does "include" mean in this context? same for the OldSiteUrl
| ``` | ||
|
|
||
| ### -OldSiteUrl | ||
| This parameter includes the target site URL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
|
||
| ## SYNOPSIS | ||
|
|
||
| Finds sites that exceed the 400 character limit for Cross Tenant User Migration moves. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rewrite agent suggestion:
Identify sites with file paths over 400 characters for cross-tenant user migration.
update in Microsoft.Online.SharePoint.PowerShell.md as well #Closed
|
Learn Build status updates of commit 93670d6: ✅ Validation status: passed
For more details, please refer to the build report. |
|
Learn Build status updates of commit 33a4b50: ✅ Validation status: passed
For more details, please refer to the build report. |
vgaddam-pm
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made changes based on feedback. Still unsure about wording for parameters description or if it is better to just remove it altogether.
|
Learn Build status updates of commit 4961be8: ✅ Validation status: passed
For more details, please refer to the build report. |
|
|
||
| ### Example 1 | ||
| ```powershell | ||
| Find-SPOCrossTenantLongFilePathsForSiteRename -OldSiteUrl [https://contoso.sharepoint.com/sites/site1](https://contoso.sharepoint.com/sites/site1) -NewSiteUrl [https://fabrikam.sharepoint.com/sites/site1](https://fabrikam.sharepoint.com/sites/site1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Find-SPOCrossTenantLongFilePathsForSiteRename -OldSiteUrl [https://contoso.sharepoint.com/sites/site1](https://contoso.sharepoint.com/sites/site1) -NewSiteUrl [https://fabrikam.sharepoint.com/sites/site1](https://fabrikam.sharepoint.com/sites/site1) | |
| https://contoso.sharepoint.com/sites/site1 | |
| ``` #Closed |
|
|
||
| ### Example 1 | ||
| ```powershell | ||
| Find-SPOCrossTenantLongFilePathsForSiteRename -OldSiteUrl [https://contoso.sharepoint.com/sites/site1](https://contoso.sharepoint.com/sites/site1) -NewSiteUrl [https://fabrikam.sharepoint.com/sites/site1](https://fabrikam.sharepoint.com/sites/site1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Find-SPOCrossTenantLongFilePathsForSiteRename -OldSiteUrl [https://contoso.sharepoint.com/sites/site1](https://contoso.sharepoint.com/sites/site1) -NewSiteUrl [https://fabrikam.sharepoint.com/sites/site1](https://fabrikam.sharepoint.com/sites/site1) | |
| https://fabrikam.sharepoint.com/sites/site1 | |
| ``` #Closed |
…l/Find-SPOCrossTenantLongFilePathsForSiteRename.md Co-authored-by: Sam M. Kabue <[email protected]>
|
Learn Build status updates of commit 88dd1bc: ✅ Validation status: passed
For more details, please refer to the build report. |
vgaddam-pm
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated according to suggestions
|
Learn Build status updates of commit d71cb69: ✅ Validation status: passed
For more details, please refer to the build report. |
|
Learn Build status updates of commit 7fd35bc: ✅ Validation status: passed
For more details, please refer to the build report. |
|
|
||
| ### Example 1 | ||
| ```powershell | ||
| https://fabrikam.sharepoint.com/sites/site1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Find-SPOCrossTenantLongFilePathsForSiteRename -OldSiteUrl https://contoso.sharepoint.com/sites/site1 -NewSiteUrl https://fabrikam.sharepoint.com/sites/site1
``` #Closed
vgaddam-pm
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed example 1
|
Learn Build status updates of commit 56f5781: ✅ Validation status: passed
For more details, please refer to the build report. |
samkabue
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
![]()
No description provided.