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

Import as JSON dialog select file button not working properly #4241

Closed
3 of 5 tasks
charlesh88 opened this issue Sep 21, 2021 · 3 comments
Closed
3 of 5 tasks

Import as JSON dialog select file button not working properly #4241

charlesh88 opened this issue Sep 21, 2021 · 3 comments
Labels
bug:regression It used to work. Now it doesn't :( severity:blocker type:bug
Projects

Comments

@charlesh88
Copy link
Contributor

Summary

Seen during issues described in #4240.

Impact Check List

  • Data loss or misrepresented data?
  • Regression? Did this used to work or has it always been broken?
  • Is there a workaround available?
  • Does this impact a critical component?
  • Is this just a visual bug?

Steps to Reproduce

Setup
Use an Open MCT deployment in which the Clock notifier is not present. Make sure a Timer object is not the main view; Folder view is a good place to do this. Why? We're keeping anything out that would trigger an Angular digest cycle.

  1. Export anything.
  2. Import the exported file. In the dialog, click "Select File..." and select a file using the resulting OS picker. Press Enter.
  3. Observe: the Select button will hang on "Select File..." rather than switching to the selected file name.

Screen Shot 2021-09-21 at 2 41 30 PM

  1. Click the Select button again: it will immediately update to the name of the selected file, but also cause the OS picker to be displayed.
  2. Continuing the process at this point will allow you to import the file.

Expected behavior
Step 3 should not occur; the button should switch to the selected file name as soon as it can and the Ok button should become enabled.

Environment

  • Open MCT Version: 1.7.8
  • Deployment Type: /testathon
  • OS:
  • Browser:

Additional Information

@akhenry
Copy link
Contributor

akhenry commented Sep 21, 2021

This is almost certainly an Angular digest issue, and related to the removal of the Angular clock component. The Angular clock had the side effect of trigger regular Angular digests. The long term fix is to rewrite the Angular forms dialog in Vue, which is underway.

A potential short term fix is simply to manually trigger a digest once the user is done selecting a file (maybe onchange could be used on the input?).

@jvigliotta
Copy link
Contributor

Verified fixed - Testathon 9/30

@kobe1104
Copy link
Contributor

Verified fixed - testathon 9/30/2021

@unlikelyzero unlikelyzero added this to Needs triage in Bug Tracker via automation Sep 30, 2021
@unlikelyzero unlikelyzero moved this from Needs triage to Done in Bug Tracker Sep 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug:regression It used to work. Now it doesn't :( severity:blocker type:bug
Projects
Development

No branches or pull requests

6 participants