-
Notifications
You must be signed in to change notification settings - Fork 66
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 Keybase widget showing grincoin#general chat #182
Conversation
Very cool @nijynot, well done as always!! 😀 Don't really have much to add aside from some nit-pick, here goes:
once again, awesome work! 👏 |
Good points!
|
Awesome! Will just check with others that there are no new-repo-objections 👍
Oh, that would be super! :D |
@nijynot repo created here, https://github.com/mimblewimble/keybase-chat-widget Perhaps it makes sense to update this pr so that it points to the new repo before we merge? |
Migrated over to the new repo and updated the link to point to it too. |
Awesome! However, Github fired a security vulnerability alert in the repo based on
Any chance we can fix this on your deployment before we merge it? |
Actually this shouldn't be a blocker I realise, sorry about that. It's not related to /site but to your deployment in the end and I will open an issue I the repo instead. LGTM 👍 |
This PR adds a Keybase chat widget showing the grincoin#general channel.
As Keybase does not have an open API to access team messages, this solution relies on me serving team message and member count from these two endpoints:
https://grin.nijynot.com/messages
andhttps://grin.nijynot.com/team
.I did not go with Chia's
keybase-live-feed
at the end as I feel it was a bit too complicated for a small chat widget, and it'd be faster to just build it from scratch (given that we'd want to change the UI).grin-keybase-chat
is instead a Node.js server on a Digital Ocean droplet and the repo can be found on mimblewimble/keybase-chat-widget.