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

Editing a Display Layout and saving throws some errors #4681

Closed
2 of 5 tasks
khalidadil opened this issue Jan 6, 2022 · 2 comments · Fixed by #4687
Closed
2 of 5 tasks

Editing a Display Layout and saving throws some errors #4681

khalidadil opened this issue Jan 6, 2022 · 2 comments · Fixed by #4687
Labels
bug:regression It used to work. Now it doesn't :( severity:medium type:bug
Projects
Bug Tracker

Comments

@khalidadil
Copy link
Contributor

khalidadil commented Jan 6, 2022 •
edited by unlikelyzero

Summary

When editing a display layout, editing the layout and then canceling editing throws an error (see screen recording attached).

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. Navigate to a display layout and click the edit button.
  2. Click the cancel editing button. You should see errors in the developer tools.

Additional Information

Screen.Recording.2022-01-06.at.12.25.02.PM.mov

Screen Shot 2022-01-06 at 12 11 16 PM
@unlikelyzero unlikelyzero added bug:regression It used to work. Now it doesn't :( severity:medium labels Jan 6, 2022
@unlikelyzero unlikelyzero added this to Needs triage in Bug Tracker via automation Jan 6, 2022
@shefalijoshi shefalijoshi mentioned this issue Jan 7, 2022
Merged
8 tasks
@khalidadil khalidadil linked a pull request Jan 7, 2022 that will close this issue
Check for ref when trying to get the style receiver. #4687
Merged
8 tasks
Bug Tracker automation moved this from Needs triage to Done Jan 7, 2022
@nikhilmandlik
Copy link
Contributor

Verified Fixed.

@khalidadil
Copy link
Contributor Author

Verified Fixed in Testathon on 01/07/2022

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 :( severity:medium type:bug
Projects
Bug Tracker
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Check for ref when trying to get the style receiver. nasa/openmct
4 participants
@nikhilmandlik @khalidadil @shefalijoshi @unlikelyzero