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

Route container config - logMask switching to false for the first time not persisted #861

Closed
tplevko opened this issue Feb 22, 2024 · 3 comments · Fixed by #980
Closed
Labels
bug Something isn't working canvas/config-form Configuration form related issues
Milestone

Comments

@tplevko
Copy link
Contributor

tplevko commented Feb 22, 2024

Describe the Bug

There is an issue when configuring the route container using the side panel - the logMask checkbox property, which has default value set to true is not persisted when switched to false for the first time. This results in the default value to be reset when switched back to the canvas. See video.

Steps to Reproduce the Bug or Issue

  1. open the route configuration sidebar
  2. switch the "logMask" property to false
  3. swith to code editor view - see the logMask wasn't configured
  4. switch back to canvas view - see the "logMask" is still set to true in the configuration sidebar

Screenshots or Videos

vokoscreenNG-2024-02-22_09-47-47.mp4

Note

Once this is fixed, we can reenable: https://github.com/KaotoIO/kaoto-next/blob/a13cb4c3dbaca25e2a4f7a5abce695df3d84edb5/packages/ui-tests/cypress/e2e/designer/rootContainersConf.cy.ts#L82

Platform

  • OS: [e.g. macOS, Windows, Linux]
  • Browser: [e.g. Chrome, Safari, Firefox]
  • Version: [e.g. 0.30.0]
@tplevko tplevko added the bug Something isn't working label Feb 22, 2024
@tplevko tplevko added the canvas/config-form Configuration form related issues label Feb 22, 2024
@lordrip
Copy link
Member

lordrip commented Feb 22, 2024

This might be happening for other forms as well 🤔

@lordrip
Copy link
Member

lordrip commented Feb 22, 2024

Oh, if I try the same, using the daemon property of the timer component, it behaves as expected, so maybe it's related to the root container only.

@tplevko
Copy link
Contributor Author

tplevko commented Feb 22, 2024

Actually it's strange, because there is Auto Startup property in the root container, which is also preset to true and it works as expected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working canvas/config-form Configuration form related issues
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants