over_react
over_react copied to clipboard
FED-2299 Utilize new ReactNode typedef
Motivation
https://github.com/Workiva/react-dart/pull/384 adds a new typedef that is an alias for Object?
. We should use it!
Changes
Use it where appropriate
Release Notes
Review
See CONTRIBUTING.md for more details on review types (+1 / QA +1 / +10) and code review process.
Please review:
QA Checklist
- [ ] Tests were updated and provide good coverage of the changeset and other affected code
- [ ] Manual testing was performed if needed
- [ ] Steps from PR author:
- [ ] Anything falling under manual testing criteria outlined in CONTRIBUTING.md
Merge Checklist
While we perform many automated checks before auto-merging, some manual checks are needed:
- [ ] A Frontend Frameworks Design member has reviewed these changes
- [ ] There are no unaddressed comments - this check can be automated if reviewers use the "Request Changes" feature
- [ ] For release PRs - Version metadata in Rosie comment is correct
Security Insights
No security relevant content was detected by automated scans.
Action Items
- Review PR for security impact; comment "security review required" if needed or unsure
- Verify
aviary.yaml
coverage of security relevant code
Questions or Comments? Reach out on Slack: #support-infosec.
Rosie is not picking up the build even after closing/re-opening.
Closing in favor of a new PR