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

Made Standard Readme compliant #28

Merged
merged 2 commits into from
Jul 28, 2016
Merged

Conversation

RichardLitt
Copy link
Member

This is a discussion repo. That means that all of the work gets done in the [issues](https://github.com/ipfs/apps/issues). Go check them out!
[![](https://img.shields.io/badge/made%20by-Protocol%20Labs-blue.svg?style=flat-square)](http://ipn.io)
[![](https://img.shields.io/badge/project-IPFS-blue.svg?style=flat-square)](http://ipfs.io/)
[![](https://img.shields.io/badge/freenode-%23ipfs-blue.svg?style=flat-square)](http://webchat.freenode.net/?channels=%23ipfs)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Personally I'd prefer this to link to https://chat.ipfs.io/#/room/#ipfs:matrix.org

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Huh, haven't seen this before. When was this set up? Seems like a lot of extra stuff more than just the IRC room.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Huh. Interesting! @jbenet @diasdavid should we use this, instead?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did. I typed messages, but didn't see them coming up in IRCCloud. :?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for testing @RichardLitt, I think I know what the problem is now, will try to get it fixed asap

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm told this problem should be fixed sometime today.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@RichardLitt can you check that https://ipfs.io/ipfs/QmcSJjw7wQ8r8uUKBHb8n2RryXsKv49MS31xFwowxkgJ53/#/room/#ipfs:matrix.org works?

@lgierth if so, can you update the chat.ipfs.io dnslink to the above hash?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@RichardLitt https://chat.ipfs.io/#/room/#ipfs:matrix.org should work now, please update the link if you're happy with it :)

@JesseWeinstein
Copy link

LGTM, aside from the discussion about linking to Matrix vs Freenode webchat.

@RichardLitt
Copy link
Member Author

@davidar Undecided at the moment about chat.ipfs.io -- I'm going to merge this, for now, and will change all of the links from freenode's webchat later.

@RichardLitt RichardLitt merged commit 6c51a6c into master Jul 28, 2016
@RichardLitt RichardLitt deleted the feature/standard-readme branch July 28, 2016 18:33
@davidar
Copy link
Member

davidar commented Jul 29, 2016

will change all of the links from freenode's webchat later.

@RichardLitt ok, I'll hold you to that ;)

@jbenet
Copy link
Member

jbenet commented Aug 26, 2016

This was brought up to me recently I think. The concerns were that the
chat.ipfs.io UX loses people. I think we can get vector UX much closer to
being purely focused on the channel.
On Fri, Jul 29, 2016 at 04:14 David A Roberts notifications@github.com
wrote:

will change all of the links from freenode's webchat later.

@RichardLitt https://github.com/RichardLitt ok, I'll hold you to that ;)


You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
#28 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAIcoaPCoqEaLJrbsnlbMynEsid7XFpnks5qabaCgaJpZM4I3YeP
.

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