sillsdev / sillsdev/TheCombine
Replace redux in Character Inventory goal
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 22
- Forks
- 10
- Avg merge
- 2d 2h
- Merged PRs (30d)
- 9
Description
Redux is used in the character inventory goal to maintain the current list of valid and rejected characters, the word list, etc. All of the state information is cleared when the component is unmounted. This suggests that useState or useContext would be a better choice:
- the information is not shared across the application
- it would allow clearer manipulation of the data in the Character Inventory goal component
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the Character Inventory goal component and tracing its Redux state for valid and rejected characters, the word list, and related information. Check how the component is mounted and unmounted, then identify any existing tests for this goal. Done means the component no longer uses Redux for this local state and retains its current behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- redux, typescript
- Domain
- frontend
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100