galaxyproject / galaxyproject/brc-analytics
fix: console errors in SampleSheetStep component
- Dominant language
- TypeScript
- Stars
- 7
- Forks
- 11
- Avg merge
- 2d 12h
- Merged PRs (30d)
- 16
Description
## Description
The `SampleSheetStep` component produces two React console errors:
1. **`isDragActive` prop on a DOM element** — React does not recognize the `isDragActive` prop on a DOM element. It should either be removed from the DOM element or spelled as lowercase `isdragactive` if intended as a custom attribute.
2. **`error` attribute receives `false`** — A non-boolean attribute `error` receives `false`. It should either be passed as a string (`error="false"`) or conditionally omitted (`error={condition ? value : undefined}`).
## Component path
`app/components/Entity/components/ConfigureWorkflowInputs/components/Main/components/Stepper/components/Step/SampleSheetStep/sampleSheetStep.tsx`
Contributor guide
No contributing guide indexed for this repository
Research direction
Open app/components/Entity/components/ConfigureWorkflowInputs/components/Main/components/Stepper/components/Step/SampleSheetStep/sampleSheetStep.tsx and inspect how isDragActive and error are passed to DOM elements. Reproduce the SampleSheetStep console output, then update the prop handling so both React warnings are gone and verify the component no longer emits these errors.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react, typescript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 65/100