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

373 cement production #2491

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

373 cement production #2491

wants to merge 4 commits into from

Conversation

gdalcengio
Copy link
Contributor

#373

This pr creates adds necessary configuration for Cement Production reporting activity

  • feat: Add cement production reporting activity migration, data, and json
  • chore: Tweak previous migration and add UISchema
  • test: add test for cement production configuration

Steps to test:

  • Make reset_db & make run
  • Login as BCEID
  • Nav to an SFO operation and set reporting activity to cement production
  • Emission should have gas type as C02
  • Field "Number of times in the reporting year that missing data procedures were followed" should exist
  • Should have 5 methodologies and CEMS should have no reporting fields

@dleard
Copy link
Contributor

dleard commented Nov 22, 2024

This one is a little more complicated than the other activities. "Month" is not really a field, but a header separation for a set of fields. Each set of fields in the excel sheet (Clinker production, emission factor...) needs to be reported for each month (ditto for quarter). So I think we'll need to create a custom schema for this where each month is an object that has a set of fields & try to display that with a custom widget in a way that doesn't make the page super long. Happy to chat about how to implement this together.

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

Successfully merging this pull request may close these issues.

2 participants