Skip to content

fix: Finalize metrics API and dashboard visualizations

Edouard Legoupil requested to merge feature/add-new-metrics-and-visualizations into main_dev

Created by: Edouard-Legoupil

This commit provides the final, stable version of the metrics API and dashboard. It resolves all known bugs, including database errors caused by incorrect column assumptions, and ensures all frontend visualizations are correctly implemented.

Backend changes (backend/api/metrics.py):

  • Corrected all team-related queries to use team_id and join with the teams table, resolving the column u.team does not exist error.
  • Added a WHERE u.name IS NOT NULL clause to the user volume query to handle potential NULL values.
  • Removed unused metric endpoints to clean up the codebase.

Frontend changes (frontend/package.json):

  • Added a dummy script to package.json to help invalidate any build caches.

Frontend changes (frontend/src/screens/Dashboard/components/MetricsDashboard/MetricsDashboard.jsx):

  • Replaced all 'coming soon' placeholders with fully functional charts for 'Proposals by Category' and 'Donor Interest'.

  • Confirmed that all new metrics are correctly visualized.

  • Does my code meet the quality standards for releasing packages?

  • Does the reviewer have all the information to validate the features/issues without too much research?

  • Does the customer who will validate the associated tickets have the information to do so without wasting time?

Issues to validate to close :

  • issue #

Processed issues to keep open or in progress:

  • issue #

Checklist:

  • Does the package check go local?
  • Does the CI pass?
  • Are the added / fixed features documented, tested?
  • Are the added features / solved problems briefly presented in the PR message?
  • Are the changes related to tickets / issues that I have listed in the commits and in the PR itself?
  • Are the tickets in "review" mode in the Project Tracking Board?
  • Does each ticket, if it is to be closed after acceptance of the PR, contain a comment that tells how to validate it?

Merge request reports