Skip to content
Snippets Groups Projects

Check if peer has connected

Merged Torsten Grote requested to merge 2036-detect-connection into master
1 unresolved thread

This is just first test code. We should check if it works with our devices.

Merge request reports

Merged by Torsten GroteTorsten Grote 4 years ago (Jun 2, 2021 6:41pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
97 99 webServerManager.startWebServer();
98 100 }
99 101
102 @Override
103 public void onDeviceConnected() {
104 Toast.makeText(getApplication(), R.string.connected_toast, LENGTH_LONG)
105 .show();
106 }
  • Sebastian
  • Torsten Grote added 1 commit

    added 1 commit

    Compare with previous version

  • Sebastian added 1 commit

    added 1 commit

    • 7186a898 - Add comment concerning SecurityException/AssertionError

    Compare with previous version

  • Sebastian approved this merge request

    approved this merge request

  • Looks good to me. I just added the comment above that we also have in the real briar app. From my point of view this can be merged now. @grote I don't assume you have any objections but still it seems more appropriate to me if you hit the merge button now.

  • Torsten Grote mentioned in commit 85b35042

    mentioned in commit 85b35042

  • merged

  • Please register or sign in to reply
    Loading