[$500] Disable logs from staging and production native apps

[mountiny]

cc @perunt @kidroca

Coming from here, we would like to disable the logs in native devices when the ENV is staging or production as we can see they can have certain effect on the performance.

We will keep the logs for web as there the performance effect is not so significant

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~0196038043ae5d7c60
• Upwork Job ID: 1719690288548622336
• Last Price Increase: 2023-11-01

[melvin-bot]

Triggered auto assignment to @trjExpensify (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details.

[trjExpensify]

@mountiny mind confirming what you want to do with this issue? Who should it get assigned to?

[mountiny]

This will be worked on externally so we might need to pay out later, waiting for @kidroca to confirm if he wants to work on this or if we should make this external

[trjExpensify]

Cool, sounds good!

[kidroca]

Thanks! I'd like to take this one

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~0196038043ae5d7c60

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @robertKozik (External)

[mountiny]

@kidroca could you please post a proposal before starting a PR, thanks!

[kidroca]

Proposal

Please re-state the problem that we are trying to solve in this issue.

The app's UI experiences sluggish performance at times. Preliminary investigation points to console logging as a significant contributor to this issue.

What is the root cause of that problem?

Console logging is a known performance bottleneck in React Native apps. It was initially disabled in production but was later re-enabled for debugging purposes. The problem lies in the excessive amount of logging, which impacts performance. For example, simple actions like switching chats generate around 400 lines of logs within a few seconds.

What changes do you think we should make in order to solve the problem?

1. Disable Console Logging: We should disable console logging in the production build to enhance app performance. This is a standard practice in React Native apps.

• we'd still keep logging for web/desktop as logging shouldn't be affecting performance noticeable there, and these are the platforms where users can actually provide the logs easily. Related slack comment.

3. Collect User Feedback: Post-implementation, we should collect user feedback to evaluate the impact on UI responsiveness and overall app performance.

What alternative solutions did you explore? (Optional)

We could implement a feature to allow users to manually send the current log buffer to technical support for debugging. However, this would require additional development time and does not immediately address the performance issue.

---

Expected Outcomes: Improved app performance and a better user experience.

References: The official React Native documentation advises minimizing the use of console logging in production builds for better performance.

[mountiny]

@kidroca since you are already assigned, just go ahead and create the PR, thank you!

[melvin-bot]

This issue has not been updated in over 15 days. @kidroca, @trjExpensify, @mountiny, @robertKozik eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

[mountiny]

@kidroca will handle payments elsewhere, I think we can close this issue now.

Thank you!

[JmillsExpensify]

Removed the invoice for @kidroca's work and I approve payment.