matrix-org / conference-bot

The conductor for your orchestra^Wconference
Apache License 2.0
44 stars 8 forks source link

Don't show error message if user doesn't have MXID set in penta and is invited via email #167

Closed kittykat closed 1 year ago

kittykat commented 1 year ago

Currently when user has email address in penta, but no MXID, the bot shows this error when trying to invite them:

‼ | Error inviting undefined / <id> to !foo:fosdem.org - ignoring

It would be more helpful if it could say

Invited <name> / <id> by email to !foo:fosdem.org

or if it doesn't know that it's definitely sent the email invite, something like:

Attempted to invite <name> / <id> by email to !foo:fosdem.org

reivilibre commented 1 year ago

Invalid; it turned out to be rate limiting by the IS. Logging to be improved by #169.