Skip to content

Commit 9573268

Browse files
committed
docs: document with the results of the code (code only) analysis
1 parent ca3d5b2 commit 9573268

File tree

1 file changed

+52
-0
lines changed

1 file changed

+52
-0
lines changed

docs/2024_code_review.md

Lines changed: 52 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,52 @@
1+
## Registration
2+
3+
- Date of first shipment for operations \*\*\*
4+
- models/operation.py
5+
- DateOfFirstShipmentChoices specific to 2024
6+
- schema/operation.py
7+
- Field for date of first shipment
8+
- api/.../test_new_entrant_application.py
9+
- test_changing_registration_purpose.py
10+
- test_operation_registration.py
11+
12+
## Reporting
13+
14+
- ReportProduction model has an "apr_dec_production" field
15+
- ReportComplianceSummaryProduct has an "apr_dec_production" field
16+
- activities' schemas
17+
All the activities' schemas are 2024 specific.
18+
-> Copy/Paste for 2025?
19+
-> Make rules to know which ones changed?
20+
21+
- reporting/json\*schemas/2024/\*\*/\_.json
22+
- tests for the configuration json is also specific to 2024 (See `reporting/tests/models/program_configuration_tests`)
23+
24+
- Compliance Summary
25+
26+
- ReportConplianceSummary.initial_compliance_period defaults to 2024
27+
- Compliance service
28+
29+
- Naics regulatory values hardcoded to fetch with an initial compliance period 2024 (probably not an issue?)
30+
- allocated_for_compliance_2024 is calculated no matter what (along with the general compliance value)
31+
- "emissions attributable for compliance" and each products "allocated compliance emissions" use the 2024-specific value
32+
- excess/credited emissions also use the 2024-specific allocation
33+
- tests for compliance service (test_compliance_service_static.py) also only test 2024 scenarios
34+
35+
## Service
36+
37+
- test_service_utils.py could use any date and not 2024
38+
- test_operation_service.py checks the date of first shipment according to the 2024 rules
39+
40+
## Frontend
41+
42+
- administrationRegistrationInformation.ts encodes the date of first shipment as 2024 in the options
43+
- newEntrantOperation.ts encodes the date of first shipment as 2024 in the options
44+
- finalReviewFields.ts includes apr-dec production for 2024
45+
- a few references to the "apr_dec" fields in
46+
47+
- ComplianceSummaryForm.tsx
48+
- finalReviewFields.ts
49+
- finalReview/reportTypes.ts
50+
- jsonSchema/complianceSummary.ts
51+
52+
- lots of compliance tests use 2024 as a base when they don't really have to

0 commit comments

Comments
 (0)