UQComputingSociety / uqcsbot-slack

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

added jobs-bulletin watchdog #376

Closed nicklambourne closed 6 years ago

nicklambourne commented 6 years ago

As per title. Fixes #344. Feedback welcome.

nicklambourne commented 6 years ago

On posting in channel example:

screen shot 2018-09-06 at 11 28 57 am
nicklambourne commented 6 years ago

Also wondering if it would be possible to make a subprocess that sticks around for ~5min after someone posts to delete the chat redirection message if the original message is deleted?

mitchmcdee commented 6 years ago

message wording looks good 👍 could "Only employers advertising jobs" be reworded though maybe? "Only job advertisements"?

will give a code review "soon"

mitchmcdee commented 6 years ago

re: "Also wondering if it would be possible to make a subprocess that sticks around for ~min after someone posts to delete the chat redirection message if the original message is deleted?"

Sounds definitely possible but 10x more work + effort, save it for another PR :^) if you think its super valuable chuck up an issue, but personally I don't think its worth

jdcaperon commented 6 years ago

I'll second the change to Only employers advertising jobs.