Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Polls show different results depending on what part of the timeline you've loaded #23756

Closed
robintown opened this issue Nov 11, 2022 · 2 comments
Closed

Polls show different results depending on what part of the timeline you've loaded #23756

robintown opened this issue Nov 11, 2022 · 2 comments
Assignees
@kerryarchibald
Labels
A-Polls O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect

Comments

@robintown
Copy link
Member

Steps to reproduce

  1. Start a poll in an active room
  2. Get a bunch of responses, interspersed with messages
  3. Wait until there are enough messages to put the first poll responses fully out of your client's view of the timeline
  4. Reply to the poll
  5. Reload Element

Outcome

What did you expect?

Polls should always show consistent and accurate results.

What happened instead?

Depending on where I scroll in the timeline, the poll results change significantly. For example if I reload Element and then view the reply to the poll, the reply preview only counts the more recent responses. If I scroll all the way back to the original posting of the poll, I see all of the responses counted (I think). If I copy the link to the poll, close Element, and then launch Element using this link, I get yet another result counting only the earlier responses.

Operating system

NixOS unstable

Browser information

Firefox 106.0.2

URL for webapp

develop.element.io

Application version

Element version: 8f8a9e1-react-e8d4fbb8ff2b-js-67f343d6f0a2 Olm version: 3.2.12

Homeserver

Synapse 1.71.0

Will you send logs?

No
@robintown robintown added T-Defect S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Uncommon Most users are unlikely to come across this or unexpected workflow A-Polls labels Nov 11, 2022
@turt2live
Copy link
Member

This will be because we don't call /relations

@kerryarchibald kerryarchibald self-assigned this Feb 19, 2023
@kerryarchibald
Copy link
Contributor

Fixed by matrix-org/matrix-react-sdk#9877
(Polls now use /relations API to fetch all votes)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kerryarchibald kerryarchibald

Labels
A-Polls O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@turt2live @kerryarchibald @robintown