diff options
| author | Joas Schilling <coding@schilljs.com> | 2021-06-16 09:17:22 +0300 |
|---|---|---|
| committer | Joas Schilling <coding@schilljs.com> | 2021-06-16 09:17:22 +0300 |
| commit | c2aa0042c1c9450311f0b1950f5eaae9f2249dfd (patch) | |
| tree | 15551b21725751310a75ef66e5972338a7d1e945 | |
| parent | bd4787dfd158a6e34023cb8809f99c7b3cccbfb1 (diff) | |
Don't refresh the room when the participant list changes
Currently when a call with 40 people is started and they join one by one
with 3.1s difference, each of the participant will fetch the conversation
40 times resulting in 1.6k requests.
At the same time the info doesn't really change anymore as the participant
details of other participants have been removed from the room API with
version 3. A refreshing of the room therefor only makes sense when once,
to update the hasCall and callFlag attributes.
The conversations callFlag however is not used at the moment and the worst
that could happen without the refreshing is the values updating on the next
refreshing of the complete room list which is done every 30 seconds.
Signed-off-by: Joas Schilling <coding@schilljs.com>
| -rw-r--r-- | src/App.vue | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/App.vue b/src/App.vue index 4b3d0883a..31c0becda 100644 --- a/src/App.vue +++ b/src/App.vue @@ -186,7 +186,6 @@ export default { if (!getCurrentUser()) { EventBus.$off('shouldRefreshConversations', this.debounceRefreshCurrentConversation) } - EventBus.$off('Signaling::participantListChanged', this.debounceRefreshCurrentConversation) document.removeEventListener('visibilitychange', this.changeWindowVisibility) }, @@ -197,7 +196,6 @@ export default { }) EventBus.$on('shouldRefreshConversations', this.debounceRefreshCurrentConversation) } - EventBus.$on('Signaling::participantListChanged', this.debounceRefreshCurrentConversation) if (this.$route.name === 'conversation') { // Update current token in the token store |