wakfi / roboboi

Discord bot
MIT License
2 stars 0 forks source link

[CNR] //poll can record 0's in the results for all options when it shouldn't #35

Open wakfi opened 3 years ago

wakfi commented 3 years ago

Short Description

Polls can report results inaccurately in the embed due to results having all 0's even when there are many reactions.

Steps to Reproduce

Step 1: Working on this part still, so far CNR with short polls and few reactions.

Step 2:

Step 3:

Step 4:

Expected Result (how should it be behaving?)

The results should be reported accurately.

Actual Result (what is the current behavior?)

The results are reported as all zeroes even though there are many responses.

Current Bot Version (run the info command and look at the footer)

d673686

wakfi commented 3 years ago

image 7 + 13 = 21, reactions are accurate to the overall vote total but not the individual options

wakfi commented 3 years ago

This may be due to the partial outage that occurred early during the poll, although I'm not certain of this. It would track as, although the poll is still in memory, it is using client.messageReactionAdd/Remove listeners instead of raw packet listeners which are what enable listening to uncached messages. It also may be due to the absolute spaghetti fest that is the poll command, which will be solved either before or after the syntax redesign.