Skip to content

Conversation

@stevelikesmusic
Copy link
Contributor

@stevelikesmusic stevelikesmusic commented Dec 5, 2025

Closes: https://linear.app/lightdash/issue/CENG-199/embed-docs-for-all-embed-types

Updates embed docs to reflect current capabilities. We add additional pages to break up functionality into more logical and searchable pieces. This gives us the framework to add more embed types and capabilities in the future.

To review:

I didn't include any references for embed UI events because Rollstack is the only customer that has asked for this.

Claude gave me a good start—used Claude in the mintlify repo but pointed it at the lightdash repo for references. This could be a useful pattern on lightdash PRs where we open additional PRs here in the mintlify repo. I still had quite a bit to edit, but editing is much easier that writing from scratch.

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@stevelikesmusic stevelikesmusic self-assigned this Dec 5, 2025
@stevelikesmusic stevelikesmusic removed the request for review from hamzahc1 December 5, 2025 16:51
@stevelikesmusic stevelikesmusic marked this pull request as ready for review December 5, 2025 16:51
@linear
Copy link

linear bot commented Dec 5, 2025

Copy link
Contributor

@rephus rephus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, there is a bit of duplicate content between charts/dashboards and how to embed (like secrets) , but that is ok.

Dashboard guide is particularly large because of all the examples, I guess that is fine.

@stevelikesmusic
Copy link
Contributor Author

I tried to be comprehensive. I'm also happy to trim it down.

@stevelikesmusic stevelikesmusic force-pushed the 12-05-feat_update_embedding_docs branch from 4148a65 to 990a4c7 Compare December 8, 2025 11:30
@stevelikesmusic stevelikesmusic merged commit 876edf7 into main Dec 8, 2025
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