Check if peer has connected (!10) · Merge requests · briar / hotspot · GitLab

Snippets Groups Projects

Merged Torsten Grote requested to merge 2036-detect-connection into master 4 years ago

1 unresolved thread

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

Activity

Torsten Grote changed milestone to %Install via Bluetooth or Wi-Fi 4 years ago

changed milestone to %Install via Bluetooth or Wi-Fi
Torsten Grote added Sponsor 6 label 4 years ago

added Sponsor 6 label
Torsten Grote @grote started a thread on the diff 4 years ago

app/src/main/java/org/briarproject/hotspot/MainViewModel.java

 		webServerManager.startWebServer();
+	}
 	@Override
 	public void onDeviceConnected() {
 		Toast.makeText(getApplication(), R.string.connected_toast, LENGTH_LONG)
 				.show();
+	}

Sebastian @sebkur started a thread on the diff 4 years ago

Resolved 4 years ago by Torsten Grote

Torsten Grote added 1 commit 4 years ago

added 1 commit

54fbc0e8 - Check if peer has connected

Compare with previous version

Sebastian added 1 commit 4 years ago

added 1 commit

7186a898 - Add comment concerning SecurityException/AssertionError

Compare with previous version

Sebastian approved this merge request 4 years ago

approved this merge request

Sebastian @sebkur · 4 years ago

Developer

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 4 years ago

mentioned in commit 85b35042

Torsten Grote merged 4 years ago

merged

Please register or sign in to reply