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

378 refactor valid date away from created_at #2913

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

Conversation

acatchpole
Copy link
Contributor

@acatchpole acatchpole commented Feb 25, 2025

Card: bcgov/cas-reporting#378

Changes

  • Activity service: GET changed to use reporting_year_id instead of created_at
  • activity_save_service: POST changed to use reporting_year_id instead of created_at
  • updated relevant tests to use reporting_year_id and hardcoded in a valid reporting_year, otherwise the model bakery try using the year -7643453423 (because the reporting_year is an int field, and not a date field)
  • Updated name of supporting function (get_report_valid_year_from_version_id -> get_report_valid_date_...) to more accurately reflect the functions purpose.

Note:

The ticket included removing facility_id after the refactor since it would no longer be used. Work in (this)[https://github.com/bcgov/cas-reporting/issues/522] ticket likely mean it is still needed.

@acatchpole acatchpole changed the title 378 refactor valid date away from created at 378 refactor valid date away from created_at Feb 25, 2025
@acatchpole acatchpole force-pushed the 378-refactor-valid-date-away-from-created-at branch from 374651a to adc948f Compare February 25, 2025 23:46
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