Page MenuHomePhabricator
Differential D3134

[native] Send PONG when PING is received
ClosedPublic
Actions

Authored by tomek on Feb 8 2022, 6:58 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 12, 1:14 AM
Unknown Object (File)
Thu, Nov 7, 7:25 AM
Unknown Object (File)
Thu, Nov 7, 4:44 AM
Unknown Object (File)
Sat, Nov 2, 4:41 AM
Unknown Object (File)
Sat, Nov 2, 4:41 AM
Unknown Object (File)
Sat, Nov 2, 4:41 AM
Unknown Object (File)
Sat, Nov 2, 4:37 AM
Unknown Object (File)
Fri, Oct 25, 12:00 AM
View All Files
Subscribers
• adrian
ashoat
atul
• benschac

Details

Reviewers
• karol
ashoat
atul
• jim
Commits
rCOMM1e26f7e47044: [native] Send PONG when PING is received
Summary

Check background notification type and react accordingly

Depends on D3133

Test Plan

Send a background notification on Android and check if the code on C++ was executed

{
  data: {
    backgroundNotifType: 'PING',
  },
}

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tomek created this revision.Feb 8 2022, 6:58 AM
Herald added subscribers: atul, • adrian, ashoat. · View Herald TranscriptFeb 8 2022, 6:58 AM
tomek added inline comments.Feb 8 2022, 6:59 AM
native/android/app/src/main/java/app/comm/android/CommNotificationsHandler.java
88–89 ↗(On Diff #9396)

We're not sending this notif yet. This handling is similar to what can be found in handleBackgroundNotification in AppDelegate.mm.

Harbormaster completed remote builds in B6631: Diff 9396.Feb 8 2022, 7:09 AM
tomek requested review of this revision.Feb 8 2022, 7:09 AM
tomek planned changes to this revision.Feb 8 2022, 9:29 AM
tomek requested review of this revision.Feb 9 2022, 4:58 AM
• karol requested changes to this revision.Feb 9 2022, 5:07 AM
• karol added inline comments.
native/android/app/src/main/java/app/comm/android/CommNotificationsHandler.java
89 ↗(On Diff #9396)

won't this break the build? In D3132 NetworkModule is an empty class.

This revision now requires changes to proceed.Feb 9 2022, 5:07 AM
• karol accepted this revision.Feb 9 2022, 5:08 AM
• karol added inline comments.
native/android/app/src/main/java/app/comm/android/CommNotificationsHandler.java
89 ↗(On Diff #9396)

Sorry I got the order of revisions wrong, all good here.

This revision is now accepted and ready to land.Feb 9 2022, 5:08 AM
Herald added a reviewer: ashoat. · View Herald TranscriptFeb 9 2022, 5:08 AM
Herald added a subscriber: • benschac. · View Herald Transcript
This revision now requires review to proceed.Feb 9 2022, 5:08 AM
ashoat accepted this revision.Feb 9 2022, 9:39 PM
ashoat added reviewers: atul, • jim.
This revision is now accepted and ready to land.Feb 9 2022, 9:39 PM
ashoat added inline comments.Feb 9 2022, 9:40 PM
native/android/app/src/main/java/app/comm/android/CommNotificationsHandler.java
89 ↗(On Diff #9396)

It would've been good to establish a "Depends On" relationship with D3132 and D3133 here

tomek edited the summary of this revision. (Show Details)Feb 10 2022, 4:07 AM
tomek added a parent revision: D3133: [native] Add sendPong method.
tomek added inline comments.Feb 10 2022, 4:35 AM
native/android/app/src/main/java/app/comm/android/CommNotificationsHandler.java
89 ↗(On Diff #9396)

Makes sense. I added it here and will add in the future

Closed by commit rCOMM1e26f7e47044: [native] Send PONG when PING is received (authored by tomek). · Explain WhyFeb 10 2022, 5:06 AM
This revision was automatically updated to reflect the committed changes.
tomek added a commit: rCOMM1e26f7e47044: [native] Send PONG when PING is received.

Revision Contents
Changeset List

PathSize
native/
android/
app/
src/
main/
java/
app/
comm/
android/
9 lines

Diff 9517

View Options

native/android/app/src/main/java/app/comm/android/CommNotificationsHandler.java

Loading...