search

UQComputingSociety / uqcsbot-slack

:mortar_board: UQCSbot: our friendly little Slack bot
https://slack.uqcs.org.au
MIT License
55 stars 44 forks source link

Create !crisis, !emergency and !mentalhealth #435

Closed d-lord closed 5 years ago

d-lord commented 5 years ago

Both do the same thing - provide a list of crisis resources for quick reference.

Happy to add any other resources that may be of use :)

TRManderson commented 5 years ago

Good guy dal strikes again ❤️

bradleysigma commented 5 years ago

Implements #175

TRManderson commented 5 years ago

All caps is standard for constants

On Sat., 18 May 2019, 14:53 David Lord, notifications@github.com wrote:

@d-lord commented on this pull request.

In uqcsbot/scripts/crisis.py https://github.com/UQComputingSociety/uqcsbot/pull/435#discussion_r285329261 :

@@ -0,0 +1,18 @@ +from uqcsbot import bot, Command + + +@bot.on_command("crisis") +@bot.on_command("mentalhealth") +def handle_crisis(command: Command):

  • '''
  • !crisis or !mentalhealth - Get a list of mental health crisis resources.
  • '''
  • response = """Mental health/crisis resources

I like Tom's a bit more. response, RESPONSE or?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/UQComputingSociety/uqcsbot/pull/435?email_source=notifications&email_token=AATDF27WUJ5YPMD53GNL6P3PV6DVFA5CNFSM4HNTCX3KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOBZA2XMI#discussion_r285329261, or mute the thread https://github.com/notifications/unsubscribe-auth/AATDF22IQQSHE5LAKPZGS2DPV6DVFANCNFSM4HNTCX3A .

d-lord commented 5 years ago

I'm wondering if it should include !emergency too. Three aliases seems a bit excessive but I want it to be intuitive/accessible. Maybe change !mentalhealth to !emergency?

d-lord commented 5 years ago

Hey fam, I've actioned the review and also added !emergency as an alias. Would appreciate a re-review so we can merge and forget about it until it's needed 😄