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

30 may live data #91

Merged
merged 14 commits into from
May 30, 2024
Merged

30 may live data #91

merged 14 commits into from
May 30, 2024

Conversation

t-surtees
Copy link
Collaborator

Pull request overview

Updating dashboard with most recent data.

Pull request checklist

Please check if your PR fulfils the following:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)
  • Tests have been run locally and are passing (run_tests_locally())
  • Code is styled according to tidyverse styling (checked locally with tidy_code())

What is the current behaviour?

Usual updates to the data.

What is the new behaviour?

Issue with git repo memory has been actioned. New SQL warehouse being used.

Anything else

There are a few different commits given the issue with the repo memory has been fixed, and the change to the SQL warehouse name, as well as the usual data updates.

@t-surtees t-surtees requested a review from c-h-rowe May 30, 2024 08:06
@t-surtees t-surtees requested a review from c-h-rowe May 30, 2024 09:09
@t-surtees t-surtees merged commit 41c12cf into main May 30, 2024
2 of 3 checks passed
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.

3 participants