Skip to content

Conversation

@brylie
Copy link
Member

@brylie brylie commented Apr 5, 2025

Introduce factories for caregiver roles and work models to streamline data creation. Enhance fake data generation with caregiver roles and work entries. Refactor duration calculations to use minutes for consistency. Update terminology in templates to reflect activities instead of work. Improve chart visualizations with a dark theme and transparent backgrounds. Add statsmodels dependency for statistical modeling.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 9 out of 11 changed files in this pull request and generated no comments.

Files not reviewed (2)
  • homes/templates/homes/home_detail.html: Language not supported
  • homes/templates/homes/home_detail_charts.html: Language not supported
Comments suppressed due to low confidence (1)

common/management/commands/make_fake_data.py:97

  • [nitpick] The subtraction for the date calculation is split across two lines, which may reduce code readability. Consider wrapping the expression in parentheses, for example: date=(today - datetime.timedelta(days=random.choice(ACTIVITY_DAYS_AGO))).
                        - datetime.timedelta(days=random.choice(ACTIVITY_DAYS_AGO)),

@brylie brylie merged commit 21af205 into main Apr 5, 2025
2 checks passed
@brylie brylie deleted the add-work-factories branch April 5, 2025 18:10
@github-project-automation github-project-automation bot moved this from Todo to Done in Caregivers Apr 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants