Skip to content

refactor(FR-2494): improve auto scaling rule v2 UI — step size sign, condition tag, preset searchable#6748

Merged
graphite-app[bot] merged 1 commit intomainfrom
04-16-refactor_fr-2494_improve_auto_scaling_rule_v2_ui_step_size_sign_condition_tag_preset_searchable
Apr 16, 2026
Merged

refactor(FR-2494): improve auto scaling rule v2 UI — step size sign, condition tag, preset searchable#6748
graphite-app[bot] merged 1 commit intomainfrom
04-16-refactor_fr-2494_improve_auto_scaling_rule_v2_ui_step_size_sign_condition_tag_preset_searchable

Conversation

@agatha197
Copy link
Copy Markdown
Contributor

resolves #NNN (FR-MMM)

Checklist: (if applicable)

  • Documentation
  • Minium required manager version
  • Specific setting for review (eg., KB link, endpoint or how to setup)
  • Minimum requirements to check during review
  • Test case(s) to demonstrate the difference of before/after

Copy link
Copy Markdown
Contributor Author

agatha197 commented Apr 16, 2026


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

github-actions Bot commented Apr 16, 2026

Coverage report for ./react

St.
Category Percentage Covered / Total
🔴 Statements
8.68% (-0% 🔻)
1738/20017
🔴 Branches
7.89% (-0.01% 🔻)
1104/13985
🔴 Functions 5.21% 283/5430
🔴 Lines
8.38% (-0% 🔻)
1630/19440

Test suite run success

847 tests passing in 38 suites.

Report generated by 🧪jest coverage report action from 439bef6

@agatha197 agatha197 marked this pull request as ready for review April 16, 2026 03:15
Copilot AI review requested due to automatic review settings April 16, 2026 03:15
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Refines the Auto Scaling Rule v2 UI to make Prometheus-based rules clearer and easier to use by improving condition labeling, step-size sign display, and preset selection UX.

Changes:

  • Show Prometheus preset name in the condition tag (instead of raw metric name) and remove the separate “Prometheus Preset” column.
  • Update Step Size display to use + / − / ± based on threshold configuration and show MIN/MAX replicas accordingly.
  • Make Prometheus preset preview show a small spinner while loading and enable searchable preset selection.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
react/src/components/AutoScalingRuleList.tsx Updates condition rendering to use preset names and adjusts step-size / replica rendering logic.
react/src/components/AutoScalingRuleEditorModal.tsx Improves Prometheus preset selection (search) and preview loading UX; tweaks label text.

Comment thread react/src/components/AutoScalingRuleEditorModal.tsx Outdated
Comment thread react/src/components/AutoScalingRuleList.tsx
Comment thread react/src/components/AutoScalingRuleEditorModal.tsx
@agatha197 agatha197 force-pushed the 04-16-refactor_fr-2494_improve_auto_scaling_rule_v2_ui_step_size_sign_condition_tag_preset_searchable branch from 8395e2c to 371c91b Compare April 16, 2026 04:51
@agatha197 agatha197 force-pushed the 04-16-fix_fr-2494_fix_time_window_ko_translation_metricname_reset_on_source_change_and_prometheus_preview_restore branch from d0d953e to fdabd22 Compare April 16, 2026 04:51
@agatha197 agatha197 force-pushed the 04-16-refactor_fr-2494_improve_auto_scaling_rule_v2_ui_step_size_sign_condition_tag_preset_searchable branch from 371c91b to d35f12e Compare April 16, 2026 04:55
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Apr 16, 2026

Merge activity

…condition tag, preset searchable (#6748)

resolves #NNN (FR-MMM)
<!-- replace NNN, MMM with the GitHub issue number and the corresponding Jira issue number. -->

<!--
Please precisely, concisely, and concretely describe what this PR changes, the rationale behind codes,
and how it affects the users and other developers.
-->

**Checklist:** (if applicable)

- [ ] Documentation
- [ ] Minium required manager version
- [ ] Specific setting for review (eg., KB link, endpoint or how to setup)
- [ ] Minimum requirements to check during review
- [ ] Test case(s) to demonstrate the difference of before/after
@graphite-app graphite-app Bot force-pushed the 04-16-fix_fr-2494_fix_time_window_ko_translation_metricname_reset_on_source_change_and_prometheus_preview_restore branch from fdabd22 to 0bf260d Compare April 16, 2026 06:47
@graphite-app graphite-app Bot force-pushed the 04-16-refactor_fr-2494_improve_auto_scaling_rule_v2_ui_step_size_sign_condition_tag_preset_searchable branch from d35f12e to 439bef6 Compare April 16, 2026 06:47
Base automatically changed from 04-16-fix_fr-2494_fix_time_window_ko_translation_metricname_reset_on_source_change_and_prometheus_preview_restore to main April 16, 2026 06:56
@graphite-app graphite-app Bot merged commit 439bef6 into main Apr 16, 2026
8 checks passed
@graphite-app graphite-app Bot deleted the 04-16-refactor_fr-2494_improve_auto_scaling_rule_v2_ui_step_size_sign_condition_tag_preset_searchable branch April 16, 2026 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L 100~500 LoC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants