Skip to content

feat(orchestrator-form): enable backward navigation in multi-step stepper#3709

Open
lokanandaprabhu wants to merge 1 commit into
redhat-developer:mainfrom
lokanandaprabhu:orchestrator/multi-step-stepper-back-nav
Open

feat(orchestrator-form): enable backward navigation in multi-step stepper#3709
lokanandaprabhu wants to merge 1 commit into
redhat-developer:mainfrom
lokanandaprabhu:orchestrator/multi-step-stepper-back-nav

Conversation

@lokanandaprabhu

Copy link
Copy Markdown
Member

Hey, I just made a Pull Request!

Story - https://redhat.atlassian.net/browse/RHIDP-15114

Summary

  • Allow users to click completed stepper steps in multi-step workflow forms to jump back to earlier steps
  • Clear stale async validation errors when navigating backward (Back or stepper click)
  • Resolve per-step error schemas correctly when jumping between steps
  • Fix useGetExtraErrors to set RJSF __errors as arrays (prevents crashes when validate:url fails)
Screen.Recording.2026-07-08.at.2.39.24.PM.mov

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or Updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)

…pper

Allow users to click completed stepper steps to jump back while keeping
the current and future steps display-only. Clear stale validation errors
when navigating backward and resolve per-step error schemas correctly.

Co-authored-by: Cursor <cursoragent@cursor.com>
@rhdh-gh-app

rhdh-gh-app Bot commented Jul 8, 2026

Copy link
Copy Markdown

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-orchestrator-form-react workspaces/orchestrator/plugins/orchestrator-form-react patch v2.9.2
@red-hat-developer-hub/backstage-plugin-orchestrator-form-widgets workspaces/orchestrator/plugins/orchestrator-form-widgets patch v1.11.2

@sonarqubecloud

sonarqubecloud Bot commented Jul 8, 2026

Copy link
Copy Markdown

@codecov

codecov Bot commented Jul 8, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 50.74627% with 33 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.55%. Comparing base (00f80fc) to head (538313e).
⚠️ Report is 4 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3709      +/-   ##
==========================================
+ Coverage   54.30%   54.55%   +0.24%     
==========================================
  Files        2349     2351       +2     
  Lines       89718    89796      +78     
  Branches    25119    25088      -31     
==========================================
+ Hits        48722    48987     +265     
+ Misses      40648    40461     -187     
  Partials      348      348              
Flag Coverage Δ *Carryforward flag
adoption-insights 83.70% <ø> (ø) Carriedforward from 00f80fc
ai-integrations 68.35% <ø> (ø) Carriedforward from 00f80fc
app-defaults 69.79% <ø> (ø) Carriedforward from 00f80fc
augment 46.39% <ø> (ø) Carriedforward from 00f80fc
boost 74.68% <ø> (ø) Carriedforward from 00f80fc
bulk-import 72.46% <ø> (ø) Carriedforward from 00f80fc
cost-management 14.10% <ø> (ø) Carriedforward from 00f80fc
dcm 61.81% <ø> (ø) Carriedforward from 00f80fc
extensions 61.53% <ø> (ø) Carriedforward from 00f80fc
global-floating-action-button 71.18% <ø> (ø) Carriedforward from 00f80fc
global-header 59.71% <ø> (ø) Carriedforward from 00f80fc
homepage 50.23% <ø> (ø) Carriedforward from 00f80fc
install-dynamic-plugins 56.77% <ø> (ø) Carriedforward from 00f80fc
konflux 91.49% <ø> (ø) Carriedforward from 00f80fc
lightspeed 68.81% <ø> (ø) Carriedforward from 00f80fc
mcp-integrations 85.46% <ø> (ø) Carriedforward from 00f80fc
orchestrator 43.07% <50.74%> (+3.27%) ⬆️
quickstart 65.63% <ø> (ø) Carriedforward from 00f80fc
sandbox 79.56% <ø> (ø) Carriedforward from 00f80fc
scorecard 82.67% <ø> (ø) Carriedforward from 00f80fc
theme 61.26% <ø> (ø) Carriedforward from 00f80fc
translations 7.25% <ø> (ø) Carriedforward from 00f80fc
x2a 78.68% <ø> (ø) Carriedforward from 00f80fc

*This pull request uses carry forward flags. Click here to find out more.


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 00f80fc...538313e. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@lholmquist lholmquist left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tested with the example quarkus workflow and worked like the demo video

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants