openfoodfoundation / openfoodnetwork

Connect suppliers, distributors and consumers to trade local produce.
https://www.openfoodnetwork.org
GNU Affero General Public License v3.0
1.1k stars 714 forks source link

[Reports] Allow user to show/hide totals with a checkbox toggle #6846

Closed lin-d-hop closed 1 year ago

lin-d-hop commented 3 years ago

Description

To increase the utility of this report we wish to enable the user to show and hide the Totals rows (as displayed in the CSV mock)

The user will have an additional checkbox filter:

When the box is checked, total row will be included. When the box is not check the CSV download and the UI report will not have totals rows

This needs to be applied to both the By Producer and By Order pivots of this report.

Acceptance Criteria & Tests

  1. The Display Total Rows checkbox is included
  2. When it is ticked the totals rows are display in CSV download and on UI
  3. When it is not ticked the totals rows are not displayed in CSV download and on UI
  4. All fields are translatable
RachL commented 2 years ago

@lin-d-hop do we still need this issue now that the checkbox will be standard on all reports?

lin-d-hop commented 1 year ago

Agree @RachL. This is now the default :)