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

🐛 correct charts in old WDI #3343

Merged
merged 3 commits into from
Oct 1, 2024
Merged

🐛 correct charts in old WDI #3343

merged 3 commits into from
Oct 1, 2024

Conversation

paarriagadap
Copy link
Contributor

@paarriagadap paarriagadap commented Sep 30, 2024

Charts were mistakingly using the previous WDI version instead of the 2024 update. Here I am just migrating the indicators.

@owidbot
Copy link
Contributor

owidbot commented Sep 30, 2024

Quick links (staging server):

Site Admin Wizard

Login: ssh owid@staging-site-wdi-migrate-charts

chart-diff: ✅
  • 13/13 reviewed charts
    • Modified: 13/13
    • New: 0/0
data-diff: ✅ No differences found
Legend: +New  ~Modified  -Removed  =Identical  Details
Hint: Run this locally with etl diff REMOTE data/ --include yourdataset --verbose --snippet

Automatically updated datasets matching weekly_wildfires|excess_mortality|covid|fluid|flunet|country_profile|garden/ihme_gbd/2019/gbd_risk are not included

Edited: 2024-10-01 08:30:08 UTC
Execution time: 4.39 seconds

@paarriagadap paarriagadap marked this pull request as ready for review October 1, 2024 08:28
@paarriagadap paarriagadap merged commit ac6606d into master Oct 1, 2024
8 checks passed
@paarriagadap paarriagadap deleted the wdi-migrate-charts branch October 1, 2024 08:31
paarriagadap added a commit that referenced this pull request Oct 6, 2024
* empty

* improve err message

* 🐛 don't get slugs of unpublished charts

---------

Co-authored-by: lucasrodes <[email protected]>
Co-authored-by: Marigold <[email protected]>
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