-
Notifications
You must be signed in to change notification settings - Fork 214
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 the Cinny client #286
Add the Cinny client #286
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few things need to be updated, but thanks for putting the energy in this PR :)
Can this be merged? |
Not quite, I would like to get @ajbura's input on the two unresolved threads (who to add as an author and the self assessed maturity of Cinny) |
okay, sounds reasonable :) |
can this be merged now? @thibaultamartin |
Just chiming in to see if there's anything that'd need changing before this can be merged @thibaultamartin |
Signed-off-by: 3nt3 <[email protected]>
Signed-off-by: 3nt3 <[email protected]>
Signed-off-by: 3nt3 <[email protected]>
Signed-off-by: 3nt3 <[email protected]>
6c5cde3
to
008bc0d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apologies for the delay, things got a little crazy pre-FOSDEM. Thanks a lot for this contribution and for keeping the Matrix ecosystem vibrant!
This PR adds the Cinny client.
Basically a cleaner version of #234.
I am not sure about the copyright of the SVG, it is licensed under
Creative Commons Attribution 4.0 International License
so it's okay to use, I just don't know where to put the attribution.