Page MenuHomeElementl

[dagit] Fix manifest
ClosedPublic

Authored by dish on Jul 22 2021, 8:09 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 14, 12:25 AM
Unknown Object (File)
May 7 2023, 2:18 AM
Unknown Object (File)
May 5 2023, 10:42 PM
Unknown Object (File)
Apr 23 2023, 2:09 PM
Unknown Object (File)
Apr 13 2023, 6:57 AM
Unknown Object (File)
Apr 10 2023, 12:36 PM
Unknown Object (File)
Apr 9 2023, 12:38 PM
Unknown Object (File)
Mar 17 2023, 12:32 PM
Subscribers
None

Details

Summary

The manifest currently fails to load in cases where auth is required on the backend, as credentials are not included in the request. Fix this, as indicated by https://developer.mozilla.org/en-US/docs/Web/Manifest#deploying_a_manifest_with_the_link_tag.

Test Plan

Load Dagit in prod build and dev build. Verify that the manifest request succeeds with no errors.

Diff Detail

Repository
R1 dagster
Branch
dish-manifest
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

Oh wow haven't seen this before, good find

This revision is now accepted and ready to land.Jul 27 2021, 6:31 PM
This revision was automatically updated to reflect the committed changes.