Skip to content
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-4772 Invoice list (fixes after QA) #3491

Merged
merged 4 commits into from
Sep 26, 2022
Merged

Conversation

kchelstowski
Copy link
Collaborator

No description provided.

},
];


const onFetchHandler = (state) => {
const offset = state.page > 0 ? (state.page) * state.pageSize : 0;
const sortKeyParser = (key) => {
// Property is named invoiceTypeCode, but in marshaller was renamed to invoiceType
Copy link
Collaborator

Choose a reason for hiding this comment

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

Then rename it in the marshaller to the invoiceTypeCode

@kchelstowski
Copy link
Collaborator Author

The problem with Search/Clear button if we wanted to refresh with the same filter values multiple times was that the state (filterParams) was not changed as after multiple submits we were providing the same values.
Fix for that is to kind of "cheat" React and pass the same values, but in a object, that is different in terms of reference.

@awalkowiak awalkowiak merged commit cfec916 into feature/ui-redesign Sep 26, 2022
@awalkowiak awalkowiak deleted the OBPIH-4772 branch September 26, 2022 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants