Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

refactor compare reasons #200

Open
ddfridley opened this issue Sep 3, 2024 · 0 comments · Fixed by #266
Open

refactor compare reasons #200

ddfridley opened this issue Sep 3, 2024 · 0 comments · Fixed by #266
Assignees
@ddfridley
Labels
javascript React

Comments

@ddfridley
Copy link
Contributor

ddfridley commented Sep 3, 2024 •
edited
Loading

Refactor app/components/compare-reasons.jsx

See the Step Component Pattern

And see specifics in the Civil Pursuit Deliberation spreadsheet

see app/components/steps/rerank.js for an example of a component refactored in this way

see app/components/steps/tests/rerank.js for a jest test of it's deriver function

see stories/rerank-step.stories.js for a refactored jest test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ddfridley ddfridley

Labels
javascript React
Projects
EnCiv Cross Repo Project Management
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Refactor Compare whys#200 EnCiv/civil-pursuit
1 participant
@ddfridley