Skip to content

Chore/584 review facility activities matches operation - #4253

Draft
acatchpole wants to merge 3 commits into
developfrom
chore/584-review-facility-activities-matches-operation
Draft

Chore/584 review facility activities matches operation#4253
acatchpole wants to merge 3 commits into
developfrom
chore/584-review-facility-activities-matches-operation

Conversation

@acatchpole

@acatchpole acatchpole commented Feb 13, 2026

Copy link
Copy Markdown
Contributor

Card: bcgov/cas-reporting#584

Changes

  • Created a new endpoint to return report_operation_activities
  • Replaced ALL Activities with report_operation_activities on FacilityReviewPage

Changes Unrelated to Card

  • Fixed a flaky test in test_report_emission_allocation_service.py

Testing

Expected behaviour:
The activity list on the Facility Review page will match those selected on the Review Operation Information page.

No other changes regarding selected/deselected or preserving or deleting activity data was included in this PR.

report_product_emission_allocations.order_by('report_product__product_id'), expected_data
report_product_emission_allocations.order_by('report_product__product_id', 'emission_category_id'),
expected_data,
):

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this change is unrelated to the card. just fixes a flaky test

@acatchpole
acatchpole force-pushed the chore/584-review-facility-activities-matches-operation branch from cdaf975 to a5f42f4 Compare February 19, 2026 18:17
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.

1 participant