refactor(classifier): survey task Choice model #6087
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add a Choice model to the survey task. Model choices as a Map of Choice types, mapped by choice ID. Update the tests where they were testing a task snapshot rather than a task model.
Please request review from
@zooniverse/frontend
team or an individual member of that team.Package
Linked Issue and/or Talk Post
Similar to #6086, this refactors
task.choices
to usetypes.map(Choice)
, rather thantypes.frozen({})
.How to Review
I used the storybook to test the changes here. Snapshot Wisconsin (project slug
zooniverse/snapshot-wisconsin
) is also useful for checking if a survey task works.https://localhost:8080/?project=zooniverse/snapshot-wisconsin&env=production
Checklist
PR Creator - Please cater the checklist to fit the review needed for your code changes.
PR Reviewer - Use the checklist during your review. Each point should be checkmarked or discussed before PR approval.
General
yarn panic && yarn bootstrap
ordocker-compose up --build
and FEM works as expectedGeneral UX
Example Staging Project: i-fancy-cats
Refactoring