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

fix: bump extension docs to v6 #4344

Merged
merged 3 commits into from
Oct 29, 2024
Merged

Conversation

nicolethoen
Copy link
Collaborator

No description provided.

@nicolethoen nicolethoen requested review from kmcfaul and dlabaj October 25, 2024 16:53
@patternfly-build
Copy link
Contributor

patternfly-build commented Oct 25, 2024

@andrew-ronaldson
Copy link
Contributor

I noticed the log viewer extension has an option to toggle dark mode.
It didn't seem to work but the dark theme switcher in the masthead does.

@nicolethoen
Copy link
Collaborator Author

nicolethoen commented Oct 25, 2024

I will open a follow up issue with the log-viewer. The dark theme should be able be enabled even when the rest of the app is in light theme. It's probably an issue with how the tokens were updated.
patternfly/react-log-viewer#82

@nicolethoen nicolethoen force-pushed the bump_exts_6 branch 2 times, most recently from 9763350 to fe019c5 Compare October 28, 2024 19:15
@kmcfaul
Copy link
Contributor

kmcfaul commented Oct 29, 2024

@nicolethoen User feedback modals are behaving oddly (appending to the page / not being on top)

@nicolethoen
Copy link
Collaborator Author

I'll open a bug for user-feedback. That was not introduced with this PR. It's already broken on staging.

@nicolethoen
Copy link
Collaborator Author

Bug fix issue for user feedback: patternfly/react-user-feedback#85

"@patternfly/quickstarts": "6.0.0",
"@patternfly/react-virtualized-extension": "6.0.0",
"@patternfly/react-templates": "6.0.0",
"@patternfly/virtual-assistant": "2.0.0"
}
},{
"name": "prerelease",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple lines down can we flip the prerelease both to hidden? Or remove it.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏻

@kmcfaul kmcfaul merged commit bd2fd88 into patternfly:main Oct 29, 2024
4 checks passed
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.

5 participants