merefield
Setembro 2, 2024, 6:55am
19
Observe que este PR foi levantado e deve fechar completamente este problema para sempre, tendo já sido testado na instância do criador do bug através de um fork do plugin de enquete:
main ← merefield:ranked_choice_voter_expansion_fix
opened 07:02AM - 20 Aug 24 UTC
Currently, for Ranked Choice results and voter list expansion, the code is repla… cing the voter list with the newly fetched page of voters instead of appending to the existing voter list. This is resolved in this PR and brings it in line with regular polls.
See: https://meta.discourse.org/t/ranked-choice-poll-does-not-reflect-change-of-votes-in-outcome/321227/18?u=merefield
I've made the tests more strict to cover this and confirmed test failure with existing code without this fix.
NB I'm aware code changes around this functionality have been a little numerous of late, so this time, in the interest of minimising review burden and curtailing the iteration, I made sure the changes in this PR were fully tested in Production on the very poll which identified the issue. We've successfully tested the solution in the Production instance of the bug raiser and they have signed off. (We removed the core plugin in app.yml and cloned a fork of Poll that includes these changes).
2 curtidas