galaxyproject / galaxyproject/brc-analytics

fix: console errors in SampleSheetStep component

Open Beginner friendly
#1,198 0 comments 0 reactions 0 assignees View on GitHub
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`

Image

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.