Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add unicast method #233

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

add unicast method #233

wants to merge 8 commits into from

Conversation

perlish
Copy link

@perlish perlish commented Aug 28, 2018

with this method user can send unicast and multicast message to specific users

src/communication.rs Outdated Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
edit variable name to be descriptive
revert debug value
@perlish
Copy link
Author

perlish commented Sep 8, 2018

i fixed variable name to be more descriptive and debug value also removed

@bjornmolin
Copy link

I would love to have this unicast method. Is there any downside with it? (or why has it not been merged to master)

@perlish
Copy link
Author

perlish commented May 5, 2019

@bjornmolin there is no downside with this

@Marcuccio
Copy link

Is the unicast method no longer available?.
It cannot be merged? why?

@perlish
Copy link
Author

perlish commented Aug 7, 2019

use this repo https://github.com/perlish/ws-rs

@Niedzwiedzw
Copy link

why on earth is it not merged?

@Niedzwiedzw
Copy link

@Innectic ?

girard-g added a commit to girard-g/ws-rs that referenced this pull request Dec 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants