Amsterdam-Music-Lab / MUSCLE

An application to easily set up and run online listening experiments for music research.
https://www.amsterdammusiclab.nl/
MIT License
4 stars 1 forks source link

Refactor: ExperimentCollection, ExperimentCollectionDashboard, Header, Score & Rank components #1140

Closed drikusroor closed 6 days ago

drikusroor commented 2 weeks ago

This pull request includes several refactors to improve the codebase and add unit tests for the Rank component. The changes include removing an unused variable, updating import paths, moving components to their own files, updating prop types, and converting components to use TypeScript. Additionally, unit tests have been added for the Rank component to ensure its functionality. These changes aim to improve code organization, maintainability, and test coverage.

Resolves #1139

sentry-io[bot] commented 2 weeks ago

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: frontend/src/components/ExperimentCollection/ExperimentCollection.tsx

Function Unhandled Issue
ExperimentCollection TypeError: null is not an object (evaluating 't.theme') ExperimentCollection(src/components/ExperimentCollection/ExperimentColl...
Event Count: 17 Affected Users: 3

Did you find this useful? React with a 👍 or 👎