Skip to content

Conversation

@mariajahja
Copy link
Member

Description

Match config behavior in doctor-visits where DAY_SHIFT is set to 0.

Changelog

  • delphi_claims_hosp/config.py

@mariajahja mariajahja self-assigned this Dec 11, 2021
Author: Maria Jahja
Created: 2020-06-01
Modified: 2020-09-27
Modified: 2021-12-11
Copy link
Contributor

Choose a reason for hiding this comment

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

is there a reason we're not relying on git to track this?

Copy link
Contributor

Choose a reason for hiding this comment

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

Possible reason: production deployments are stripped of all .git metadata, so markup like this would be the only way to verify which changes are in effect. It's not a good reason (perennially backburnered is having deployment add a VERSION file with the gitref, which would be better) but it's better than nothing

Copy link
Contributor

@krivard krivard left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@chinandrew chinandrew left a comment

Choose a reason for hiding this comment

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

:shipit: :shipit: :shipit:

@krivard krivard merged commit 270c168 into main Dec 13, 2021
@krivard krivard deleted the claims_hosp/remove_day_shift branch December 13, 2021 14:59
@mariajahja
Copy link
Member Author

@krivard Just a note that the server running the daily generation has been updated.

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.

4 participants