Skip to content

Conversation

@AlexVelezLl
Copy link
Member

Summary

  • Adds user type column to csv logs export
  • Adds a labels dictionary in the user_kinds module to store the translated user kind labels in the backend.

References

Closes #12674.

Reviewer guidance

  • Generate a session log, and summary logs for different types of users.
  • Go to Facility > Data and generate session and summary logs.
  • Check that users have the correct user type

@AlexVelezLl AlexVelezLl requested a review from rtibbles August 6, 2025 19:55
@github-actions github-actions bot added the DEV: backend Python, databases, networking, filesystem... label Aug 6, 2025
Copy link
Member

@rtibbles rtibbles left a comment

Choose a reason for hiding this comment

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

Code changes make sense to me - don't see anything in here that might be a performance concern either.

@pcenov pcenov self-requested a review August 7, 2025 13:05
Copy link
Member

@pcenov pcenov left a comment

Choose a reason for hiding this comment

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

Thanks @AlexVelezLl - I confirm that the User type column is added in both the session and summary logs csv files and I am seeing the correct values in it.

@AlexVelezLl
Copy link
Member Author

Thanks @pcenov @rtibbles!

@AlexVelezLl AlexVelezLl merged commit 8f455b1 into learningequality:develop Aug 7, 2025
52 checks passed
@AlexVelezLl AlexVelezLl deleted the add-user-type-to-csv-logs branch August 7, 2025 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DEV: backend Python, databases, networking, filesystem... SIZE: small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add user type information into the session and summary logs

3 participants