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

After a Domain Object is modified, composition changes are not being saved #5097

Closed
2 of 5 tasks
charlesh88 opened this issue Apr 21, 2022 · 5 comments · Fixed by #5132
Closed
2 of 5 tasks

After a Domain Object is modified, composition changes are not being saved #5097

charlesh88 opened this issue Apr 21, 2022 · 5 comments · Fixed by #5132
Labels
help_wanted Help the Open MCT project! needs:e2e Needs an e2e test severity:critical type:bug

Comments

@charlesh88
Copy link
Contributor

charlesh88 commented Apr 21, 2022

Summary

When creating new or editing an existing domain object, composition changes are sporadically not being persisted. For example, only 2 out of 3 objects added to an object will be persisted. This seems to occur more frequently with newly created objects, but has been observed occurring with previously created objects as well.

Expected vs Current Behavior

All edit actions should be persisted on the Save action.

Steps to Reproduce

  1. Create a domain object that has composition (layout, plot, table, etc.)
  2. Add one or more domain objects to the composition via drag and drop.
  3. Modify the domain object (add styling, change layout if nested within a display layout, name change from browse bar, etc.)
  4. Add one or more domain objects to the composition via drag and drop.
  5. Save and Finish editing.
  6. Observe: Original composition changes prior to object modification are saved.
  7. Observe: Composition changes after object modification are not saved.

Environment

  • Open MCT Version:
  • Deployment Type:
  • OS: MacOS
  • Browser: Chrome

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?

Additional Information

@davetsay
Copy link
Contributor

davetsay commented Apr 26, 2022

Troubleshooting notes added to #5033

I haven't been able to replicate losing composition, but this is easily reproducible so hoping its related.

Was able to replicate this issue. #5033 is a separate issue.

@davetsay
Copy link
Contributor

If an object is modified, the composition changes (only composition add was tested) are dropped on save.

Examples

  • Add telemetry to flexible layout, modify number of panes, toggle horizontal/vertical panes, add more telemetry. Only the original telemetry will be saved.
  • Add telemetry to a table to a display layout, modify the table (change dimensions, apply style), add more telemetry. Only the original telemetry will be saved.

@davetsay davetsay changed the title Domain object composition is sporadically not being saved After a Domain Object is modified, composition changes are not being saved Apr 27, 2022
@shefalijoshi
Copy link
Contributor

I THINK it's fixed? Unless I didn't follow the steps correctly. :)

@rukmini-bose
Copy link
Contributor

Verified 05/02– Made 2 different flexible layouts and added different tables and such and everything seems to save correctly.

@unlikelyzero unlikelyzero added needs:e2e Needs an e2e test and removed unverified labels May 2, 2022
@unlikelyzero unlikelyzero added the help_wanted Help the Open MCT project! label May 3, 2022
@unlikelyzero
Copy link
Collaborator

e2e testing notes

  1. Create Telemetry object
  2. Create Flexible Layout
  3. Navigate to Flexible Layout
  4. Modify Number of Panes
  5. Add Telemetry
  6. Save
  7. Verify Number of Panes and Telemetry changes are both saved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help_wanted Help the Open MCT project! needs:e2e Needs an e2e test severity:critical type:bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants