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

Learning insights dash #624

Merged
merged 15 commits into from
Jan 13, 2025
Merged

Conversation

corypride
Copy link
Contributor

Description of the change

Update the Admin Layer 2 Dashboard for student activity (named Learning Insights) so that it shows the data relative to courses, student enrollments and activity time, all which have the option to be filtered by facility

Screenshot(s)

@corypride
Copy link
Contributor Author

corypride commented Jan 7, 2025 via email

@corypride corypride requested a review from calisio January 8, 2025 16:33
@corypride corypride marked this pull request as ready for review January 8, 2025 16:33
@corypride corypride requested a review from calisio January 9, 2025 13:18
Copy link
Contributor

@jtucholski jtucholski left a comment

Choose a reason for hiding this comment

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

I added some comments about splitting up the GetAdminLayer2Info function call and leaving the logic to aggregate it in the handler. This is similar to what we also arrived at for the Operational Insights.

backend/src/database/activity.go Outdated Show resolved Hide resolved
backend/src/database/activity.go Show resolved Hide resolved
backend/src/database/activity.go Outdated Show resolved Hide resolved
@corypride corypride requested a review from jtucholski January 9, 2025 23:52
@corypride corypride requested a review from PThorpe92 January 12, 2025 03:55
@PThorpe92 PThorpe92 merged commit 0a8c319 into UnlockedLabs:main Jan 13, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants