sillsdev / sillsdev/TheCombine
[MergeDups] Connect(Droppable) console warning on dev
Open
Nobody has claimed this yet.
frontend
goal: MergeDup
maintenance
- Dominant language
- TypeScript
- Stars
- 22
- Forks
- 10
- Avg merge
- 2d 2h
- Merged PRs (30d)
- 9
Description
Run npm start, open a project, and go to Merge Duplicates. When the first set loads, the console throws this Warning:
Warning: Connect(Droppable): Support for defaultProps will be removed from memo components in a future major release. Use JavaScript default parameters instead. Error Component Stack
at div (<anonymous>)
at emotion-element-43c6fea0.browser.esm.js:37:1
at Grid (Grid.js:367:1)
at div (<anonymous>)
at emotion-element-43c6fea0.browser.esm.js:37:1
at Grid (Grid.js:367:1)
at Provider (Provider.js:8:1)
at App (react-beautiful-dnd.esm.js:6834:1)
at ErrorBoundary (react-beautiful-dnd.esm.js:97:1)
at DragDropContext (react-beautiful-dnd.esm.js:6963:1)
at MergeDragDrop (index.tsx:33:1)
at div (<anonymous>)
at MergeDupsStep (index.tsx:12:1)
at MergeDups (index.tsx:11:1)
at InnerLoadable (loadable.esm.mjs:176:1)
at LoadableWithChunkExtractor (<anonymous>)
at Loadable (<anonymous>)
at BaseGoalScreen (BaseGoalScreen.tsx:58:1)
at LoadingGoalScreen (BaseGoalScreen.tsx:39:1)
at InnerLoadable (loadable.esm.mjs:176:1)
at LoadableWithChunkExtractor (<anonymous>)
at Loadable (<anonymous>)
at RenderedRoute (hooks.tsx:657:1)
at Routes (components.tsx:504:1)
at RtlProvider (index.js:10:1)
at ThemeProvider (ThemeProvider.js:31:1)
at ThemeProvider (ThemeProvider.js:43:1)
at ThemeProvider (ThemeProvider.js:13:1)
at LocalizationProvider (LocalizationProvider.js:27:1)
at DatePickersLocalizationProvider (DatePickersLocalizationProvider.tsx:21:1)
at AppWithBar (AppLoggedIn.tsx:37:1)
at InnerLoadable (loadable.esm.mjs:176:1)
at LoadableWithChunkExtractor (<anonymous>)
at Loadable (<anonymous>)
at RequireAuth (index.tsx:13:1)
at RenderedRoute (hooks.tsx:657:1)
at RenderErrorBoundary (hooks.tsx:583:1)
at DataRoutes (index.tsx:752:1)
at Router (components.tsx:413:1)
at RouterProvider (index.tsx:487:1)
at Suspense (<anonymous>)
at div (<anonymous>)
at App (<anonymous>)
at PersistGate (react.js:33:1)
at Provider (Provider.tsx:60:1)
at SnackbarProvider (SnackbarProvider.tsx:43:1)
at RtlProvider (index.js:10:1)
at ThemeProvider (ThemeProvider.js:31:1)
at ThemeProvider (ThemeProvider.js:43:1)
at ThemeProvider (ThemeProvider.js:13:1)
at StyledEngineProvider (StyledEngineProvider.js:20:1)
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
Run npm start, open a project, and navigate to Merge Duplicates to reproduce the warning. Inspect MergeDragDrop at index.tsx:33 and the surrounding MergeDupsStep entry point, then confirm the Connect(Droppable) warning no longer appears when the first set loads.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react, typescript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 52/100