You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
For images, the announcement could be changed from a sendMessage to a sendPhoto, though the text of the message will become a "caption" instead—and will be limited to 1024 characters.
We could do something like pass the caption text through metabot.util.html.sanitize, and revert to sendMessage if its length is greater than 1024.
Alternatively we could capture that error and retry using sendMessage.
While working on this, we should also just improve error handling in the daily announcement loop so one announcement's failure won't cause all announcements for that time slot to be skipped (see 26c7734).
The text was updated successfully, but these errors were encountered:
While working on this, we should also just improve error handling in the daily announcement loop so one announcement's failure won't cause all announcements for that time slot to be skipped (see 26c7734).
This past Wednesday's 8 a.m. Pacific announcement failed:
From #74:
metabot.util.html.sanitize
, and revert tosendMessage
if its length is greater than 1024.sendMessage
.While working on this, we should also just improve error handling in the daily announcement loop so one announcement's failure won't cause all announcements for that time slot to be skipped (see 26c7734).
The text was updated successfully, but these errors were encountered: