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

Edit Properties and cancel throws console error #4864

Closed
1 of 5 tasks
charlesh88 opened this issue Feb 14, 2022 · 3 comments
Closed
1 of 5 tasks

Edit Properties and cancel throws console error #4864

charlesh88 opened this issue Feb 14, 2022 · 3 comments
Labels
bug:regression It used to work. Now it doesn't :( needs:e2e Needs an e2e test severity:trivial type:bug
Milestone
Target:2.0.7

Comments

@charlesh88
Copy link
Contributor

Summary

See Steps below.

Expected vs Current Behavior

No console errors.

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 with no functional impact?

Steps to Reproduce

  1. Display the dev tools console for your browser session.
  2. Edit Properties on any object view:

image

  1. From the Edit Properties dialog, click "Cancel".
  2. Observe in the console that an error is thrown:

image

Environment

  • Open MCT Version: 1.8.5
  • Deployment Type: /testathon
  • OS: All
  • Browser: All
@unlikelyzero unlikelyzero added the bug:regression It used to work. Now it doesn't :( label Feb 14, 2022
@akhenry akhenry assigned nikhilmandlik and unassigned akhenry Mar 22, 2022
@shefalijoshi shefalijoshi mentioned this issue Apr 8, 2022
Merged
15 tasks
@unlikelyzero unlikelyzero added the needs:e2e Needs an e2e test label May 18, 2022
@ozyx
Copy link
Contributor

ozyx commented Jun 9, 2022

Verified Testathon 6.9.2022

@davetsay
Copy link
Contributor

verified

@unlikelyzero unlikelyzero added this to the Sprint:2.0.7 milestone Jul 17, 2022
@unlikelyzero
Copy link
Contributor

We need an explicit test for this action

@ozyx ozyx mentioned this issue Sep 30, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug:regression It used to work. Now it doesn't :( needs:e2e Needs an e2e test severity:trivial type:bug
Projects
None yet
Milestone
Target:2.0.7
Development

No branches or pull requests
7 participants
@charlesh88 @nikhilmandlik @davetsay @akhenry @shefalijoshi @ozyx @unlikelyzero