Skip to content

Conversation

@carlo-kontent
Copy link
Contributor

@carlo-kontent carlo-kontent commented Aug 1, 2025

Motivation

Which issue does this fix?
New-found issue.

If no issue exists, what is the fix or new feature? Were there any reasons to fix/implement things that are not obvious?

Migrate Content & Sync Run suppress message flag (--sw) was incorrect (eg. suppress this message with --sw parameter) and did not match the suppress command flags for each command (eg. skipConfirmation). Migrate Content & Sync Run suppress flags and messages have been updated to match the Clean command (eg. --s)

Checklist

  • Code follows coding conventions held in this repo
  • Automated tests have been added
  • Tests are passing
  • Docs have been updated (if applicable)
  • Temporary settings (e.g. variables used during development and testing) have been reverted to defaults

How to test

When using the Migrate Content & Sync Run, the suppress flag has been updated from m--sw to --s. An alias for each command has also been created.

… (eg. suppress this message with --sw parameter) and did not match the suppress command flags for each command (eg. skipConfirmation). Migrate Content & Sync Run suppress flags and messages have been updated to match the Clean command (eg. --s)
@carlo-kontent carlo-kontent requested review from a team and JiriLojda as code owners August 1, 2025 04:21
@IvanKiral
Copy link
Contributor

can you please also add information about the parameter to the migrateContent run command readme.md?

.option("skipConfirmation", {
type: "boolean",
describe: "Skip confirmation message.",
alias: "s",
Copy link
Contributor

Choose a reason for hiding this comment

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

we use "s" as alias for sourceEnvironmentId

.option("skipConfirmation", {
type: "boolean",
describe: "Skip confirmation message.",
alias: "s",
Copy link
Contributor

Choose a reason for hiding this comment

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

we use "s" as alias for sourceEnvironmentId

@IvanKiral IvanKiral closed this Aug 1, 2025
@carlo-kontent carlo-kontent deleted the suppress_message branch August 1, 2025 07:30
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.

2 participants