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.

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

[Story] Edit a poll #2179

Closed
6 tasks done
Tracked by #2020
alfogrillo opened this issue Nov 8, 2023 · 0 comments
Closed
6 tasks done
Tracked by #2020

[Story] Edit a poll #2179

alfogrillo opened this issue Nov 8, 2023 · 0 comments

Comments

@alfogrillo
Copy link

alfogrillo commented Nov 8, 2023

Description

  • As a poll creator
  • I want to edit a poll
  • So that I can amend polls quickly

Acceptance criteria

  • Poll creators see the "edit poll" button in the timeline when no one voted yet
    • Alternatively users can use the timeline menu to edit a poll
  • If someone voted, the "Edit poll" button becomes "End poll" an the edit isn't allowed anymore
    • In this case the timeline menu should't show the "Edit poll" option anymore
  • After the edit starts, the apps shows a similar form as for the creation of a poll but with all the fields filled with the data from the editing poll (design).
  • To complete the edit, the poll should pass all the preconditions we check for poll's creation (max 20 options, no empty options, no empty poll's question)
  • After the tap on "Done", the new event (the edited poll) is sent in the timeline.
    • The editor is allowed to complete the edit even if votes arrive during the edit. Votes sent during the edit should be considered "lost" and the edited poll must not take them into account.
  • Tap on "Cancel" shows this alert just if at least a change has been made in the form.
    • The same alert should be shown in the "create new poll" flow (copies need to updated).
  • Tap on "Delete a Poll" will show a confirmation alert. If the action is confirmed, the poll is deleted from the timeline.
  • Display the "(edited)" mark next to edited polls in the timeline

Size estimate

M

Dependencies

Preconditions

  • Editor must be the poll creator.
  • Poll must have no votes cast

Sign-off

Android

  • Design sign-off on completion
  • QA sign-off on completion
  • Product sign-off on completion

iOS

  • Design sign-off on completion
  • QA sign-off on completion
  • Product sign-off on completion

Out of scope

  • TBD

Open questions

  • Can room's administrators (people having enough power levels) edit the poll as poll creators?
  • What happens if votes are received while an a poll edit is in progress?
    • Legacy clients ignore the problem and allow the edit anyway. In this case the votes preceding the edit are not rendered in the timeline.
    • @VolkerJunginger suggests to disable the edit/raise an error when this happens
  • Is local echo working for edited event? What happens on the timeline if the edit fails?

Subtasks

Android

iOS

No tasks being tracked yet.

Other

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants