Skip to content
This repository was archived by the owner on Apr 29, 2026. It is now read-only.

Better log handling for student activity section#328

Merged
clpetersonucf merged 4 commits intodjango-workingfrom
logs/student-activity-log-updates
Mar 11, 2026
Merged

Better log handling for student activity section#328
clpetersonucf merged 4 commits intodjango-workingfrom
logs/student-activity-log-updates

Conversation

@clpetersonucf
Copy link
Copy Markdown
Member

  • The student activity section will populate if ANY logs are available for a given instance, not just completed logs.
  • The distribution graph renders a warning message instead of summary content if no completed logs are available.
  • The Individual Scores tab renders a warning message instead of an error message if no logs are available (for example, if no logs are visible due to provisional access.)
  • The Export Options dialog renders an error message if the associated endpoint returns a non-200 error code, and specifically indicates that no logs are available if the response method is 404.

Copy link
Copy Markdown
Member

@FrenjaminBanklin FrenjaminBanklin left a comment

Choose a reason for hiding this comment

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

Provides better feedback in cases where there's nothing to report. Only one nitpick that's not really worth holding this up, otherwise looks good to me.

Comment thread app/api/serializers.py
@clpetersonucf clpetersonucf merged commit d58992a into django-working Mar 11, 2026
4 checks passed
@clpetersonucf clpetersonucf deleted the logs/student-activity-log-updates branch March 11, 2026 13:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants