-
Notifications
You must be signed in to change notification settings - Fork 556
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add AccountSelector
to Box
and Field
children
#2774
Conversation
This comment was marked as spam.
This comment was marked as spam.
@GuillaumeRx Needs some updates to tests it seems |
Yes, I'm also adding more things to this PR. Hold on :) |
AccountSelector
to Box
childrenAccountSelector
to Box
and Field
children
@@ -341,7 +341,6 @@ export const AccountSelectorStruct: Describe<AccountSelectorElement> = element( | |||
'AccountSelector', | |||
{ | |||
name: string(), | |||
title: string(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does the account selector have a static title now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dropped this in favour of the usage of the Field
's label
prop :)
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2774 +/- ##
=======================================
Coverage 94.41% 94.41%
=======================================
Files 482 482
Lines 10261 10261
Branches 1563 1563
=======================================
Hits 9688 9688
Misses 573 573 ☔ View full report in Codecov by Sentry. |
This reverts commit 6cc4791.
This reverts commit 6cc4791.
This reverts commit 6cc4791.
This PR fixes the forgotten addition of the
AccountSelector
toBox
childrens list.It also allows the usage of
AccountSelector
in theField
component.The
title
prop of theAccountSelector
was also dropped in favor of wrapping this component inside of aField
.