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

[ui] Custom Variables are not validated in Application edit mode #299

Closed
torchiaf opened this issue Aug 7, 2023 · 8 comments · Fixed by #325
Closed

[ui] Custom Variables are not validated in Application edit mode #299

torchiaf opened this issue Aug 7, 2023 · 8 comments · Fixed by #325
Assignees
Labels
kind/bug Something isn't working
Milestone

Comments

@torchiaf
Copy link
Contributor

torchiaf commented Aug 7, 2023

Setup

  • Epinio UI version: 1.9.0

To Reproduce

1- Edit an application having at least a custom variable with constraints (i.e. numeric variable with min value)
2- Set an invalid value. (i.e. min=5 value= 2)

Result
The Save button is enabled

Expected Result

Save button should be disabled.

Screenshots

image

@torchiaf torchiaf added the kind/bug Something isn't working label Aug 7, 2023
@torchiaf torchiaf self-assigned this Aug 7, 2023
@torchiaf torchiaf added this to UI Aug 7, 2023
@torchiaf torchiaf added this to the v1.10.0 milestone Aug 7, 2023
@andreas-kupries andreas-kupries moved this to Icebox in Epinio planning Aug 8, 2023
@torchiaf torchiaf moved this to In progress in UI Aug 8, 2023
@andreas-kupries andreas-kupries moved this from Icebox to Scheduled in Epinio planning Aug 21, 2023
@torchiaf torchiaf moved this from In progress to Review in UI Sep 1, 2023
@richard-cox richard-cox moved this from Scheduled to In Progress in Epinio planning Sep 4, 2023
@github-project-automation github-project-automation bot moved this from Review to Done in UI Sep 4, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Done in Epinio planning Sep 4, 2023
@torchiaf torchiaf moved this from Done to To Test in UI Sep 4, 2023
@torchiaf torchiaf reopened this Sep 4, 2023
@github-project-automation github-project-automation bot moved this from To Test to Next Up in UI Sep 4, 2023
@richard-cox richard-cox moved this from Next Up to To Test in UI Sep 4, 2023
@thehejik
Copy link
Contributor

thehejik commented Sep 5, 2023

EDIT: 09/23 the comment is unrelated, this issue is about Edit Config on Application (not Service)

Edit mode still not working, everything inactive so can't say if the values are validated but I can still press Save even if the form is inactive (the valid values entered when creating the service).

I guess some piece is still missing for the service Edit Mode enablement.
image

@thehejik
Copy link
Contributor

thehejik commented Sep 6, 2023

Moving to In Progress until #252 is in.

@thehejik thehejik moved this from To Test to In progress in UI Sep 6, 2023
@richard-cox
Copy link
Member

@thehejik This issue is about application chart values, rather than service chart values. Would it be ok to create a validation issue for 1.11 for service chart values?

@richard-cox
Copy link
Member

I've added a note to the edit service values issue to ensure values are validated #252 (comment).

@richard-cox richard-cox moved this from In progress to To Test in UI Sep 18, 2023
@thehejik thehejik moved this from To Test to In progress in UI Sep 18, 2023
@thehejik thehejik moved this from In progress to To Test in UI Sep 18, 2023
@thehejik
Copy link
Contributor

thehejik commented Sep 18, 2023

The validation works absolutely ok in chromium/vivaldi.

But doesn't work at all in Firefox, note the appListeningPort is integer with min: 0
image

@thehejik
Copy link
Contributor

This is from chromium/vivaldi, all the fields are validated and Save button is Inactive when even one value is out of range, also I was unable to write any letters into integer/number fields.

image

@thehejik
Copy link
Contributor

Given the PR is merged and firefox issue opened I'm closing this one.

@github-project-automation github-project-automation bot moved this from To Test to Done in UI Sep 19, 2023
@mmartin24
Copy link
Contributor

Verified in v1.10.0-rc2 using Brave browser

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
Archived in project
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants