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

Clear Add Namespace and Add secret form data on close #1771

Merged
merged 1 commit into from Dec 28, 2020

Conversation

pashevskii
Copy link
Contributor

@pashevskii pashevskii commented Dec 14, 2020

The commit fixes clearing data on closing next forms:
Configuration - Secrets - Add secret
Namespaces - Add Namespace

[Fixed]There is some UI issue: validation of inputs is triggered earlier than form is closed despite that I clear data after isOpen property switching, how can I fix it?
[Solution] Reset data before form appearing

@pashevskii pashevskii changed the title Clear Add Namespace and Add secret form data on close [WIP]Clear Add Namespace and Add secret form data on close Dec 14, 2020
@pashevskii pashevskii changed the title [WIP]Clear Add Namespace and Add secret form data on close Clear Add Namespace and Add secret form data on close Dec 14, 2020
Signed-off-by: Pavel Ashevskii <pashevskii@mirantis.com>
@Nokel81 Nokel81 requested review from aleksfront and a team December 14, 2020 17:31
@Nokel81 Nokel81 added area/ui enhancement New feature or request labels Dec 14, 2020
@jakolehm jakolehm added this to the 4.1.0 milestone Dec 28, 2020
@jakolehm
Copy link
Contributor

@aleksfront @ixrock PTAL

Copy link
Contributor

@aleksfront aleksfront left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aleksfront aleksfront merged commit a325ffd into lensapp:master Dec 28, 2020
@pashevskii pashevskii deleted the fix-clear-form-data-on-close branch December 28, 2020 09:45
@jakolehm jakolehm mentioned this pull request Jan 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ui enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants