Skip to content

fix: auto-switch to My Agents tab after creation and fix intent dialog reopen#3216

Closed
rrbanda wants to merge 1 commit into
redhat-developer:mainfrom
rrbanda:pr/post-create-redirect
Closed

fix: auto-switch to My Agents tab after creation and fix intent dialog reopen#3216
rrbanda wants to merge 1 commit into
redhat-developer:mainfrom
rrbanda:pr/post-create-redirect

Conversation

@rrbanda
Copy link
Copy Markdown
Contributor

@rrbanda rrbanda commented May 21, 2026

Summary

  • After creating an agent, the Marketplace now auto-switches to the "My Agents" tab so the user immediately sees their new draft agent
  • Fixes the intent dialog reopening after image-deploy: the wizard's onClose (handleWizardBack) was triggering setIntentDialogOpen(true) even after onCreated had already closed the wizard. A ref now suppresses the reopen.
  • Adds switchToMyAgentsKey prop to MarketplacePage to allow external tab control

Part of Epic #3208

Test plan

  • Create an agent via image deploy → wizard closes, marketplace switches to My Agents tab
  • Create an agent via source deploy → same behavior after build completes
  • Intent dialog does NOT reopen after creation completes
  • Clicking "Back" in wizard (without creating) still returns to intent dialog

…g reopen

After creating an agent, the user was left on the Explore tab where
drafts are invisible. Now the Marketplace automatically switches to the
My Agents tab so the user sees their new draft immediately.

Also fixes the intent dialog reopening after image-deploy: the wizard's
onClose (handleWizardBack) was triggering setIntentDialogOpen(true) even
after onCreated had already closed the wizard. A ref now suppresses the
reopen when an agent was just created.

Part of Epic redhat-developer#3208
@rrbanda rrbanda requested review from a team and pkliczewski as code owners May 21, 2026 15:52
@rhdh-gh-app
Copy link
Copy Markdown

rhdh-gh-app Bot commented May 21, 2026

Missing Changesets

The following package(s) are changed by this PR but do not have a changeset:

  • @red-hat-developer-hub/backstage-plugin-augment

See CONTRIBUTING.md for more information about how to add changesets.

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-augment workspaces/augment/plugins/augment none v0.1.0

@sonarqubecloud
Copy link
Copy Markdown

@codecov
Copy link
Copy Markdown

codecov Bot commented May 21, 2026

Codecov Report

❌ Patch coverage is 0% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 53.41%. Comparing base (7d93262) to head (e8a725f).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3216      +/-   ##
==========================================
- Coverage   53.41%   53.41%   -0.01%     
==========================================
  Files        2373     2373              
  Lines       84895    84905      +10     
  Branches    23568    23570       +2     
==========================================
  Hits        45349    45349              
- Misses      39200    39210      +10     
  Partials      346      346              
Flag Coverage Δ *Carryforward flag
adoption-insights 83.58% <ø> (ø) Carriedforward from 7d93262
ai-integrations 70.03% <ø> (ø) Carriedforward from 7d93262
app-defaults 69.60% <ø> (ø) Carriedforward from 7d93262
augment 46.66% <0.00%> (-0.02%) ⬇️
bulk-import 72.86% <ø> (ø) Carriedforward from 7d93262
cost-management 16.49% <ø> (ø) Carriedforward from 7d93262
dcm 32.85% <ø> (ø) Carriedforward from 7d93262
extensions 61.79% <ø> (ø) Carriedforward from 7d93262
global-floating-action-button 74.30% <ø> (ø) Carriedforward from 7d93262
global-header 61.68% <ø> (ø) Carriedforward from 7d93262
homepage 50.92% <ø> (ø) Carriedforward from 7d93262
konflux 91.01% <ø> (ø) Carriedforward from 7d93262
lightspeed 68.33% <ø> (ø) Carriedforward from 7d93262
mcp-integrations 81.59% <ø> (ø) Carriedforward from 7d93262
orchestrator 36.36% <ø> (ø) Carriedforward from 7d93262
quickstart 62.88% <ø> (ø) Carriedforward from 7d93262
sandbox 79.49% <ø> (ø) Carriedforward from 7d93262
scorecard 83.84% <ø> (ø) Carriedforward from 7d93262
theme 64.54% <ø> (ø) Carriedforward from 7d93262
translations 8.49% <ø> (ø) Carriedforward from 7d93262
x2a 78.59% <ø> (ø) Carriedforward from 7d93262

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


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7d93262...e8a725f. 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.

@rrbanda
Copy link
Copy Markdown
Contributor Author

rrbanda commented May 21, 2026

Closing: superseded by Phase 2 (#3223) which consolidates post-create redirect, dialog fix, and role-aware buttons.

@rrbanda rrbanda closed this May 21, 2026
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.

1 participant