Skip to content

Fixes #38849: update forms/FormField to use pf5#10733

Draft
kfamilonidis wants to merge 1 commit into
theforeman:developfrom
kfamilonidis:pf5-form-field
Draft

Fixes #38849: update forms/FormField to use pf5#10733
kfamilonidis wants to merge 1 commit into
theforeman:developfrom
kfamilonidis:pf5-form-field

Conversation

@kfamilonidis

Copy link
Copy Markdown
Contributor

wip

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it seems that formField is not used outside the definition of the component and could be removed altogether.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pls. double check if that's the case.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I took a different approach and removed the enzyme test from the codebase #11083

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants