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 the Cinny client #286

Merged
merged 4 commits into from
Feb 20, 2023
Merged

Add the Cinny client #286

merged 4 commits into from
Feb 20, 2023

Conversation

3nt3
Copy link
Contributor

@3nt3 3nt3 commented Oct 12, 2022

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.

Copy link
Contributor

@thibaultamartin thibaultamartin left a 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 :)

src/open/clients/cinny.js Outdated Show resolved Hide resolved
src/open/clients/cinny.js Outdated Show resolved Hide resolved
src/open/clients/cinny.js Outdated Show resolved Hide resolved
src/open/clients/cinny.js Outdated Show resolved Hide resolved
src/open/clients/cinny.js Outdated Show resolved Hide resolved
src/open/clients/index.js Outdated Show resolved Hide resolved
@3nt3
Copy link
Contributor Author

3nt3 commented Oct 18, 2022

Can this be merged?

@thibaultamartin
Copy link
Contributor

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)

@3nt3
Copy link
Contributor Author

3nt3 commented Oct 18, 2022

okay, sounds reasonable :)

src/open/clients/Cinny.js Outdated Show resolved Hide resolved
@3nt3 3nt3 requested review from ajbura and thibaultamartin and removed request for thibaultamartin and ajbura October 30, 2022 16:09
@3nt3
Copy link
Contributor Author

3nt3 commented Nov 15, 2022

can this be merged now? @thibaultamartin

@3nt3
Copy link
Contributor Author

3nt3 commented Nov 26, 2022

Just chiming in to see if there's anything that'd need changing before this can be merged @thibaultamartin

@3nt3 3nt3 requested review from kfiven and removed request for thibaultamartin December 15, 2022 15:03
Copy link
Contributor

@thibaultamartin thibaultamartin left a 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!

@thibaultamartin thibaultamartin merged commit 7b62332 into matrix-org:main Feb 20, 2023
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.

4 participants