-
Notifications
You must be signed in to change notification settings - Fork 33
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
Fix motion list structure level #2828
Fix motion list structure level #2828
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
structure level and number is still missing in some views:
- autopilot (LoS, assignment poll)
- assignment-detail (candidate list, assignment poll)
This is not part of the issue nor the PR! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We do not want to add new fieldsets because within the subscriptions we do not have any direct reference to these and therefore cannot see which fields will be subscribed.
client/src/app/site/pages/meetings/pages/motions/motions.subscription.ts
Show resolved
Hide resolved
I did it anyway, since it's basically a copy-and-paste measure |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The structure level and number of motion supporter was not loaded in motion detail - after reload this page.
follow: [ | ||
{ | ||
idField: `user_id`, | ||
fieldset: [...UserFieldsets.FullNameSubscription.fieldset, `meeting_user_ids`] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removal of this tracked in #2789
Closes #2827