errbotio / err-backend-slackv3

Slack Events and RTM backend for Errbot
GNU General Public License v3.0
27 stars 28 forks source link

Fix channel mentions #57 #58

Closed nzlosh closed 2 years ago

alpinweis commented 2 years ago

nice to see the channelid property added to StackRoom. Until now, one had to do something like this to get it w/o throwing an error.

channel_id = msg.to.id if msg.is_group else msg.to.channelid

Would this also be added to other backends (e.g. for compatibility and consistency)?

nzlosh commented 2 years ago

The original slack backend SlackRoom class has channelid which is an alias to the id property. There's no intention to generalise channelid across backends at this point. If it makes sense to do that, PRs would be welcome.