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

Saving an app with invalid state bugs the UI #11

Open
robert-sanfeliu opened this issue Jul 12, 2024 · 0 comments
Open

Saving an app with invalid state bugs the UI #11

robert-sanfeliu opened this issue Jul 12, 2024 · 0 comments
Assignees
@fotisp @vkefalas-exz
Labels
bug Something isn't working deployed A fix has been commited

Comments

@robert-sanfeliu
Copy link

How to replicate on dev environment:

  1. create a new APP.
  2. fill in the Vela
  3. remove all "Keys" an environment variables
  4. Click save. UI will complain because there are empty metrics/utility function
  5. Try to fix the issue, go to the affected steps of the wizzard and remove the empty metrics/utility function
  6. Try to save again. It shows the following error:
    image

Noting can be done to fix the error. The UI is bugged.
@robert-sanfeliu robert-sanfeliu added the bug Something isn't working label Jul 12, 2024
vkefalas-exz added a commit that referenced this issue Sep 2, 2024
@vkefalas-exz
Application saving and traversing through the ui fixed #8 #11
3d1eea4
@vkefalas-exz vkefalas-exz added the deployed A fix has been commited label Sep 2, 2024
fotisp added a commit that referenced this issue Sep 2, 2024
@fotisp @vkefalas-exz
Dev (#20)
4ea534c 
* Synchronizing code from bitbucket repository

* Removing TomSelect because it is no longer updated

* Metric template remove restriction

* Multiple environment variables fix

* - Setting dark mode by default in the login window
- Providing better view of current build id and build environment for debugging
- Added environment variables to set_env.sh so that they can be set during build

* Application saving and traversing through the ui fixed #8 #11

* Fixing ts errors on Error handling

* Adding build specific information for better debugging

---------

Co-authored-by: Vasilis Kefalas <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fotisp fotisp

@vkefalas-exz vkefalas-exz

Labels
bug Something isn't working deployed A fix has been commited
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@fotisp @robert-sanfeliu @vkefalas-exz