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

Upstream Tox support added here #24

Open
patcon opened this issue Oct 28, 2018 · 2 comments
Open

Upstream Tox support added here #24

patcon opened this issue Oct 28, 2018 · 2 comments

Comments

@patcon
Copy link
Contributor

patcon commented Oct 28, 2018

Hey y'all! Just found your fork, and seems like you go things working :)

Might you be open to upstreaming your changes so that others can use them? I know that there were others who expressed interest in Tox support in the bridge suggestion thread

Anyhow, happy to help if I can!

@envsh
Copy link
Owner

envsh commented Dec 26, 2018

Hey, I very expect to think of merge to upstream.

Because tox group chat feature still not enough, so here use extra sqlite db to save group members info. Seems not very clear codebase like other implemeted protocols in upstream repo.

I think make a thin part of tox protocol bridge to merge to upstream first if possible.
So you can help me analysis and discuss this, maybe later make some code, since I have no mature idea now.

@patcon
Copy link
Contributor Author

patcon commented Dec 31, 2018

Cool! Will come back to this after holidays :) thanks for reply

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

No branches or pull requests

2 participants