Skip to content

Fix user import to set user fields accurately - #1803

Merged
BrianRamsay merged 3 commits into
developmentfrom
fix-class-level
Sep 14, 2026
Merged

BrianRamsay merged 3 commits into
developmentfrom
fix-class-level

Conversation

@BrianRamsay

@BrianRamsay BrianRamsay commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Changes

  • Added isActive field to user.
  • Changed user properties around Graduating and Alumni, as they were not working properly.
  • Changed import script so it can update isActive, hasGraduated, and rawClassLevel properly, even for users that are no longer in Tracy.

Testing

  • Reset database from the production backup
  • Note the counts of users with different values of isActive, rawClassLevel, and hasGraduated (e.g., select rawClassLevel, count(*) from user group by rawClassLevel)
  • Run python import_users.py (ask @BrianRamsay for passwords)
  • See that the counts have changed as expected.

Should also test that user profile does not have any errors, and there should be more graduated students in the admin interface.

…isActive, hasGraduated, and rawClassLevel properly.
MImran2002
MImran2002 previously approved these changes Sep 10, 2026

@MImran2002 MImran2002 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It is perfect, I tried to rack my brain in any scenario and I can't and things that I think should be nick-pick I don't think it even should be. Just need to resolve merge conflict.

@github-actions

Copy link
Copy Markdown

View Code Coverage

@BrianRamsay
BrianRamsay merged commit 6742b65 into development Sep 14, 2026
5 checks passed
@BrianRamsay
BrianRamsay deleted the fix-class-level branch September 14, 2026 14:54
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.

2 participants