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

Motion detail view with amendments: Changing amendment status needs reload #4303

Open
MSoeb opened this issue Oct 29, 2024 · 7 comments · Fixed by #4412 · May be fixed by #4491
Open

Motion detail view with amendments: Changing amendment status needs reload #4303

MSoeb opened this issue Oct 29, 2024 · 7 comments · Fixed by #4412 · May be fixed by #4491
Assignees
Labels
Milestone

Comments

@MSoeb
Copy link

MSoeb commented Oct 29, 2024

Describe the bug
It is possible to change the status of an amendment in the motion detail view, but the change is not displayed. The status switches up, but the old status is still displayed. An update that updates the visulization seems to be missing here.

How to Reproduce
Steps to reproduce the behavior:

  1. open a meeting
  2. open motions > create a motion with an amendment
  3. open the motion in the detail view
  4. mark the checkbox 'Show all changes' > try to change the amendment status via the three-dot-menu behind the amendment

Expected behavior
Changing the amendmend status via the three-dot-menu should have visual effect. New status should be immediately visible

@MSoeb MSoeb added the bug label Oct 29, 2024
@MSoeb MSoeb added this to the 4.2 milestone Oct 29, 2024
@bastianjoel bastianjoel self-assigned this Nov 12, 2024
@bastianjoel
Copy link
Member

Cannot reproduce

@bastianjoel bastianjoel assigned MSoeb and unassigned bastianjoel Nov 12, 2024
@Elblinator
Copy link
Member

Elblinator commented Nov 20, 2024

Reproduction:

  1. Create amotion
  2. Create Two Amendments
  3. Navigate to the motion detail view
  4. Tick "Show all changes" box
  5. Update state from amendment

I have the theory that the states update if you update the state from every amendment
If you update the second amendment in after step 5 then the states on the page update

@reiterl
Copy link
Member

reiterl commented Nov 27, 2024

It seems like the changes in the MotionDetailDiffComponent doesn't change after a setAmendmentState or a resetAmendmentState.

@reiterl
Copy link
Member

reiterl commented Nov 27, 2024

The AmendementControlerService has a strange timestamplogic in getViewModelListObservableFor() in it, which breaks the stuff.

@bastianjoel
Copy link
Member

Fix should be applied to motion-change-recommendation-controller.service.ts too.

@bastianjoel
Copy link
Member

While at it my comments in #4412 should also be implemented. Might be a good idea to add an helper for changed checks for view model lists.

Also it might need to be checked if viewModelUpdateTimestamp works correctly or if there are any cases in the base repo where the proxy object is skipped.

@Elblinator Elblinator modified the milestones: 4.2, 4.3 Dec 13, 2024
@reiterl reiterl removed their assignment Dec 16, 2024
@reiterl
Copy link
Member

reiterl commented Dec 16, 2024

Yes, nice idea. Deassigned.

@bastianjoel bastianjoel self-assigned this Dec 16, 2024
@bastianjoel bastianjoel linked a pull request Dec 16, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment