patternfly / patternfly/patternfly-react
Alert - remove hasAnimations prop during breaking change
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 862
- Forks
- 392
- Ø Merge
- 4 T. 8 Std.
- Gemergte PRs (30 T.)
- 9
Beschreibung
Describe the enhancement or change
Followup to animations PR https://github.com/patternfly/patternfly-react/pull/11495 where hasAnimations was added as beta, with the intent being it'd be used for testing purposes only. In the next breaking change we would expect consumers to update tests to reflect animations within an AlertGroup and should remove the prop.
Is this request originating from a Red Hat product team? If so, which ones and is there any sort of deadline for this enhancement?
Any other information?
Jira Issue: PF-2207
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, hasAnimations in der AlertGroup-Komponente und deren Tests nachzuverfolgen, wobei der Animations-PR #11495 als Kontext dient. Entferne das beta-Prop im Rahmen der Breaking Change und aktualisiere die zugehörigen Tests, sodass deren erwartetes Animationsverhalten von AlertGroup der neuen API entspricht.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- react, typescript
- Bereich
- frontend
- Issue-Typ
- Refactoring
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100