DOC-655 Docs for updated asset selection syntax #27672
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: nikki everett <[email protected]>
neverett
changed the title
…assets docs, link to new asset selection syntax doc Signed-off-by: nikki everett <[email protected]>
Signed-off-by: nikki everett <[email protected]>
Signed-off-by: nikki everett <[email protected]>
Signed-off-by: nikki everett <[email protected]>
neverett
changed the title
salazarm
reviewed
Feb 19, 2025
salazarm
reviewed
Feb 19, 2025
Signed-off-by: nikki everett <[email protected]>
Signed-off-by: nikki everett <[email protected]>
neverett
commented
Feb 19, 2025
Signed-off-by: nikki everett <[email protected]>
![graphite-app[bot]](https://avatars.githubusercontent.com/in/158384?s=60&v=4){kind=small}
Signed-off-by: nikki everett <[email protected]>
Signed-off-by: nikki everett <[email protected]>
salazarm
reviewed
Feb 20, 2025
docs/src/code-examples-content.js
Outdated
| @@ -1,610 +0,0 @@ | |||
|
|
|||
There was a problem hiding this comment.
deleting this seems unintentional ?
There was a problem hiding this comment.
This file is rebuilt in production, so doesn't need to be checked in (although checking it in won't cause any harm)
salazarm
reviewed
Feb 20, 2025
salazarm
reviewed
Feb 20, 2025
salazarm
reviewed
Feb 20, 2025
salazarm
reviewed
Feb 20, 2025
salazarm
reviewed
Feb 20, 2025
salazarm
reviewed
Feb 20, 2025
salazarm
reviewed
Feb 20, 2025
salazarm
reviewed
Feb 27, 2025
salazarm
reviewed
Feb 27, 2025
salazarm
reviewed
Feb 27, 2025
salazarm
reviewed
Feb 27, 2025
|
Hmm, I kind of want the path examples to stand out a bit more, It's lost in the sea of examples and I think when people see it a lightbulb will go off on their head that maybe we should move it to the top? |
Signed-off-by: nikki everett <[email protected]>
Signed-off-by: nikki everett <[email protected]>
Signed-off-by: nikki everett <[email protected]>
Signed-off-by: nikki everett <[email protected]>
Signed-off-by: nikki everett <[email protected]>
Signed-off-by: nikki everett <[email protected]>
docs/docs/guides/build/assets/asset-selection-syntax/reference.md
Outdated
Show resolved
Hide resolved
Signed-off-by: nikki everett <[email protected]>
Signed-off-by: nikki everett <[email protected]>
Signed-off-by: nikki everett <[email protected]>
Signed-off-by: nikki everett <[email protected]>
docs/docs/guides/build/assets/asset-selection-syntax/reference.md
Outdated
Show resolved
Hide resolved
Signed-off-by: nikki everett <[email protected]>
salazarm
reviewed
Feb 28, 2025
salazarm
reviewed
Feb 28, 2025
Co-authored-by: Marco polo <[email protected]>
Co-authored-by: Marco polo <[email protected]>
salazarm
reviewed
Feb 28, 2025
| sidebar_position: 100 | ||
| --- | ||
|
|
||
| This page contains common example asset selection queries and their implementation in Python, CLI, and the Dagster UI. For a full explanation of the filters, layers, operands, and functions that you can use to construct your own queries, see "[Asset selection syntax reference](reference)". |
There was a problem hiding this comment.
This is the first page in the dropdown and immediately directs you to the second page, so maybe the reference should go first?
There was a problem hiding this comment.
Yeah, I can put the reference page first -- I put the examples page first in order to make the path examples as prominent as possible, but those should be easier to find now that they're at the top of the page, regardless of the page's position in the sidebar.
Signed-off-by: nikki everett <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary & Motivation
Updates "Asset selection syntax" doc with the new asset selection syntax.
How I Tested These Changes
Local build.
Changelog