Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Add keyPrefix to presence updates fixes #515#516

Open
denga wants to merge 1 commit into
tlaverdure:masterfrom
denga:fix-presence-publishing
Open

Add keyPrefix to presence updates fixes #515#516
denga wants to merge 1 commit into
tlaverdure:masterfrom
denga:fix-presence-publishing

Conversation

@denga

@denga denga commented May 3, 2020

Copy link
Copy Markdown

Publishing of update events also use the keyPrefix.

@hcw12889

hcw12889 commented Aug 7, 2020

Copy link
Copy Markdown

Hello, very thankful that the changes in database/redis.ts fixes the error I'm encountering in #537 .
However why the changes in subscribers/redis-subscriber.ts is necessary? It prevents the echo-server from logging the presence updates instead.

@amumayjang

amumayjang commented Jun 15, 2021

Copy link
Copy Markdown

merge fixes for master. please !

@amora2972

Copy link
Copy Markdown

yes please merge this pr if it has no issues.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants