angular / angular/components

MatDialog `open` method `R` generic is too loose

Offen
#12,898 6 Kommentare 3 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
area: material/dialog P4 refactoring
Vorherrschende Sprache
TypeScript
Sterne
25k
Forks
6.8k
Ø Merge
1 T. 8 Std.
Gemergte PRs (30 T.)
91

Beschreibung

Because the `R` generic on `MatDialog.open` isn't constrained by arguments to the function, it's effectively always `any`. We should remove the method generic and change the return generic to `{}`, which will force people to explicitly state which type it is when consuming it.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne damit, die Deklaration der Methode MatDialog.open zu finden und zu prüfen, wie ihr generisches R definiert und zurückgegeben wird. Aktualisiere die generische Typisierung wie beschrieben und überprüfe anschließend, dass Verbraucher von Dialogergebnissen ihren beabsichtigten Typ explizit angeben müssen und dass die bestehenden TypeScript-Prüfungen weiterhin erfolgreich sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
angular, typescript
Bereich
frontend
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.