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

docs: Describe chart preview via API + expose DebugPanel via flag #1206

Merged
merged 3 commits into from
Oct 6, 2023

Conversation

bprusinowski
Copy link
Collaborator

@bprusinowski bprusinowski commented Oct 6, 2023

Closes #1200.

As the types can differ a lot between different chart types, and there are some hidden dependencies to some properties (e.g. multi filter connected to temporal X field, some sorting options only available for column charts, segmentation not available for maps, etc, etc), I decided to point to the file with types in the docs and expose the configurator & interactive filters debug panel via flag.

By doing this, editors will be able to edit a chart, tailor it to their needs, and dump the state to the console to copy and modify it slightly to fit their needs. With this we don't need to prepare a complete list of all combination possibilities between the types, which would be really difficult, but rather encourage to pre-create a chart and edit it afterwards, making sure that all necessary side effects have been applied.

@bprusinowski bprusinowski requested a review from ptbrowne as a code owner October 6, 2023 13:34
@vercel
Copy link

vercel bot commented Oct 6, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
visualization-tool ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 6, 2023 1:54pm

@bprusinowski bprusinowski force-pushed the docs/update-chart-config-docs branch from ae54d56 to 5de620a Compare October 6, 2023 13:52
@bprusinowski bprusinowski merged commit 80d3e0c into main Oct 6, 2023
3 of 4 checks passed
@bprusinowski bprusinowski deleted the docs/update-chart-config-docs branch October 6, 2023 14:39
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.

Update Documentation: Parameteres to call up chart (API)
2 participants