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

Add metadata items to tables and databases (CaDeT) #1040

Merged
merged 10 commits into from
Nov 15, 2024

Conversation

murdo-moj
Copy link
Contributor

@murdo-moj murdo-moj commented Nov 12, 2024

Added metadata items

  • Audience: Internal
  • Provider: Analytical Platform We don't know what provider means. The audience item tells users if they can share data or not.
  • Metadata last updated
  • Data last updated Temporarily hidden until the run_results are reliable
  • Refresh schedule: This is based on available tags in CaDeT

Changed

  • I make use now of the timestamp formatter in Django so I have changed timestamps to just be raw as datahub collects them

Questions for reviewers:

  • Is it worth making Audience and Provider an Enum type? I have done this

@murdo-moj murdo-moj requested a review from a team as a code owner November 12, 2024 14:59
@murdo-moj murdo-moj changed the title Add metadata items to tables and databases Add metadata items to tables and databases (CaDeT) Nov 12, 2024
@murdo-moj murdo-moj merged commit 9ba56ef into main Nov 15, 2024
10 checks passed
@murdo-moj murdo-moj deleted the fmd-1023-add-cadet-metadata-fields branch November 15, 2024 15:12
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