Skip to content

Improve amendment list performance#5748

Merged
bastianjoel merged 8 commits intoOpenSlides:mainfrom
bastianjoel:amendment-list-performance
Mar 16, 2026
Merged

Improve amendment list performance#5748
bastianjoel merged 8 commits intoOpenSlides:mainfrom
bastianjoel:amendment-list-performance

Conversation

@bastianjoel
Copy link
Copy Markdown
Member

@bastianjoel bastianjoel commented Jan 17, 2026

resolves #3014

@bastianjoel bastianjoel self-assigned this Jan 19, 2026
@Elblinator
Copy link
Copy Markdown
Member

can this be merged before/after/ doesn't matter of #5340

@bastianjoel bastianjoel marked this pull request as draft January 20, 2026 16:09
@bastianjoel
Copy link
Copy Markdown
Member Author

This is not finished

@bastianjoel bastianjoel marked this pull request as ready for review February 11, 2026 20:52
@bastianjoel bastianjoel changed the title Improve motion diff performance Improve amendment list performance Feb 11, 2026
@bastianjoel bastianjoel assigned Elblinator and MSoeb and unassigned bastianjoel Feb 11, 2026
@Elblinator Elblinator assigned bastianjoel and unassigned Elblinator and MSoeb Feb 12, 2026
@Elblinator Elblinator added enhancement General enhancement which is neither bug nor feature performance labels Feb 12, 2026
@bastianjoel bastianjoel assigned Elblinator and MSoeb and unassigned bastianjoel Feb 12, 2026
@Elblinator Elblinator removed their assignment Feb 12, 2026
@bspekker bspekker requested review from bspekker February 20, 2026 13:27
@bspekker bspekker self-assigned this Feb 20, 2026
Copy link
Copy Markdown

@bspekker bspekker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small improvements.

@bastianjoel
Copy link
Copy Markdown
Member Author

This should improve the performance drastically.
If not there is either something wrong with this PR or we should not merge this.

@Elblinator Elblinator removed the request for review from bspekker March 3, 2026 14:24
@Elblinator Elblinator added this to the 4.4 milestone Mar 3, 2026
@bastianjoel bastianjoel force-pushed the amendment-list-performance branch from 2936b69 to 5e469da Compare March 13, 2026 09:28
@bastianjoel bastianjoel assigned bspekker and Elblinator and unassigned MSoeb Mar 13, 2026
Copy link
Copy Markdown
Member

@Elblinator Elblinator left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code okay but linter fails

@Elblinator Elblinator assigned bastianjoel and unassigned Elblinator Mar 13, 2026
@bastianjoel bastianjoel force-pushed the amendment-list-performance branch from 42f3437 to b0793d2 Compare March 13, 2026 13:00
@bastianjoel bastianjoel removed their assignment Mar 13, 2026
@bastianjoel bastianjoel merged commit 97e7d53 into OpenSlides:main Mar 16, 2026
3 checks passed
@bastianjoel bastianjoel deleted the amendment-list-performance branch March 16, 2026 08:40
bastianjoel added a commit to bastianjoel/openslides-client that referenced this pull request Mar 16, 2026
Borreg0 pushed a commit to Borreg0/openslides-client that referenced this pull request Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement General enhancement which is neither bug nor feature performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Motion list / amendment list: Performance Issue

4 participants