-
-
Notifications
You must be signed in to change notification settings - Fork 8.5k
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
feat(theme): show unlisted/draft banners in dev mode #10376
Conversation
✅ [V2]
To edit notification comments on pull requests, go to your Netlify site configuration. |
⚡️ Lighthouse report for the deploy preview of this PR
|
Size Change: -638 B (-0.01%) Total Size: 11.5 MB
ℹ️ View Unchanged
|
LGTM thanks Did some refactoring to cleanup but this works well |
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
FYI this broke our builds since we were using overrides for |
@B4nan it's not marked as a safe component to swizzle so we keep the right to do refactors like this one in minor versions. I'll see how we could make it easier for those like you who swizzle unsafe components. Maybe listing changes in our release notes would be helpful? |
The module declaration was changed in facebook/docusaurus#10376
- Upgrade Docusaurus packages to `^3.5.0` - Fix `@theme/Unlisted` import, since the module declaration was changed in facebook/docusaurus#10376. - Update imports to reflect the movement of some exports from `@docusaurus/theme-common/internal` into `@docusaurus/plugin-content-docs/client`. See facebook/docusaurus#10316.
- Upgrade Docusaurus packages to `^3.5.0` - Fix `@theme/Unlisted` import, since the module declaration was changed in facebook/docusaurus#10376. - Update imports to reflect the movement of some exports from `@docusaurus/theme-common/internal` into `@docusaurus/plugin-content-docs/client`. See facebook/docusaurus#10316. Also updates config.json to correct branch names.
- Upgrade Docusaurus packages to `^3.5.0` - Fix `@theme/Unlisted` import, since the module declaration was changed in facebook/docusaurus#10376. - Update imports to reflect the movement of some exports from `@docusaurus/theme-common/internal` into `@docusaurus/plugin-content-docs/client`. See facebook/docusaurus#10316. Also updates config.json to correct branch names.
Motivation
Fix #8285
Test Plan
ci & unit test & argos
Test links
Deploy preview: https://deploy-preview-10376--docusaurus-2.netlify.app/