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

Unify tallying procedures #357

Merged
merged 1 commit into from
Apr 27, 2023
Merged

Unify tallying procedures #357

merged 1 commit into from
Apr 27, 2023

Conversation

edulix
Copy link
Contributor

@edulix edulix commented Apr 5, 2023

Parent issue: https://github.com/sequentech/meta/issues/65

Parent issue: https://github.com/sequentech/meta/issues/65

  • Problem description

Admin console shows two possible avenues for launching the tally: tally all votes and tally active users. But they work very differently, one works with parent-children elections (tally active votes) and the other doesn't.

However this inconvenience is not reflected in the UI. To make things worse, the default selected method is the tally all votes, even for parent-children elections which don't work with that mode.

  • Proposed solution

Make both modes (tally all census and tally active voters) work with parent-children elections and make the tally active voters the default.

  • Tasks

    • Change the admin-console to show differently the launch-tally dialog to allow selecting which children elections to choose for both modes.
    • Change the admin-console to request the tally always to the IAM.

Fixes sequentech/meta#65

- Problem description

Admin console shows two possible avenues for launching the tally: tally all votes and tally active users. But they work very differently, one works with parent-children elections (tally active votes) and the other doesn't.

However this inconvenience is not reflected in the UI. To make things worse, the default selected method is the tally all votes, even for parent-children elections which don't work with that mode.

- Proposed solution

Make both modes (tally all census and tally active voters) work with parent-children elections and make the tally active voters the default.

- Tasks

  - Change the `admin-console` to show differently the launch-tally dialog to allow selecting which children elections to choose for both modes.
  - Change the `admin-console` to request the tally always to the IAM.
@edulix edulix force-pushed the feat/7.2.x/unify-launch-tally branch from a7a792c to 28e59a5 Compare April 27, 2023 07:56
@edulix edulix changed the base branch from 7.2.x to 7.3.x April 27, 2023 08:02
@edulix edulix merged commit eff6f96 into 7.3.x Apr 27, 2023
@edulix edulix deleted the feat/7.2.x/unify-launch-tally branch April 27, 2023 09:00
edulix added a commit that referenced this pull request Apr 27, 2023
Parent issue: sequentech/meta#65

- Problem description

Admin console shows two possible avenues for launching the tally: tally
all votes and tally active users. But they work very differently, one
works with parent-children elections (tally active votes) and the other
doesn't.

However this inconvenience is not reflected in the UI. To make things
worse, the default selected method is the tally all votes, even for
parent-children elections which don't work with that mode.

- Proposed solution

Make both modes (tally all census and tally active voters) work with
parent-children elections and make the tally active voters the default.

- Tasks

- Change the `admin-console` to show differently the launch-tally dialog
to allow selecting which children elections to choose for both modes.
  - Change the `admin-console` to request the tally always to the IAM.
edulix added a commit that referenced this pull request Apr 27, 2023
Parent issue: sequentech/meta#65

- Problem description

Admin console shows two possible avenues for launching the tally: tally
all votes and tally active users. But they work very differently, one
works with parent-children elections (tally active votes) and the other
doesn't.

However this inconvenience is not reflected in the UI. To make things
worse, the default selected method is the tally all votes, even for
parent-children elections which don't work with that mode.

- Proposed solution

Make both modes (tally all census and tally active voters) work with
parent-children elections and make the tally active voters the default.

- Tasks

- Change the `admin-console` to show differently the launch-tally dialog
to allow selecting which children elections to choose for both modes.
  - Change the `admin-console` to request the tally always to the IAM.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant