-
-
Notifications
You must be signed in to change notification settings - Fork 392
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
OBPIH-4770 Create new product list #3497
Conversation
const [filterParams, setFilterParams] = useState({}); | ||
|
||
useEffect(() => { | ||
props.fetchTranslations(props.locale, 'purchaseOrder'); |
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.
wrong translation fetched
change to
props.fetchTranslations(props.locale, 'productsList');
label: 'react.productsList.includeSubcategories.label', | ||
defaultMessage: 'Include all products in all subcategories', | ||
attributes: { | ||
filterElement: true, |
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.
what does the attribute filterElement: true
do for CheckboxField
?
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.
it is added to filter form elements not to have to play around with position styles included in form-utils
, so it is just to position filter elements, nothing functional.
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.
that's right I remember now. The naming implies that it has something to do with filtering, kind of misleading
a9bd9dc
to
d003b6e
Compare
I also found in the mean time some minor issues on PO list and changed some state names on invoiceList which were copied from PO list and not changed by mistake.