Skip to content

[WIP] Upgrade to PatternFly 6 and rework to use the PatternFly ChatBot extension components #856

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

[WIP] Upgrade to PatternFly 6 and rework to use the PatternFly ChatBot extension components #856

wants to merge 10 commits into from
+17,533 −9,137

Conversation

kyoto
Copy link
Member

@kyoto kyoto commented Jul 29, 2025

No description provided.

@kyoto kyoto requested a review from jgbernalp July 29, 2025 09:35
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 29, 2025
@openshift-ci openshift-ci bot requested review from JoaoFula and joshuawilson July 29, 2025 09:35
@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Jul 29, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign xrajesh for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

jgbernalp
jgbernalp reviewed Jul 29, 2025
View reviewed changes
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Aug 5, 2025
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Aug 7, 2025
@kyoto kyoto force-pushed the pattern-fly-6 branch 7 times, most recently from c3d0060 to e118d41 Compare August 13, 2025 02:06
@kyoto kyoto mentioned this pull request Aug 25, 2025
Closed
@kyoto kyoto force-pushed the pattern-fly-6 branch 4 times, most recently from d9c677c to aa43741 Compare September 2, 2025 06:54
@kyoto
Copy link
Member Author

kyoto commented Sep 2, 2025

/retest

@kyoto kyoto force-pushed the pattern-fly-6 branch 3 times, most recently from 426394b to 8e1f05b Compare September 5, 2025 10:05
@kyoto
Copy link
Member Author

kyoto commented Sep 6, 2025

/retest

kyoto added 7 commits September 7, 2025 15:37
@kyoto
PatternFly 6 fixes
fe52884 
Run `npx @patternfly/pf-codemods@latest . --v6 --fix`
Pass menu toggle icons to the `icon` prop instead of as children
  - This is now the recommended approach from PatternFly 6
Fix Dropdown's onSelect call
  - This now takes a string ID rather than the pod object
Run `npx @patternfly/class-name-updater . --v6 --fix`
Run `npx @patternfly/css-vars-updater . --fix`
Switch from Page to Stack for layout
@kyoto
Remove PatternFly CSS that is no longer required
4cb5098
@kyoto
Reduce padding injected at bottom of web console pages
412ba85 
Less padding is needed for the 4.19 web console layout
@kyoto
Use useUserSettings to simplify useHideLightspeed
38d8439
@kyoto
Install @patternfly/chatbot and upgrade PatternFly packages
c9cf386
@kyoto
Refactor to use PatternFly AI Chatbot's components
838d87f 
- Use PatternFly AI Chatbot's ChatbotHeader, MessageBar, Message &
  feedback components
- Combines all of the prompt input logic into it's own component
- Update tests
@kyoto
Use useUserSettings hook to determine the current web console theme
1f3e46b
@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Sep 7, 2025

@kyoto: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JoaoFula JoaoFula Awaiting requested review from JoaoFula

@joshuawilson joshuawilson Awaiting requested review from joshuawilson

1 more reviewer

@jgbernalp jgbernalp jgbernalp left review comments

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kyoto @jgbernalp @openshift-merge-robot