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

Replace updated_at date with created_at date in CodelistVersion view #2292

Merged
merged 2 commits into from
Jan 15, 2025

Conversation

Jongmassey
Copy link
Contributor

@Jongmassey Jongmassey commented Jan 9, 2025

For certain types of codelist (dm+d, SNOMED CT etc)
every time the coding system is updated the
versions of every codelist using this system
get updated and thus the updated_at field
is not very useful to understand the timeline
of codelist versions.

Therefore, switch to created_at which is
hopefully less confusing.

Part fix for #2285

@Jongmassey Jongmassey force-pushed the Jongmassey/display-codelistversion-created-at branch from 342761d to 1a30b51 Compare January 14, 2025 17:18
@Jongmassey Jongmassey changed the title updated_at -> created_at Replace updated_at date with created_at date in CodelistVersion view Jan 14, 2025
@Jongmassey Jongmassey marked this pull request as ready for review January 14, 2025 17:26
For certain types of codelist (dm+d, SNOMED CT etc)
every time the coding system is updated the
versions of every codelist using this system
get updated and thus the updated_at field
is not very useful to understand the timeline
of codelist versions.

Therefore, switch to created_at which is
hopefully less confusing
@Jongmassey Jongmassey force-pushed the Jongmassey/display-codelistversion-created-at branch from 1a30b51 to e0a27f2 Compare January 14, 2025 17:59
@Jongmassey Jongmassey merged commit a49f968 into main Jan 15, 2025
5 checks passed
@Jongmassey Jongmassey deleted the Jongmassey/display-codelistversion-created-at branch January 15, 2025 22:38
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