Skip to content
This repository has been archived by the owner on Jun 1, 2020. It is now read-only.

Thread Accessibility Messaging #74

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

sha1sum
Copy link

@sha1sum sha1sum commented Feb 2, 2020

This adds messaging for new user join about accessibility and Slack threads. It also adds a @gopher trigger for the same message.

@sha1sum sha1sum requested review from theckman and freeformz February 2, 2020 15:11
@sha1sum
Copy link
Author

sha1sum commented Feb 3, 2020

Not sure who generally reviews these so I looked at a couple and it seems @freeformz does these things sometimes. I was also talking to @theckman about this previously on Slack. This is why I have two reviewers.

@sha1sum sha1sum requested review from vcabbage and removed request for freeformz February 9, 2020 15:22
handlers/join.go Outdated Show resolved Hide resolved
gopher.go Outdated Show resolved Hide resolved
sha1sum and others added 2 commits March 1, 2020 22:49
@theckman
Copy link
Collaborator

Would you mind re-raising this against https://github.com/gobridge/gopherbot for consideration? I'm going to be archiving this repo.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants