Open IuliiaHerets opened 1 week ago
Triggered auto assignment to @puneetlath (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.
We think that this bug might be related to #wave-control
@puneetlath FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors
Search filters - Selected categories are not arranged alphabetically in Category field
On saving selected items, we are not sorting selected items before updating filter form with the selected items https://github.com/Expensify/App/blob/2d1f8cd1cb072f0b9c9672effa65199727553ab9/src/pages/Search/SearchAdvancedFiltersPage/SearchFiltersCategoryPage.tsx#L34
We should sort the selected items first before updating filter form with the selected items
const onSaveSelection = useCallback((values: string[]) => SearchActions.updateAdvancedFilters({category: values.sort(localeCompare)}), [])
We can sort the selected items in SearchMultipleSelectionPicker
, if we want to sort all fields on the filter page
onSaveSelection(
selectedItems
.sort((a, b) => localeCompare(a.name, b.name))
.map((item) => item.value)
.flat(),
Search filters - Selected categories are not arranged alphabetically in Category field
We don't sort the filterArray
in the getFilterDisplayTitle
function.
So, the text created from the categories will not be ordered
We should sort the categories in filterArray
so the text will be sorted.
return filterArray.sort(localeCompare).join(', ');
This change will also sort the currency and tag fields.
Job added to Upwork: https://www.upwork.com/jobs/~016ce9ccc4019cea35
Triggered auto assignment to Contributor-plus team member for initial proposal review - @shubham1206agra (External
)
It's not really a bug, more of an improvement request. But we can go ahead and do it for $125.
Upwork job price has been updated to $125
@puneetlath Do you also want to sort the currencies similarly?
Yes, I'd think we'd do it for any of the multi-select fields.
@Tony-MK Please update your proposal
@shubham1206agra can you please comment on my proposal https://github.com/Expensify/App/issues/48496#issuecomment-2327166782. My alternative solution covers sorting currency, and other fields. We can also repeat my main solution on currency pages and others.
Let's go with @Tony-MK's proposal.
πππ C+ reviewed
Current assignee @puneetlath is eligible for the choreEngineerContributorManagement assigner, not assigning anyone new.
π£ @shubham1206agra π An offer has been automatically sent to your Upwork account for the Reviewer role π Thanks for contributing to the Expensify app!
π£ @Tony-MK π An offer has been automatically sent to your Upwork account for the Contributor role π Thanks for contributing to the Expensify app!
Offer link Upwork job Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review π§βπ» Keep in mind: Code of Conduct | Contributing π
If you havenβt already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 9.0.28-0 Reproducible in staging?: Y Reproducible in production?: Y Email or phone of affected tester (no customers): applausetester+kh010901@applause.expensifail.com Issue reported by: Applause Internal Team
Action Performed:
Expected Result:
The selected categories will be arranged alphabetically in Category field.
Actual Result:
The selected categories are not arranged alphabetically in Category field.
Workaround:
Unknown
Platforms:
Screenshots/Videos
https://github.com/user-attachments/assets/6324e158-ac70-4503-88f0-7212e713b8f5
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @shubham1206agra