Page MenuHomePhabricator
Differential D14297

[native] Replace some alerts with logs
ClosedPublic
Actions

Authored by tomek on Feb 5 2025, 10:46 AM.
Tags
None
Referenced Files
F4910612: D14297.diff
Fri, Mar 14, 6:22 PM
Unknown Object (File)
Fri, Mar 14, 7:59 AM
Unknown Object (File)
Fri, Mar 14, 7:59 AM
Unknown Object (File)
Fri, Mar 14, 7:59 AM
Unknown Object (File)
Fri, Mar 14, 7:34 AM
Unknown Object (File)
Fri, Mar 14, 7:30 AM
Unknown Object (File)
Fri, Mar 14, 4:44 AM
Unknown Object (File)
Fri, Mar 14, 4:34 AM
View All 59 Files
Subscribers
ashoat

Details

Reviewers
kamil
angelika
Commits
rCOMMac174fe9c31c: [native] Replace some alerts with logs
Summary

It is important that we don't replace alerts that are critical. Also, on the web we aren't handling the logs yet, so we should use the old mechanism.

https://linear.app/comm/issue/ENG-10140/replace-staff-alerts-with-a-log

Depends on D14296

Test Plan

Called addLog on the web and mobile and checked if the behavior is correct.

Diff Detail

Repository
rCOMM Comm
Branch
logs
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

tomek created this revision.Feb 5 2025, 10:46 AM
Herald added a subscriber: ashoat. · View Herald TranscriptFeb 5 2025, 10:46 AM
Harbormaster returned this revision to the author for changes because remote builds failed.Feb 5 2025, 11:07 AM
Harbormaster failed remote builds in B33398: Diff 46925!
tomek updated this revision to Diff 46935.Feb 6 2025, 1:54 AM

Rebase

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 6 2025, 2:08 AM
Harbormaster failed remote builds in B33408: Diff 46935!
tomek updated this revision to Diff 46941.Feb 6 2025, 4:30 AM

Rebase

Harbormaster completed remote builds in B33412: Diff 46941.Feb 6 2025, 4:56 AM
tomek requested review of this revision.Feb 6 2025, 4:56 AM
kamil accepted this revision.Feb 10 2025, 3:24 AM
This revision is now accepted and ready to land.Feb 10 2025, 3:24 AM
tomek updated this revision to Diff 47008.Feb 11 2025, 8:29 AM

Rebase

Harbormaster completed remote builds in B33475: Diff 47008.Feb 11 2025, 8:47 AM
Closed by commit rCOMMac174fe9c31c: [native] Replace some alerts with logs (authored by tomek). · Explain WhyFeb 12 2025, 3:18 AM
This revision was automatically updated to reflect the committed changes.
tomek added a commit: rCOMMac174fe9c31c: [native] Replace some alerts with logs.

Revision Contents
Changeset List

PathSize
lib/
components/
20 lines
shared/
dm-ops/
16 lines
tunnelbroker/
16 lines
web/
79 lines

Diff 46925

View Options

lib/components/debug-logs-context-provider.react.js

Loading...
View Options

lib/shared/dm-ops/process-dm-ops.js

Loading...
View Options

lib/tunnelbroker/use-peer-to-peer-message-handler.js

Loading...
View Options

web/app.react.js

Loading...