Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
4.03k stars 3.03k forks source link

perf: Calculate shouldReportBeInoptionList once with a bitwise flag #55742

Open hannojg opened 3 weeks ago

hannojg commented 3 weeks ago

Explanation of Change

WIP

Performance improvement to calculate a bitwise flag for the different possible states of a report. This way we avoid calculating these fields over and over for a report. For calculating the reports for the sidebar and search this function is called most often, so I think improving it will yield a great performance gain throughout different fea

Fixed Issues

$ 46590 PROPOSAL:

Tests

Offline tests

QA Steps

// TODO: These must be filled out, or the issue title must include "[No QA]."

PR Author Checklist

Screenshots/Videos

Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari
MacOS: Desktop