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

Update to refer to slack chatroom #58

Merged
merged 1 commit into from
Jan 19, 2017
Merged

Update to refer to slack chatroom #58

merged 1 commit into from
Jan 19, 2017

Conversation

murphyke
Copy link
Member

Fix #57.

I installed Jekyll and browsed and link-checked the results.

@bruth I don't know if you have control of the verbiage on http://harvest-platform.slack.com or http://harvest-platform.slack.com/signup, but if so, they could be made a little more inviting.

http://harvest-platform.slack.com mentions the pre-approved email domains but gives the impression that anybody else is out of luck. It should probably suggest sending email to the harvest info email address for people who will need invitations.

http://harvest-platform.slack.com/signup says "Contact your Team Administrator for an invitation" but doesn't say how to do that.

@bruth
Copy link
Contributor

bruth commented Jan 18, 2017

Thanks, this is great!

I don't know if you have control of the verbiage [...]

I perused the admin settings and it does not appear to be. Also they only offer invitation only or by domain for signups, so I presume that is why they have that "contact the administrator" message.

@bruth
Copy link
Contributor

bruth commented Jan 18, 2017

Rather than removing the chat page, could you include most of what is in the _includes/chatroom.md snippet? Otherwise the http://harvest.reserach.chop.edu/chat/ link will be broken.

Signed-off-by: Kevin Murphy <murphyke@email.chop.edu>
@murphyke
Copy link
Member Author

Good call; re-pushed.

@bruth bruth merged commit 68ff6d6 into gh-pages Jan 19, 2017
@bruth bruth deleted the issue-57-gh-pages branch January 19, 2017 15:32
@bruth
Copy link
Contributor

bruth commented Jan 19, 2017

Deployed: http://harvest.research.chop.edu/chat/ Thanks!

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.

2 participants