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

[REF] runbot: rewrite stats page using owl #1035

Open
wants to merge 8 commits into
base: 18.0
Choose a base branch
from

Conversation

Williambraecky
Copy link
Contributor

Removing very old javascript and using new tools will allow us to more efficiently change and maintain the stat page in the futur. This commit is meant to produce a 1:1 copy of the stats page like it was before.
Further refactoring will happen to improve performance, layout and component logic.

Removing very old javascript and using new tools will allow us to more
efficiently change and maintain the stat page in the futur.
This commit is meant to produce a 1:1 copy of the stats page like it was
before.
Further refactoring will happen to improve performance, layout and
component logic.
Adds a new selection field to change trigger directly from the stat
page.
Instead of having the spinner in the configuration section, display it
as an overlay over the chart.
Rewrite the page layout to have good separation within components.
Adds support for multiple bundles at the same time in the stats page.
Support is pretty basic and visual does not differentiate between builds
from one bundle or another.
When selecting multiple bundles, differentiate stats between bundles
instead of displaying them all as the same.
Prevents chart data from being recomputed every time some display
settings change.
@Williambraecky Williambraecky marked this pull request as ready for review January 13, 2025 11:15
@C3POdoo C3POdoo requested a review from a team January 13, 2025 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant