Skip to content

Fix description of Import action parameters#10253

Merged
ktoliver merged 1 commit intoMicrosoftDocs:livefrom
ducciomugnaini-ascom:patch-1
Dec 18, 2025
Merged

Fix description of Import action parameters#10253
ktoliver merged 1 commit intoMicrosoftDocs:livefrom
ducciomugnaini-ascom:patch-1

Conversation

@ducciomugnaini-ascom
Copy link
Copy Markdown
Contributor

Bug: Incorrect wording on "SqlPackage Import" page (SQL Server v17)

Article URL

Affected section

  • Introduction / description of the Import action

Observed error

  • Current text:

    The Export action requires a target connection where a new database will be created by SqlPackage or where a blank database is present. This is specified either through a combination of:

  • Problem: The sentence refers to Export, but the page is about Import.

Proposed correction

  • Correct text:

    The Import action requires a target connection where a new database will be created by SqlPackage or where a blank database is present. This is specified either through a combination of:

Reason

  • The page documents the Import action, so mentioning Export is a typo.

Impact

  • This may confuse readers and lead them to think the syntax applies to Export.

Suggestion

  • Check if other sections contain incorrect references to Export and fix them.

doc-bug sqlpackage import `sql-server-ver17

@prmerger-automator
Copy link
Copy Markdown
Contributor

@ducciomugnaini-ascom : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@ktoliver
Copy link
Copy Markdown
Contributor

@dzsquared

Can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged tracking label for the PR review team label Dec 18, 2025
@dzsquared
Copy link
Copy Markdown
Member

@ducciomugnaini-ascom great catch, thank you for the PR
#sign-off

@ktoliver
Copy link
Copy Markdown
Contributor

Closing and reopening the PR to see if the build will finish

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit da9846b:

💡 Validation status: suggestions

File Status Preview URL Details
docs/tools/sqlpackage/sqlpackage-import.md 💡Suggestion Details

docs/tools/sqlpackage/sqlpackage-import.md

  • Line 10, Column 11: [Suggestion: value-deprecated-replace - See documentation] The 'ms.topic: conceptual' you used is now deprecated and can no longer be used. We suggest you replace it with 'ms.topic: article'.

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.

@ktoliver ktoliver merged commit ba26833 into MicrosoftDocs:live Dec 18, 2025
2 checks passed
@prmerger-automator
Copy link
Copy Markdown
Contributor

PR 10253 has been merged from ducciomugnaini-ascom:patch-1 to MicrosoftDocs:live by ktoliver.

@ducciomugnaini-ascom, @dzsquared

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants