Skip to content

fix(Wizard): add useEffect import#12538

Open
LightOfHeaven1994 wants to merge 1 commit into
patternfly:mainfrom
LightOfHeaven1994:fix-wizard-use-effect-import
Open

fix(Wizard): add useEffect import#12538
LightOfHeaven1994 wants to merge 1 commit into
patternfly:mainfrom
LightOfHeaven1994:fix-wizard-use-effect-import

Conversation

@LightOfHeaven1994

@LightOfHeaven1994 LightOfHeaven1994 commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

What:

  1. Fix for Wizard Progress after submission so it doesn't fail on useEffect https://www.patternfly.org/components/wizard#progress-after-submission
  2. Fix for deprecated Wizard Finish so it doesn't fail on not-imported layout https://www.patternfly.org/components/wizard/react-deprecated/#finished

Closes #12537

Summary by CodeRabbit

  • Documentation
    • Updated the Wizard example to include an additional React hook in its import, keeping it aligned with the current example usage.
    • Updated the deprecated Wizard example documentation to import the appropriate layout styling module for the Bullseye layout.

@coderabbitai

coderabbitai Bot commented Jul 7, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

The Wizard example markdown files were updated to include missing imports: useEffect in the current example and layout in the deprecated example.

Changes

Wizard Documentation Fix

Layer / File(s) Summary
Fix example imports
packages/react-core/src/components/Wizard/examples/Wizard.md, packages/react-core/src/deprecated/components/Wizard/examples/Wizard.md
The Wizard example markdown files add the missing React hook and Bullseye layout imports used by the documented examples.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Related issues: #12537

Suggested reviewers: kmcfaul, thatblindgeye

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The PR addresses both reported examples by fixing the missing useEffect import and the deprecated Wizard layout import.
Out of Scope Changes check ✅ Passed No unrelated changes are shown; both edits directly support the Wizard example fixes in the linked issue.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: adding the missing useEffect import for Wizard.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@LightOfHeaven1994 LightOfHeaven1994 force-pushed the fix-wizard-use-effect-import branch from 7f8854f to a289531 Compare July 7, 2026 11:24
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.

Bug - [Component] - [short description]

1 participant