Page MenuHomeElementl
Differential D9065

[dagit] Modify App initialization config
ClosedPublic
Actions

Authored by • dish on Jul 26 2021, 9:21 PM.
Tags
None
Referenced Files
F2989829: D9065.diff
Thu, Jun 8, 8:37 AM
Unknown Object (File)
Thu, May 11, 5:22 AM
Unknown Object (File)
May 9 2023, 1:32 AM
Unknown Object (File)
May 6 2023, 7:28 PM
Unknown Object (File)
Apr 25 2023, 3:00 PM
Unknown Object (File)
Apr 7 2023, 8:34 PM
Unknown Object (File)
Apr 7 2023, 4:21 AM
Unknown Object (File)
Apr 2 2023, 10:33 AM
View All Files
Subscribers
None

Details

Reviewers
• dgibson
• bengotow
• prha
Commits
R1:7a03bb0ffe30: [dagit] Modify App initialization config
Summary

A couple things:

  • The subscriptionParams config value is not currently used
  • We want to have the flexibility to add headers to HttpLink

These headers will also be applied as the connectionParams. This is a little awkward, but seems like it might be the best way for us to get initialization info to the server.

Test Plan

Buildkite. Run Dagit, verify that all http and ws connections behave correctly.

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

• dish created this revision.Jul 26 2021, 9:21 PM
Harbormaster completed remote builds in B34219: Diff 42297.Jul 26 2021, 9:30 PM
• dish requested review of this revision.Jul 26 2021, 9:30 PM
• dish edited the summary of this revision. (Show Details)Jul 27 2021, 2:19 PM
• dish edited the test plan for this revision. (Show Details)
• dish added reviewers: • bengotow, • prha.
• prha accepted this revision.Jul 27 2021, 3:20 PM
This revision is now accepted and ready to land.Jul 27 2021, 3:20 PM
Closed by commit R1:7a03bb0ffe30: [dagit] Modify App initialization config (authored by • dish). · Explain WhyJul 27 2021, 3:30 PM
This revision was automatically updated to reflect the committed changes.
• dish added a commit: R1:7a03bb0ffe30: [dagit] Modify App initialization config.

Revision Contents
Changeset List

PathSize
js_modules/
dagit/
packages/
core/
src/
app/
12 lines

Diff 42360

View Options

js_modules/dagit/packages/core/src/app/AppProvider.tsx

Loading...