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

[Statistics] Fix site statistics page loading #9354

Merged
merged 1 commit into from
Sep 24, 2024

Conversation

driusan
Copy link
Collaborator

@driusan driusan commented Sep 24, 2024

Fix type errors on the page loaded from the link on the LORIS dashboard for "data entry incomplete" in my tasks.

Fixes #9353

@regisoc
Copy link
Contributor

regisoc commented Sep 24, 2024

@driusan tested on my end, but one automatic test failed.

This is what I can see now, which looks good.
Recording 2024-09-24 at 14 12 47

@driusan
Copy link
Collaborator Author

driusan commented Sep 24, 2024

@regisoc I think the test failure is an unrelated failure on the main branch.. I've already asked @kongtiaowang to look into it (tests were added that passed but I think there were incompatible changes on the main branch between the time the PR was sent and the time it was merged.)

Copy link
Contributor

@regisoc regisoc left a comment

Choose a reason for hiding this comment

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

I will approve it now because it looks good to me.
@kongtiaowang Let me know if this failed test is not linked to the mentioned incompatibility.

@kongtiaowang
Copy link
Contributor

kongtiaowang commented Sep 24, 2024

@regisoc #9355 will fix it. This failed test is not linked to this PR.

Fix type errors on the page loaded from the link on the
LORIS dashboard for "data entry incomplete" in my tasks.

Fixes aces#9353
@driusan driusan closed this Sep 24, 2024
@driusan driusan reopened this Sep 24, 2024
@driusan driusan merged commit e207dee into aces:main Sep 24, 2024
10 checks passed
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.

[statistics] 500 coming from dashboard
3 participants