Skip to content

Commit a38bf9d

Browse files
authored
Add room sid to various room-related events (#162)
* Add room sid to various room-related events Fixes #137 * Fix circle.yml * Start migrating circle.yml * One step closer
1 parent 645f87e commit a38bf9d

File tree

3 files changed

+62
-33
lines changed

3 files changed

+62
-33
lines changed

android/src/main/java/com/twiliorn/library/CustomTwilioVideoView.java

Lines changed: 15 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -585,7 +585,8 @@ private Room.Listener roomListener() {
585585
public void onConnected(Room room) {
586586
localParticipant = room.getLocalParticipant();
587587
WritableMap event = new WritableNativeMap();
588-
event.putString("room", room.getName());
588+
event.putString("roomName", room.getName());
589+
event.putString("roomSid", room.getSid());
589590
List<RemoteParticipant> participants = room.getRemoteParticipants();
590591

591592
WritableArray participantsArray = new WritableNativeArray();
@@ -597,20 +598,24 @@ public void onConnected(Room room) {
597598
pushEvent(CustomTwilioVideoView.this, ON_CONNECTED, event);
598599

599600
for (RemoteParticipant participant : participants) {
600-
addParticipant(participant);
601+
addParticipant(room, participant);
601602
}
602603
}
603604

604605
@Override
605606
public void onConnectFailure(Room room, TwilioException e) {
606607
WritableMap event = new WritableNativeMap();
608+
event.putString("roomName", room.getName());
609+
event.putString("roomSid", room.getSid());
607610
event.putString("reason", e.getExplanation());
608611
pushEvent(CustomTwilioVideoView.this, ON_CONNECT_FAILURE, event);
609612
}
610613

611614
@Override
612615
public void onDisconnected(Room room, TwilioException e) {
613616
WritableMap event = new WritableNativeMap();
617+
event.putString("roomName", room.getName());
618+
event.putString("roomSid", room.getSid());
614619
event.putString("participant", localParticipant.getIdentity());
615620
pushEvent(CustomTwilioVideoView.this, ON_DISCONNECTED, event);
616621

@@ -628,12 +633,12 @@ public void onDisconnected(Room room, TwilioException e) {
628633

629634
@Override
630635
public void onParticipantConnected(Room room, RemoteParticipant participant) {
631-
addParticipant(participant);
636+
addParticipant(room, participant);
632637
}
633638

634639
@Override
635640
public void onParticipantDisconnected(Room room, RemoteParticipant participant) {
636-
removeParticipant(participant);
641+
removeParticipant(room, participant);
637642
}
638643

639644
@Override
@@ -649,10 +654,12 @@ public void onRecordingStopped(Room room) {
649654
/*
650655
* Called when participant joins the room
651656
*/
652-
private void addParticipant(RemoteParticipant participant) {
657+
private void addParticipant(Room room, RemoteParticipant participant) {
653658
Log.i("CustomTwilioVideoView", "ADD PARTICIPANT ");
654659

655660
WritableMap event = new WritableNativeMap();
661+
event.putString("roomName", room.getName());
662+
event.putString("roomSid", room.getSid());
656663
event.putMap("participant", buildParticipant(participant));
657664

658665
pushEvent(this, ON_PARTICIPANT_CONNECTED, event);
@@ -675,8 +682,10 @@ private void addParticipant(RemoteParticipant participant) {
675682
/*
676683
* Called when participant leaves the room
677684
*/
678-
private void removeParticipant(RemoteParticipant participant) {
685+
private void removeParticipant(Room room, RemoteParticipant participant) {
679686
WritableMap event = new WritableNativeMap();
687+
event.putString("roomName", room.getName());
688+
event.putString("roomSid", room.getSid());
680689
event.putMap("participant", buildParticipant(participant));
681690
pushEvent(this, ON_PARTICIPANT_DISCONNECTED, event);
682691
//something about this breaking.

circle.yml

Lines changed: 42 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,43 @@
1-
machine:
2-
node:
3-
version: 7.8.0
4-
environment:
5-
PATH: "${PATH}:${HOME}/${CIRCLE_PROJECT_REPONAME}/node_modules/.bin"
1+
attach_workspace: &attach_workspace
2+
attach_workspace:
3+
at: ~/react-native-twilio-video-webrtc
64

7-
dependencies:
8-
pre:
9-
- rm -rf ~/.yarn
10-
- npm i -g yarn
11-
- yarn -v
12-
override:
13-
- yarn
14-
cache_directories:
15-
- ~/.cache/yarn
16-
17-
test:
18-
override:
19-
- yarn run ci
20-
21-
test:
22-
override:
23-
- yarn run ci
5+
docker_defaults: &docker_defaults
6+
docker:
7+
- image: circleci/python:3.6.4-stretch-node-browsers
8+
working_directory: ~/react-native-twilio-video-webrtc
9+
10+
jobs:
11+
build:
12+
<<: *docker_defaults
13+
steps:
14+
- checkout
15+
- restore_cache:
16+
name: Restore node_modules cache
17+
keys:
18+
- v1-npm-deps-{{ checksum "yarn.lock" }}
19+
- run:
20+
name: Node Dependencies
21+
command: yarn install
22+
- save_cache:
23+
name: Save node_modules cache
24+
key: v1-npm-deps-{{ checksum "yarn.lock" }}
25+
paths:
26+
- node_modules
27+
- ~/.cache/yarn
28+
- persist_to_workspace:
29+
root: ~/react-native-twilio-video-webrtc
30+
paths:
31+
- node_modules
32+
test:
33+
machine:
34+
node:
35+
version: 7.8.0
36+
environment:
37+
PATH: "${PATH}:${HOME}/${CIRCLE_PROJECT_REPONAME}/node_modules/.bin"
38+
steps:
39+
- checkout
40+
- *attach_workspace
41+
- run:
42+
name: Run tests
43+
command: yarn run ci

ios/RCTTWVideoModule.m

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -352,13 +352,13 @@ - (void)didConnectToRoom:(TVIRoom *)room {
352352
TVILocalParticipant *localParticipant = room.localParticipant;
353353
[participants addObject:[localParticipant toJSON]];
354354

355-
[self sendEventWithName:roomDidConnect body:@{ @"roomName" : room.name , @"participants" : participants }];
355+
[self sendEventWithName:roomDidConnect body:@{ @"roomName" : room.name , @"roomSid": room.sid, @"participants" : participants }];
356356
}
357357

358358
- (void)room:(TVIRoom *)room didDisconnectWithError:(nullable NSError *)error {
359359
self.room = nil;
360360

361-
NSMutableDictionary *body = [@{ @"roomName": room.name } mutableCopy];
361+
NSMutableDictionary *body = [@{ @"roomName": room.name, @"roomSid": room.sid } mutableCopy];
362362

363363
if (error) {
364364
[body addEntriesFromDictionary:@{ @"error" : error.localizedDescription }];
@@ -370,7 +370,7 @@ - (void)room:(TVIRoom *)room didDisconnectWithError:(nullable NSError *)error {
370370
- (void)room:(TVIRoom *)room didFailToConnectWithError:(nonnull NSError *)error{
371371
self.room = nil;
372372

373-
NSMutableDictionary *body = [@{ @"roomName": room.name } mutableCopy];
373+
NSMutableDictionary *body = [@{ @"roomName": room.name, @"roomSid": room.sid } mutableCopy];
374374

375375
if (error) {
376376
[body addEntriesFromDictionary:@{ @"error" : error.localizedDescription }];
@@ -383,11 +383,11 @@ - (void)room:(TVIRoom *)room didFailToConnectWithError:(nonnull NSError *)error{
383383
- (void)room:(TVIRoom *)room participantDidConnect:(TVIRemoteParticipant *)participant {
384384
participant.delegate = self;
385385

386-
[self sendEventWithName:roomParticipantDidConnect body:@{ @"roomName": room.name, @"participant": [participant toJSON] }];
386+
[self sendEventWithName:roomParticipantDidConnect body:@{ @"roomName": room.name, @"roomSid": room.sid, @"participant": [participant toJSON] }];
387387
}
388388

389389
- (void)room:(TVIRoom *)room participantDidDisconnect:(TVIRemoteParticipant *)participant {
390-
[self sendEventWithName:roomParticipantDidDisconnect body:@{ @"roomName": room.name, @"participant": [participant toJSON] }];
390+
[self sendEventWithName:roomParticipantDidDisconnect body:@{ @"roomName": room.name, @"roomSid": room.sid, @"participant": [participant toJSON] }];
391391
}
392392

393393
# pragma mark - TVIRemoteParticipantDelegate

0 commit comments

Comments
 (0)