-
Notifications
You must be signed in to change notification settings - Fork 182
Open
Labels
good first issueA good task for a newcomer to start withA good task for a newcomer to start withhelp wantedReady to be picked up by anyone in the communityReady to be picked up by anyone in the community
Description
This repo has two very similar implementations of DraggableList:
- https://github.com/openedx/frontend-app-authoring/blob/fdf98a140/src/generic/DraggableList/DraggableList.jsx
- https://github.com/openedx/frontend-app-authoring/blob/fdf98a140/src/course-outline/drag-helper/DraggableList.tsx
The historic reason is that component 1 was part of the editors code, which used to be a separate repo. Now that the editors live in the same repo, there are quite a few redundancies like this.
If possible, they should be merged into a single DraggableList component that works for all the use cases currently covered by both. Make sure the final result uses TypeScript and has no console warnings.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issueA good task for a newcomer to start withA good task for a newcomer to start withhelp wantedReady to be picked up by anyone in the communityReady to be picked up by anyone in the community