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.

Sign up for GitHub

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

Jump to bottom

✨ all-charts block updates #4514

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

✨ all-charts block updates #4514

wants to merge 1 commit into from
+70 −3

Conversation

ikesau
Copy link
Member

@ikesau ikesau commented Jan 31, 2025

Resolves #4426

Changes

  • Adds a button to the data catalog on the all charts block
  • Adds tags to the gdoc AttachmentsContext
  • Uses a gdoc's first tag to automatically generate the heading of the all charts block
all.charts.demo.mp4

@ikesau ikesau requested a review from rakyi January 31, 2025 22:32
@ikesau ikesau self-assigned this Jan 31, 2025
@ikesau ikesau force-pushed the all-charts-updates branch from 17e1039 to 36da2f2 Compare January 31, 2025 22:38
@owidbot
Copy link
Contributor

owidbot commented Jan 31, 2025

Quick links (staging server):

Site Dev Site Preview Admin Wizard Docs

Login: ssh owid@staging-site-all-charts-updates

SVG tester:

Number of differences (default views): 0 ✅
Number of differences (all views): 0 ✅

Edited: 2025-01-31 23:00:50 UTC
Execution time: 1.24 seconds

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rakyi rakyi Awaiting requested review from rakyi

Assignees

@ikesau ikesau

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Cycle 2025.1: All charts block updates
2 participants
@ikesau @owidbot