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.1.27-2
Reproducible in staging?: yes
Reproducible in production?: yes
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @shubham1206agra
Slack conversation (hyperlinked to channel name): https://expensify.slack.com/archives/C08CZDJFJ77/p1743166420942949
In order to speed up lookup of search terms for the FastSearch implementation, we want to sort the search terms to search results for the longest word first. After that, search can further filter the results based on the other words.
Before we were using .sort() to find the longest word, which is incorrect, as its sorts strings by their numerical representation, and not by length.
cc @hannojg @Nodonisko @shubham1206agra
Platforms:
Select the officially supported platforms where the issue was reproduced:
Platforms Tested:
On which of our officially supported platforms was this issue tested:
Screenshots/Videos
Add any screenshot/video evidence
View all open jobs on GitHub
Issue Owner
Current Issue Owner: @greg-schroeder
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.1.27-2
Reproducible in staging?: yes
Reproducible in production?: yes
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @shubham1206agra
Slack conversation (hyperlinked to channel name): https://expensify.slack.com/archives/C08CZDJFJ77/p1743166420942949
In order to speed up lookup of search terms for the FastSearch implementation, we want to sort the search terms to search results for the longest word first. After that, search can further filter the results based on the other words.
Before we were using
.sort()to find the longest word, which is incorrect, as its sorts strings by their numerical representation, and not by length.cc @hannojg @Nodonisko @shubham1206agra
Platforms:
Select the officially supported platforms where the issue was reproduced:
Platforms Tested:
On which of our officially supported platforms was this issue tested:Screenshots/Videos
Add any screenshot/video evidence
View all open jobs on GitHub
Issue Owner
Current Issue Owner: @greg-schroeder