Skip to content
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.

Sign up for GitHub

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

Jump to bottom

SqlCmdVariables are not getting populated at publish time #17682

Closed
Benjin opened this issue May 11, 2023 · 2 comments · Fixed by #17683
Closed

SqlCmdVariables are not getting populated at publish time #17682

Benjin opened this issue May 11, 2023 · 2 comments · Fixed by #17683
Assignees
@Benjin @kisantia
Labels
Area - SQL Project Bug Pri: 1 Priority 1 Triage: Done
Milestone
May 2023 Release

Comments

@Benjin
Copy link
Contributor

Benjin commented May 11, 2023

VSCode duplicate of microsoft/azuredatastudio#23079

Same issue is causing SqlCmdVariables to not get saved to publish profiles.
@Benjin Benjin added this to the May 2023 Release milestone May 11, 2023
@Benjin Benjin self-assigned this May 11, 2023
@Benjin Benjin mentioned this issue May 11, 2023
Merged
@Benjin Benjin reopened this May 11, 2023
@Benjin Benjin mentioned this issue May 11, 2023
Merged
@kisantia
Copy link
Contributor

Generate script is now failing with this error in the console after this was checked in:
image

@kisantia kisantia self-assigned this May 11, 2023
This was referenced May 11, 2023
Merged
Merged
@Benjin
Copy link
Contributor Author

Benjin commented May 15, 2023

Verified in release build

@Benjin Benjin closed this as completed May 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Benjin Benjin

@kisantia kisantia

Labels
Area - SQL Project Bug Pri: 1 Priority 1 Triage: Done
Projects
None yet
Milestone
May 2023 Release
Development

Successfully merging a pull request may close this issue.

Fixing bug where SQLCMD vars weren't getting JSONified microsoft/vscode-mssql
2 participants
@Benjin @kisantia