search

microsoft / vscode-azurestaticwebapps

Azure Static Web Apps extension for VS Code
https://marketplace.visualstudio.com/items?itemName=ms-azuretools.vscode-azurestaticwebapps
MIT License
92 stars 35 forks source link

[Suggestion] It would be better to add single quote for the setting name in the popped up error message #592

Closed v-xinda closed 2 years ago

v-xinda commented 2 years ago

OS: Win10 Build Version: 20211221.1 Regression: Not a regression

Repro Steps:

  1. Open the local.settings.json file -> Add a setting with name "tes t" (or contains special characters). image
  2. Right click the "Application Settings" -> Click "Upload Local Settings...".
  3. Check the popped up error message.

Actual: There is no single quote for the setting name in the popped up error message. image

Suggestion: It would be better to add single quote for the setting name in the popped up error message, it's convenient for reading.

nturinski commented 2 years ago

While I agree with this suggestion, the error is coming directly from Azure so we can't actually change the error message without intercepting and injecting quotes into it so we should close it.