Repository navigation
Make less calls to log api command - #97334
Conversation
Reviewer Checklist
Screenshots/VideosAndroid: HybridAppAndroid: mWeb ChromeiOS: HybridAppiOS: mWeb SafariMacOS: Chrome / Safari |
|
🚧 rlinoz has triggered a test Expensify/App build. You can view the workflow run here. |
|
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
|
🧪🧪 Use the links below to test this adhoc build on Android, iOS, and Web. Happy testing! 🧪🧪
|
|
🚀 Deployed to staging by https://github.com/rlinoz in version: 9.4.46-0 🚀
|
|
🤖 No help site changes required. I reviewed the changes in this PR against Expensify's help site content under Both changes are internal client-side logging/telemetry tuning with no user-facing impact:
Neither change alters any feature, tab, setting, button, workflow, billing, or integration behavior that the help site documents. The help site describes how customers use Expensify, and these changes are invisible to that experience — they only affect how frequently the app batches diagnostic logs to the server. Because nothing customer-facing changed, I did not create a draft help site PR. |
|
🚀 Deployed to production by https://github.com/marcaaron in version: 9.4.46-10 🚀
Bundle Size Analysis (Sentry): |
Explanation of Change
Context https://expensify.slack.com/archives/C08CZDJFJ77/p1785328475917909?thread_ts=1785235703.173929&cid=C08CZDJFJ77
Tests
Checked logs still arrive properly to the server
Offline tests
No
QA Steps
// TODO: These must be filled out, or the issue title must include "[No QA]."
Checked logs still arrive properly to the server by searching for
[App] Processing 'Log' for 'expensify.com'PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectionAvatar, I verified the components usingAvatarare working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))npm run compress-svg)Avataris modified, I verified thatAvataris working as expected in all cases)Designlabel and/or tagged@Expensify/designso the design team can review the changes.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.Screenshots/Videos
Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari