bcgov / cas-reporting

This is for the Clean Growth Digital Services team for work related to reporting.
Apache License 2.0
0 stars 0 forks source link

Activity page - General Stationary Combustion (other than non-compression and non-processing) #302

Closed dleard closed 1 month ago

dleard commented 2 months ago

Description:

Wireframe:

Following #253 we have a full stack example of how to generate an activity emissions form using the API form builder service.The implementation here will be really similar to GSC - without line tracing. Most of the needed database records will already exist.

Documentation on the full stack steps necessary to create an activity page are in our docs

Use the excel file (BCIERS Reporting Fields V2 in the SME channel files - Reporting App directory) provided by IRC to determine what the necessary activity, source type, gas type, methodology, reporting fields & related configurations that need to be added are.

Acceptance Criteria:

Given I am on the GSC other than non-compression activity form page When I first land on the page Then the activity schema is fetched & displayed

Given the schema has rendered correctly When I make selections that have dependencies (gas type, methodology) Then the correct options and fields as defined from the configuration are shown

Development Checklist:

Definition of Ready (Note: If any of these points are not applicable, mark N/A)

Definition of Done (Note: If any of these points are not applicable, mark N/A)

Notes:

-

ayeshmcg commented 1 month ago

@suhafa

Url for testing : https://cas-bciers-frontend-dev.apps.silver.devops.gov.bc.ca/reporting/reports/5/facilities/1/activities/gsc_other_than_non_compression

Adding the fuel dropdowns was waiting on a discussion about the emission categories, fuels & how to apply that relationship. As a result, these dropdowns will not be visible on the activity page for now.

suhafa commented 1 month ago

Thanks @ayeshmcg for your work on this! Design review time:

  1. This has to do more with the UI, but noticing that emissions field area is offsetting the solid fill towards the right side and jutting out like so:

Image We can remove the solid fill from the Emissions element to not have this issue? Not a priority

  1. In the task list, the 'Mobile Combustion' is highlighted instead of 'General stationary combustion (other than non-compression and non-processing). Could just be a routing thing?

Image

  1. For the bread crumbs, the activity shows up with '_' as a placeholder for space. Can we have it instead say 'GSC: other than non-compression and non-processing combustion'

Image

ayeshmcg commented 1 month ago

Thanks, @suhafa

  1. I can implement a quick fix for the first issue.
  2. The sync between the task list and the multi-step form needs to be addressed separately. Currently, this issue affects all the activity pages.
  3. Regarding the breadcrumbs, they currently reflect the URL path. We can modify the component to display custom text as needed.
ayeshmcg commented 1 month ago

@suhafa
I have added the fix for spacing in the activity page.

patriciarussellCAS commented 1 month ago

Thanks @ayeshmcg - it looks like Dylan may have also made a ticket for this; #319 Can you confirm that we can close this? Or should we keep 319 open in case the problem occurs again?

ayeshmcg commented 1 month ago

@patriciarussellCAS I think we can close this one, as we already have https://github.com/bcgov/cas-reporting/issues/319.