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

fix: Search Page Order #6971

Merged
merged 8 commits into from
Jan 14, 2022
Prev Previous commit
Next Next commit
Merge branch 'main' of github.com:Expensify/Expensify.cash into sortO…
…rder
  • Loading branch information
parasharrajat committed Jan 10, 2022

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
commit a84793c20913f5e7f39c2bb552555e0676aaf697
2 changes: 1 addition & 1 deletion src/libs/OptionsListUtils.js
Original file line number Diff line number Diff line change
@@ -560,7 +560,7 @@ function getOptions(reports, personalDetails, activeReportID, {
recentReportOptions.push(...personalDetailsOptions);
personalDetailsOptions = [];
recentReportOptions = lodashOrderBy(recentReportOptions, [(option) => {
if (option.isDefaultChatRoom || option.isArchivedRoom) {
if (option.isChatRoom || option.isArchivedRoom) {
return 3;
}
if (!option.login) {
You are viewing a condensed version of this merge commit. You can view the full changes here.