-
Notifications
You must be signed in to change notification settings - Fork 23
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 "invalid scale configuration" error and more #1003
base: main
Are you sure you want to change the base?
Conversation
The statistics page (and statistics tab in the event and series details) is currently broken. The web console shows the error "Invalid scale configuration for scale: XAxes". This fixes the error and a few less critical bugs to get the statistics page up and running again.
Use Run test server using develop.opencast.org as backend:
Specify a different backend like stable.opencast.org:
It may take a few seconds for the interface to spin up. |
This pull request is deployed at test.admin-interface.opencast.org/1003/2024-12-02_15-03-46/ . |
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.
Thanks for this fix! I have tested this PR with statistics data from our test system and the display of graphs, download etc. works as expected. The code changes look reasonable to me (and are certainly an improvement).
This pull request has conflicts ☹ |
Use Run test server using develop.opencast.org as backend:
Specify a different backend like stable.opencast.org:
It may take a few seconds for the interface to spin up. |
import type { ChartDataset, ChartOptions } from 'chart.js'; | ||
import type { ChartOptions } from 'chart.js'; | ||
import { AsyncThunk } from "@reduxjs/toolkit"; | ||
import { AsyncThunkConfig } from "@reduxjs/toolkit/dist/createAsyncThunk"; |
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.
Needs fixin', does not build w/o :)
...due to dependency updates
This pull request is deployed at test.admin-interface.opencast.org/1003/2025-01-31_15-25-49/ . |
This pull request has conflicts ☹ |
Fixes #282.
The statistics page (and statistics tab in the event and series details) is currently broken. The web console shows the error "Invalid scale configuration for scale: XAxes".
This fixes the error and a few less critical bugs to get the statistics page up and running again.
Unfortunately, testing this is not straightforward. To thoroughly test this, you would need an Opencast with a statistics provider set up and configured, which is more difficult than it sounds. The interested may check the docs: https://docs.opencast.org/develop/admin/#configuration/admin-ui/statistics/