Merge branch Feature/aggregate statistics generator into develop

UniversityOfSaskatchewanCMPT371 / term-project-2024-team-4

Apache License 2.0

2 stars 2 forks source link

Merge branch Feature/aggregate statistics generator into develop #109

Closed AdamStanger closed 8 months ago

AdamStanger commented 8 months ago

Files added:

frontend/src/components/StatisticsPage.jsx: Page containing the tabe for all the projectile points in the catalogue
frontend/src/components/StatisticsModal.jsx: Modal for displaying charts for the generated statistics
frontend/tests/statisticsPage.unit.test.jsx

Modifications:

Functions in backend/controllers/aggregateStatisticsGenerator.js changed to be async to fix issue when passing data to the frontend
Unit tests for those functions updated based on refactor
Path to statistics page added (localhost:8080/stats)

brj496 commented 8 months ago

Please view my comments for some changes. Additionally, am I correct in assuming that all of the routes were changed because they now use the corresponding helperFile? Or were the changes to the route files accidental from the merge?

each route was changed to use its corresponding helperFile. They are correct as they are right now