Page MenuHomePhabricator
Differential D13431

[native] Avoid Promise.allSettled
ClosedPublic
Actions

Authored by ashoat on Sep 23 2024, 10:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 18, 3:26 AM
Unknown Object (File)
Sun, Nov 17, 3:08 PM
Unknown Object (File)
Fri, Nov 8, 7:02 PM
Unknown Object (File)
Fri, Nov 8, 7:02 PM
Unknown Object (File)
Fri, Nov 8, 7:01 PM
Unknown Object (File)
Fri, Nov 8, 2:08 AM
Unknown Object (File)
Mon, Oct 21, 8:18 AM
Unknown Object (File)
Oct 18 2024, 9:08 PM
View All 29 Files
Subscribers
tomek

Details

Reviewers
marcin
Commits
rCOMMe11444b5ba9d: [native] Avoid Promise.allSettled
Summary

This addresses ENG-9233, which was caused by Promise.allSettled being undefined on Android.

Depends on D13430

Test Plan

Confirmed that notifs sent from Android for a new session worked

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ashoat created this revision.Sep 23 2024, 10:16 PM
Herald added a subscriber: tomek. · View Herald TranscriptSep 23 2024, 10:16 PM
ashoat added a child revision: D13432: [native] Add log for when all_thread_members is used for a thread missing from ThreadStore.Sep 23 2024, 10:18 PM
Harbormaster completed remote builds in B31788: Diff 44454.Sep 23 2024, 10:32 PM
ashoat requested review of this revision.Sep 23 2024, 10:32 PM
tomek added inline comments.Sep 24 2024, 1:43 AM
lib/push/send-utils.js
996–1014

Can we add it to our utils /Users/palys/code/comm/lib/utils/promises.js?

marcin accepted this revision.Sep 24 2024, 4:08 AM
This revision is now accepted and ready to land.Sep 24 2024, 4:08 AM
ashoat added inline comments.Sep 24 2024, 10:07 AM
lib/push/send-utils.js
996–1014

We could, but then we would have to maintain the types. I looked into this, and I'm a bit concerned because the types we'd need for our current version of Flow don't match the types in their main branch. This implies that we'd have to update it at some point... I'm not sure it's worth the effort

Closed by commit rCOMMe11444b5ba9d: [native] Avoid Promise.allSettled (authored by ashoat). · Explain WhySep 24 2024, 3:04 PM
This revision was automatically updated to reflect the committed changes.
ashoat added a commit: rCOMMe11444b5ba9d: [native] Avoid Promise.allSettled.

Revision Contents
Changeset List

PathSize
lib/
push/
20 lines

Diff 44454

View Options

lib/push/send-utils.js

Loading...