Page MenuHomeElementl

[dagit] Settings page

Authored by dish on Jun 28 2021, 4:10 PM.
Referenced Files
F2894493: D8567.diff
Mon, Mar 27, 12:31 AM
Unknown Object (File)
Wed, Mar 15, 7:50 AM
Unknown Object (File)
Fri, Mar 10, 8:02 PM
Unknown Object (File)
Tue, Mar 7, 12:46 AM
Unknown Object (File)
Feb 24 2023, 2:39 AM
Unknown Object (File)
Feb 13 2023, 8:45 AM
Unknown Object (File)
Feb 12 2023, 7:26 AM
Unknown Object (File)
Feb 9 2023, 8:06 PM



Restore the Settings page originally created for the earlier left navigation experiment. This gives us a place to:

  • Move the timezone selector, which occupies valuable left nav space in spite of being a one-time setting for users
  • Surface feature flags for enabling/disabling features without our needing to direct users to find /flags
Test Plan

Load dagit, verify that the settings cog renders and behaves correctly in the top nav. View Settings, verify that everything behaves correctly when clicked. View left nav, verify that scrolling is correct, inside and outside of Crag flag.

Diff Detail

R1 dagster
Lint Not Applicable
Tests Not Applicable

Event Timeline

I think it'd be nice to go ahead and rip out /flags or have it redirect to this new page, seems like having both isn't super useful.

It doesn't really make sense given we only have two flags, but it'd be really nice to differentiate between "Advanced Options"-style flags (eg: debug logging), and "Upcoming Features"-style flags. The mode/pipeline one is sort of on a different tier than the debug logging. That said, also seems having a subheader for each one is a bit silly...

This revision is now accepted and ready to land.Jun 30 2021, 4:14 AM
This revision was automatically updated to reflect the committed changes.