Angush / trellobot

A Discord bot for logging Trello events. Note: this is no longer functional and not maintained. Forks may be more up-to-date.
52 stars 46 forks source link

Inaccurate event timestamp check #12

Open Spaghetti657 opened 5 years ago

Spaghetti657 commented 5 years ago

Based on this article, it looks like only the first 8 characters of a Trello event ID contain the timestamp, but trello-events compares the entire ID when trying to determine whether an event is newer. I noticed the problem when my bot instance started posting the same event over and over again.

The fix is simple (line 76 in trello-events/index.js): actionId = parseInt(boardActions[ix].id.slice(0, 8), 16);