bcgov / cas-ggircs

Climate Action Secretariat's Greenhouse Gas Industrial Reporting and Control System
Apache License 2.0
3 stars 4 forks source link

As a CAS user, I want to define the charges associated with each fuel defined in the Carbon Tax Act #431

Closed dleard closed 2 years ago

dleard commented 2 years ago

Description:

CAS users should be able to add/edit the fuel charges for each CTA fuel via a UI. This would be changing data in the ggircs_parameters.fuel_charge table. The UI for this can re-use some aspects of the un-mapped fuels component.

Acceptance Criteria:

Given I am logged in to the ggircs app When I am on the landing page Then I can navigate to a CTA management page And I can see a selectable list of the CTA fuels on the left side of the screen

Given I am in the CTA management UI When I select a CTA fuel type Then I can see the related fuel charges and tax periods for that fuel

Given I am in the CTA management UI When I need to add a fuel charge for an upcoming reporting year Then I can add fuel charge data for that year

Given I am in the CTA management UI When I need to edit a fuel charge Then I can edit fuel charge data

Development Checklist:

dleard commented 2 years ago

Hey team! Please add your planning poker estimate with ZenHub @BCerki @gurjmatharu @matthieu-foucault @pbastia

dleard commented 2 years ago

@LindsayMacfarlane To test this:

LindsayMacfarlane commented 2 years ago

Thanks @dleard! I appreciate the instructions :) Perhaps I don't have the needed permissions as I'm not able to select (or see) any fuel types in order to edit/create rate periods (see screenshot). image.png

dleard commented 2 years ago

Ah, yes that's because of the issue here in #430, which could really be a bug rather than tech debt I suppose. The permissions are not in sync with the database being dropped in dev. I cycled the pod and you should be able to see fuels now.

LindsayMacfarlane commented 2 years ago

Perfect - thank you! Tested in dev by:

This was really intuitive to use!

Meets acceptance criteria. Thanks for all your work on this one @dleard