Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "[ui] Upgrade styled-components to v6 (#26280)" #26639

Merged
merged 1 commit into from
Dec 20, 2024

Conversation

hellendag
Copy link
Member

@hellendag hellendag commented Dec 20, 2024

Summary & Motivation

We've seen reports of styles failing to be applied in the app in certain (very difficult to reproduce) conditions, and the recent styled-components upgrade seems like a possible culprit. Revert to try to resolve the issue.

How I Tested These Changes

TS/lint/jest. Load the app, verify that everything renders as expected.

Changelog

[ui] FIx an issue where the app sometimes loads with styles missing.

Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@hellendag hellendag marked this pull request as ready for review December 20, 2024 17:00
Copy link

github-actions bot commented Dec 20, 2024

Deploy preview for dagit-storybook ready!

✅ Preview
https://dagit-storybook-bd4drhqn1-elementl.vercel.app
https://dish-revert-sc-6.components-storybook.dagster-docs.io

Built with commit f989e2e.
This pull request is being automatically deployed with vercel-action

Copy link

github-actions bot commented Dec 20, 2024

Deploy preview for dagit-core-storybook ready!

✅ Preview
https://dagit-core-storybook-75uhk1lnj-elementl.vercel.app
https://dish-revert-sc-6.core-storybook.dagster-docs.io

Built with commit f989e2e.
This pull request is being automatically deployed with vercel-action

@hellendag hellendag force-pushed the dish/revert-sc-6 branch 2 times, most recently from ce0a46b to eb14788 Compare December 20, 2024 17:21
This reverts commit 1a2a067.

[INTERNAL_BRANCH=dish/plus-revert-sc-6]
@hellendag hellendag merged commit 916ceae into master Dec 20, 2024
2 of 3 checks passed
@hellendag hellendag deleted the dish/revert-sc-6 branch December 20, 2024 17:32
gibsondan pushed a commit that referenced this pull request Dec 20, 2024
## Summary & Motivation

We've seen reports of styles failing to be applied in the app in certain
(very difficult to reproduce) conditions, and the recent
styled-components upgrade seems like a possible culprit. Revert to try
to resolve the issue.

## How I Tested These Changes

TS/lint/jest. Load the app, verify that everything renders as expected.

## Changelog

[ui] FIx an issue where the app sometimes loads with styles missing.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants