openedx / openedx/frontend-app-learner-record
Remove `redux` as it is not being used anywhere in the repo
Open
Beginner friendly
Nobody has claimed this yet.
dependencies
good first issue
help wanted
- Dominant language
- JavaScript
- Stars
- 7
- Forks
- 26
- Avg merge
- 3m
- Merged PRs (30d)
- 1
Description
Redux is not being used anywhere in this repo but it is installed. We also have two renovate PRs to update redux that are currently not passing CI. Since we don’t use redux, we should remove it and then close those PRs.
Contributor guide
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
The issue names Redux but no repository file; start by locating the dependency declaration and inspect the two Renovate PRs and their CI failures. Remove the unused Redux dependency, verify the application and CI still pass, and close the Renovate PRs as requested.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, redux
- Domain
- frontend, tooling
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Clearly specified
- Newbie friendliness
- 68/100