Details

Reviewers

dgibson
prha
johann
schrockn
bengotow

Commits

R1:e739c881d90b: [dagit] Consolidate pieces into "Instance details"

Summary

Proposed changes to the "Instance details" page and left navigation, following Scheduled Runs chat earlier.

Remove "Daemons" page in favor of tabs on "Instance details" page, as a location to render instance-level information about schedules and sensors.
Move "Instance details" link down to be a top-level link rather than a mini-link next to the version number.
Move and resize left nav items following the above change.

Test Plan

Verify changes described above.

Diff Detail

Repository

R1 dagster

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

dish created this revision.Jan 5 2021, 9:23 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Jan 5 2021, 9:28 PM

Harbormaster failed remote builds in B23664: Diff 28777!

Jest

Harbormaster completed remote builds in B23666: Diff 28780.Jan 5 2021, 9:47 PM

dish requested review of this revision.Jan 5 2021, 9:47 PM

This looks great to me! I'm a big fan of this change because I think it'll more clearly communicate the repo vs. instance distinction between the two UIs that show sensors and schedules 👍

js_modules/dagit/src/instance/InstanceSchedules.tsx

This isn't super relevant to this diff, but I've been working with another internal library recently that writes out spacers explicitly. So far I've found it's a little easier to visualize what the UI looks like than with nested groups where the spacing declaration is further away from where the spacing actually appears, might be an interesting exploration one day:

(Eg:

<Group direction="column">
    <SchedulerTimezoneNote schedulerOrError={scheduler} />
    <Spacer size={12} />
    <SchedulerInfo schedulerOrError={scheduler} daemonHealth={instance.daemonHealth} />
    <Spacer size={32} />
    {withSchedules.map((repository) => (
    <>
        <Subheading>{`${repository.name}@${repository.location.name}`}</Subheading>
        <Spacer size={12} />
        <SchedulesTable
            repoAddress={{name: repository.name, location: repository.location.name}}
            schedules={repository.schedules}
        />
    </>
    ))}
</Group>

This revision is now accepted and ready to land.Jan 6 2021, 4:38 PM

Agreed, I like this a lot better

<Subheading>{`${repository.name}@${repository.location.name}`}</Subheading>
<Spacer size={12} />
<SchedulesTable
  repoAddress={{name: repository.name, location: repository.location.name}}
  schedules={repository.schedules}
/>

My thought on this is that we should probably have a Section component that encompasses and standardizes the subheading+spacing+content, which should help us deduplicate some of the nesting and repetition.

Add 'Instance' header and rename page, following discussion.

dish retitled this revision from [dagit] RFC: Consolidate pieces into "Instance details" to [dagit] Consolidate pieces into "Instance details".Jan 7 2021, 8:27 PM

Harbormaster failed remote builds in B23846: Diff 28989!Jan 7 2021, 8:29 PM

dashboard icon

Harbormaster failed remote builds in B23852: Diff 28996!Jan 7 2021, 8:57 PM

Test, use CSS for uppercase headers

Harbormaster completed remote builds in B23853: Diff 28999.Jan 7 2021, 9:10 PM

Closed by commit R1:e739c881d90b: [dagit] Consolidate pieces into "Instance details" (authored by dish). · Explain WhyJan 7 2021, 10:52 PM

This revision was automatically updated to reflect the committed changes.

dish added a commit: R1:e739c881d90b: [dagit] Consolidate pieces into "Instance details".

[dagit] Consolidate pieces into "Instance details"
ClosedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 29024

js_modules/dagit/src/WebsocketStatus.tsx

js_modules/dagit/src/tests/App.test.tsx

js_modules/dagit/src/instance/DaemonList.tsx

js_modules/dagit/src/instance/InstanceConfig.tsx

js_modules/dagit/src/instance/InstanceDaemonRoot.tsx

js_modules/dagit/src/instance/InstanceDetails.tsx

js_modules/dagit/src/instance/InstanceHealthFragment.tsx

js_modules/dagit/src/instance/InstanceRoot.tsx

js_modules/dagit/src/instance/InstanceSchedules.tsx

js_modules/dagit/src/instance/InstanceSensors.tsx

js_modules/dagit/src/instance/InstanceStatusRoot.tsx

js_modules/dagit/src/instance/types/InstanceConfigQuery.ts

js_modules/dagit/src/instance/types/InstanceDaemonRootQuery.ts

js_modules/dagit/src/instance/types/InstanceDetailsQuery.ts

js_modules/dagit/src/instance/types/InstanceHealthQuery.ts

js_modules/dagit/src/nav/InstanceDetailsLink.tsx

js_modules/dagit/src/nav/LeftNav.tsx

js_modules/dagit/src/nav/RepositoryPicker.tsx

js_modules/dagit/src/nav/types/InstanceDetailSummaryQuery.ts

js_modules/dagit/src/schedules/ScheduleUtils.tsx

js_modules/dagit/src/sensors/SensorInfo.tsx

js_modules/dagit/src/sensors/SensorsRoot.tsx

js_modules/dagit/src/testing/ApolloTestProvider.test.tsx

[dagit] Consolidate pieces into "Instance details"ClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 29024

js_modules/dagit/src/WebsocketStatus.tsx

js_modules/dagit/src/__tests__/App.test.tsx

js_modules/dagit/src/instance/DaemonList.tsx

js_modules/dagit/src/instance/InstanceConfig.tsx

js_modules/dagit/src/instance/InstanceDaemonRoot.tsx

js_modules/dagit/src/instance/InstanceDetails.tsx

js_modules/dagit/src/instance/InstanceHealthFragment.tsx

js_modules/dagit/src/instance/InstanceRoot.tsx

js_modules/dagit/src/instance/InstanceSchedules.tsx

js_modules/dagit/src/instance/InstanceSensors.tsx

js_modules/dagit/src/instance/InstanceStatusRoot.tsx

js_modules/dagit/src/instance/types/InstanceConfigQuery.ts

js_modules/dagit/src/instance/types/InstanceDaemonRootQuery.ts

js_modules/dagit/src/instance/types/InstanceDetailsQuery.ts

js_modules/dagit/src/instance/types/InstanceHealthQuery.ts

js_modules/dagit/src/nav/InstanceDetailsLink.tsx

js_modules/dagit/src/nav/LeftNav.tsx

js_modules/dagit/src/nav/RepositoryPicker.tsx

js_modules/dagit/src/nav/types/InstanceDetailSummaryQuery.ts

js_modules/dagit/src/schedules/ScheduleUtils.tsx

js_modules/dagit/src/sensors/SensorInfo.tsx

js_modules/dagit/src/sensors/SensorsRoot.tsx

js_modules/dagit/src/testing/ApolloTestProvider.test.tsx

[dagit] Consolidate pieces into "Instance details"
ClosedPublic
Actions

Revision Contents
Changeset List

js_modules/dagit/src/tests/App.test.tsx