Skip to content

fix: modularize admin dashboard statistics with reusable helpers - #555

Merged
tudoramariei merged 1 commit into
mainfrom
cache-metrics
Nov 3, 2025
Merged

fix: modularize admin dashboard statistics with reusable helpers#555
tudoramariei merged 1 commit into
mainfrom
cache-metrics

Conversation

@tudoramariei

Copy link
Copy Markdown
Member
  • extract yearly stats logic into stats_helpers_yearly.py
  • extract metrics-related logic into stats_helper_metrics.py
  • extract chart-related logic into stats_helper_chart.py
  • refactor admin dashboard to use new helpers for stats and metrics

@tudoramariei
tudoramariei requested review from Copilot and danniel July 15, 2025 09:31

This comment was marked as outdated.

This comment was marked as resolved.

@tudoramariei
tudoramariei marked this pull request as ready for review October 30, 2025 09:06
@tudoramariei
tudoramariei force-pushed the cache-metrics branch 6 times, most recently from 4139ca5 to 5fb173f Compare November 3, 2025 10:27
@tudoramariei
tudoramariei requested a review from Copilot November 3, 2025 10:28

This comment was marked as resolved.

@tudoramariei
tudoramariei force-pushed the cache-metrics branch 3 times, most recently from 5dede4a to bbaf912 Compare November 3, 2025 11:47
@tudoramariei
tudoramariei requested a review from Copilot November 3, 2025 11:47

Copilot AI 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.

Pull Request Overview

Copilot reviewed 23 out of 28 changed files in this pull request and generated 8 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docker/s6-rc.d/init/init.sh Outdated
Comment thread backend/donations/management/commands/generate_redirections_stats.py Outdated
Comment thread backend/stats/api.py Outdated
Comment thread backend/donations/models/stat_configs.py
Comment thread backend/donations/models/stat_configs.py
Comment thread backend/donations/models/stat_configs.py
Comment thread backend/donations/models/stat_configs.py
Comment thread backend/donations/management/commands/schedule_stats_generator_ngos.py Outdated
- create a stats model to cache all stats
- create schedulers and tasks to update stats
- refactor schedulers to use standard template
@tudoramariei
tudoramariei merged commit b133f6f into main Nov 3, 2025
14 checks passed
@tudoramariei
tudoramariei deleted the cache-metrics branch November 3, 2025 14:07
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.

3 participants