Skip to content

Conversation

@yashin4112
Copy link
Contributor

This PR includes windows support for Sitecore CMS.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds Windows support for Sitecore CMS by addressing cross-platform path separator compatibility issues. The changes enable the migration tooling to work correctly on Windows systems by using the platform-specific path separator instead of hardcoded Unix-style forward slashes.

  • Adds 'blob' folder to the list of recognized Sitecore folders
  • Updates path splitting logic to use platform-specific separators (path.sep) instead of hardcoded forward slashes

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
upload-api/src/helper/index.ts Adds 'blob' to the sitecoreFolders array to recognize an additional Sitecore folder structure
upload-api/migration-sitecore/libs/contenttypes.js Replaces hardcoded '/' separator with path.sep in two locations for cross-platform path handling

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sayalijoshi27 sayalijoshi27 merged commit 24c201a into dev Dec 17, 2025
7 checks passed
@sayalijoshi27 sayalijoshi27 deleted the feature/windows branch December 17, 2025 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants