Page MenuHomeElementl

[dagit] Extract ContentRoot to be an App child
ClosedPublic

Authored by dish on Aug 9 2021, 4:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 25, 9:00 PM
Unknown Object (File)
Fri, Mar 10, 7:01 PM
Unknown Object (File)
Fri, Mar 10, 7:01 PM
Unknown Object (File)
Fri, Mar 10, 7:01 PM
Unknown Object (File)
Thu, Mar 9, 12:18 PM
Unknown Object (File)
Feb 20 2023, 12:18 AM
Unknown Object (File)
Feb 11 2023, 4:05 AM
Unknown Object (File)
Feb 10 2023, 6:36 PM
Subscribers
None

Details

Summary

Provide ContentRoot from the index level. This allows better customization of the route tree.

I had considered making route configuration more abstract (and even landed some code for it) but it's just not worth introducing that level of complexity yet, and there are more pressing needs that this helps with.

Test Plan

Run Dagit, verify that workspace, instance, and settings routes all work correctly.

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This is so that the cloud index can add cloud-specific routes?

This revision is now accepted and ready to land.Aug 9 2021, 4:36 PM

This is so that the cloud index can add cloud-specific routes?

Correct.