Skip to content

Conversation

@netrajpatel
Copy link
Contributor

No description provided.

@netrajpatel netrajpatel changed the base branch from main to development June 9, 2025 14:47

This comment was marked as outdated.

cs-raj
cs-raj previously approved these changes Jun 12, 2025
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@netrajpatel netrajpatel marked this pull request as ready for review June 12, 2025 09:44
@netrajpatel netrajpatel requested a review from a team as a code owner June 12, 2025 09:44
aman19K
aman19K previously approved these changes Jun 12, 2025

This comment was marked as outdated.

@netrajpatel netrajpatel requested a review from Copilot June 25, 2025 14:07
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 extends the CLI’s region support by adding the new AWS-AU region everywhere it’s used: configuration commands, helper utilities, integration tests, and documentation.

  • Added AWS-AU (and related GCP-EU/GCP-NA) entries to region handler, interactive prompts, config mappings, and personalize endpoints.
  • Refactored integration tests to import a centralized REGION_MAP and initialize new regions.
  • Bumped package versions, updated README usage examples, and cleaned up test stubs.

Reviewed Changes

Copilot reviewed 68 out of 72 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
packages/contentstack-config/src/utils/region-handler.ts Added AU and AWS-AU in region definitions
packages/contentstack-config/src/utils/interactive.ts Updated region choices for prompts
packages/contentstack-config/messages/index.json Default region message updated to AWS-NA
packages/contentstack-utilities/messages/config.json Default region message updated to AWS-NA
packages/contentstack-utilities/src/helpers.ts Normalized path sanitization formatting
packages/contentstack-config/test/unit/commands/region.test.ts Expanded unit tests for AWS-* regions and removed old stubs
packages/contentstack-import/src/config/index.ts Extended personalize baseURL mapping for new regions
packages/contentstack-import/test/integration/config.json Included AWS-AU in test configurations
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (1)

packages/contentstack-config/src/utils/interactive.ts:9

  • The code still accepts plain 'AU' in the region handler but the interactive choices no longer include {name: 'AU', value: 'AU'}. For consistency, either add the plain 'AU' choice here or remove support for 'AU' in the handler.
      { name: 'AWS-NA', value: 'AWS-NA' },

@netrajpatel netrajpatel requested review from aman19K and shafeeqd959 and removed request for aman19K July 2, 2025 07:21
@harshithad0703 harshithad0703 merged commit 985209d into development Jul 3, 2025
10 checks passed
@harshithad0703 harshithad0703 deleted the feat/dx-3117 branch July 3, 2025 11:54
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.

7 participants