Skip to content

Remove redundant string type #9332

@thatblindgeye

Description

@thatblindgeye

Describe the enhancement or change
Several spots we're typing a prop as both "string" and "React.ReactNode", we should remove "string" since ReactNode will include the string type.

  • NotificationDrawerGroup
  • OptionsMenuItemGroup
  • Select (deprecated)
  • TableTypes
  • Table MD file

All of the above are using string | React.ReactNode. Follow up from comment #9278 (comment)

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?

Metadata

Metadata

Labels

No labels
No labels

Type

No type

Projects

Status

Done

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions