downshift-js/downshift

React.HTMLProps is not a thing in Flow

Open

#657 geöffnet am 29. Jan. 2019

Auf GitHub ansehen
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (11.761 Stars) (980 Forks)batch import
help wanted

Beschreibung

Problem description:

Your Flow type definitions make use of React.HTMLProps, but this type doesn't exist in Flow.

https://github.com/paypal/downshift/blob/master/flow-typed/npm/downshift_v2.x.x.js.flow#L143

Suggested solution:

Maybe $Shape<HTMLSomething> will produce something similar.

Contributor Guide

Tech Stack
javascriptreact
Domain
frontend
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Flow typingHTML element attributes
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85
Research-Richtung
The issue is that React.HTMLProps is not a valid Flow type. The suggested fix is to use $Shape<HTMLSomething> instead. The contributor should examine the file flow typed/npm/downshift v2.x.x.js.flow, specifically around line 143, and replace React.HTMLProps with an appropriate Flow type like $Shape<HTMLInputElement> or $Shape<HTMLSelectElement> depending on usage. They should verify the change by running Flow on the project and ensure no new type errors arise. It may also be helpful to review existing Flow type definitions in the repository for similar patterns.