HomeElementl

Add snapshotId support to PiplineAndMode, display mode in more places

Description

Add snapshotId support to PiplineAndMode, display mode in more places

Summary:
This diff refactors PiplineAndMode a bit so that you can optionally include the snapshotId, which is rendered after the pipeline name and before the mode (per discussion with Alex).

I also expanded the places PipelineAndMode is used so that the mode is visible in a few more places, and renamed it PipelineReference which I think makes more sense now that renders the full specification of name+snapshot+mode with linking.

I also gave the component more ownership of the link generation, because I think asking people to provide a RepoAddress or explicitly type "repo-unknown" will encourage use of the RepoAddress whenever it's available, and also make sure you have to explicitly opt-out of linking anywhere the pipleine is referenced.

image.png (1×1 px, 435 KB)

image.png (1×1 px, 464 KB)

image.png (1×1 px, 287 KB)

image.png (1×1 px, 296 KB)

Test Plan: Review screenshots

Reviewers: dish

Reviewed By: dish

Differential Revision: https://dagster.phacility.com/D6824

Details

Provenance
bengotowAuthored on Mar 8 2021, 4:59 PM
Reviewer
dish
Differential Revision
D6824: Add snapshotId support to PiplineAndMode, display mode in more places
Parents
R1:979d4a1b1737: Eagerly set mode when changing partition sets #3711
Branches
Unknown
Tags
Unknown