Skip to content

Commit 35da9c5

Browse files
author
Daisuke Baba
committed
Remove subscribed status update as it's unnecessarily complicated when used in conjunction with read() function
1 parent dc3ce95 commit 35da9c5

File tree

1 file changed

+1
-6
lines changed

1 file changed

+1
-6
lines changed

src/generic-ble.js

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -490,8 +490,6 @@ export default function(RED) {
490490
if (err) {
491491
this.emit('error', err);
492492
this.log(`subscription error: ${err.message}`);
493-
} else {
494-
this.emit('subscribed');
495493
}
496494
});
497495
if (period > 0) {
@@ -510,7 +508,7 @@ export default function(RED) {
510508
});
511509
}
512510
};
513-
['connected', 'disconnected', 'subscribed', 'error', 'timeout'].forEach(ev => {
511+
['connected', 'disconnected', 'error', 'timeout'].forEach(ev => {
514512
this.on(ev, () => {
515513
try {
516514
Object.keys(this.nodes).forEach(id => {
@@ -558,9 +556,6 @@ export default function(RED) {
558556
payload: payload
559557
});
560558
});
561-
this.on('subscribed', () => {
562-
this.status({fill:'green',shape:'dot',text:`generic-ble.status.subscribed`});
563-
});
564559
}
565560
this.on('connected', () => {
566561
this.status({fill:'green',shape:'dot',text:`generic-ble.status.connected`});

0 commit comments

Comments
 (0)