Skip to content

feat(FR-2762): add deployment preset detail button to VFolderDeployModal#7192

Closed
agatha197 wants to merge 1 commit intographite-base/7192from
04-30-feat_fr-2762_add_deployment_preset_detail_button_to_vfolderdeploymodal
Closed

feat(FR-2762): add deployment preset detail button to VFolderDeployModal#7192
agatha197 wants to merge 1 commit intographite-base/7192from
04-30-feat_fr-2762_add_deployment_preset_detail_button_to_vfolderdeploymodal

Conversation

@agatha197
Copy link
Copy Markdown
Contributor

@agatha197 agatha197 commented Apr 30, 2026

Resolves #7122 (FR-2762)

Summary

  • Add info button (InfoCircleOutlined) next to the preset BAISelect in VFolderDeployModal, following the VFolderSelect pattern (BAIFlex + Space.Compact)
  • Clicking the button opens DeploymentPresetDetailModal for the currently selected preset
  • Button is disabled when no preset is selected
  • Fix rowIdid field reference in ServiceLauncherPageContent deployment preset query (DeploymentRevisionPreset exposes id, not rowId)

Copy link
Copy Markdown
Contributor Author

agatha197 commented Apr 30, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • flow:merge-queue - adds this PR to the back of the merge queue
  • flow:hotfix - for urgent changes, fast-track this PR to the front of the merge queue

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions
Copy link
Copy Markdown
Contributor

Coverage report for ./react

St.
Category Percentage Covered / Total
🔴 Statements
7.97% (+0% 🔼)
1859/23315
🔴 Branches 7.17% 1187/16546
🔴 Functions
4.72% (+0.01% 🔼)
297/6292
🔴 Lines
7.75% (+0% 🔼)
1750/22582

Test suite run success

865 tests passing in 40 suites.

Report generated by 🧪jest coverage report action from 3f123ff

@agatha197 agatha197 changed the base branch from 04-28-feat_fr-2762_add_deployment_preset_detail_view_in_service_launcher to graphite-base/7192 April 30, 2026 15:17
@agatha197
Copy link
Copy Markdown
Contributor Author

Folded into #7127

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

Labels

size:M 30~100 LoC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant