ErythroGuild / irene

<Erythro>'s server admin bot.
Mozilla Public License 2.0
0 stars 0 forks source link

Add TaskQueue to interactables #338

Closed Ernest314 closed 1 year ago

Ernest314 commented 1 year ago

There's currently a chance for updates from component interactions and interactable cleanup (and update, in the case of Selection) to cause a race condition. Putting all that logic in a TaskQueue would prevent this.

Ernest314 commented 1 year ago

Don't forget about Farm.Selection!

Ernest314 commented 1 year ago

Turns out Confirm does need a TaskQueue, because the ephemeral message can still be sent to multiple devices if the same person has the channel open on multiple instances (e.g. devices) at once.

Ernest314 commented 1 year ago

See #362 for Farm.Selection TaskQueue implementation.

Ernest314 commented 1 year ago

Closed in 9ea673c8a6a2b981bec276e75f97eaf96fd22923.