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.

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

Fail to add a new application setting which name start with number #4000

Closed
v-ruizh opened this issue Mar 1, 2024 · 1 comment · Fixed by #4006 or microsoft/vscode-azuretools#1705
Labels
AT-CTI Created by CTI team

Comments

@v-ruizh
Copy link

v-ruizh commented Mar 1, 2024

OS: All
Build Version: 20240229.2
Regression: Not a Regression

Repro Steps:

  1. Create a containerized function app in VS Code.
  2. Right click "Application Settings" node and select "Add New Setting...".
  3. Enter a name "234" as the application setting name -> Enter a value for this setting to create it.
  4. Check whether succeeds to add a new application setting with the name "234".

Expect:
Succeed to add a new application setting with the name "234".

Actual:
Fail to add a new application setting with the name "234".
image

More Info:

  1. There is no validation when entering "234" as the application setting name in the creation option.
  2. Succeed to add a new application setting with the name "234" in regular function app.
@v-ruizh v-ruizh added the AT-CTI Created by CTI team label Mar 1, 2024
@v-ruizh v-ruizh changed the title Fail to add a new application setting which named start with number Fail to add a new application setting which name start with number Mar 1, 2024
@v-ruizh
Copy link
Author

v-ruizh commented Mar 14, 2024

@motm32 I tried to create an app setting which name start with number from portal, it still failed with the below error. So the app setting name may be invalid if starts with number.
image

I tried this on the new extension build, there is an error when entering app setting name start with number.
image
It seems that the validation message is incorrect. If the app setting name can't begin with number, it would be better to remove the "number" word from the validation (the highlight part).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
AT-CTI Created by CTI team
Projects
None yet
1 participant