Skip to content

Feat jinhua clusters blacklist 260224#671

Draft
Yang1998-jiale wants to merge 9 commits intomainfrom
feat-jinhua-clustersBlacklist-260224
Draft

Feat jinhua clusters blacklist 260224#671
Yang1998-jiale wants to merge 9 commits intomainfrom
feat-jinhua-clustersBlacklist-260224

Conversation

@Yang1998-jiale
Copy link
Collaborator

Description

Related Issue

Motivation and Context

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation Update (if none of the other choices apply)

Checklist

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.

杨家乐 added 9 commits February 4, 2026 14:32
Signed-off-by: 杨家乐 <yjl516914@digital-engine.com>
Signed-off-by: 杨家乐 <yjl516914@digital-engine.com>
Signed-off-by: 杨家乐 <yjl516914@digital-engine.com>
Signed-off-by: 杨家乐 <yjl516914@digital-engine.com>
Allow viewing blacklist details dialog even with single item
- Restructure blacklist UI with separate applications, urls, tags, priorities fields
- Fix priorities type conversion from string to int for backend compatibility
- Add duplicate combination validation for service/task type/feature
- Improve data preservation when updating blacklist fields
- Sync blacklist updates to preheats/new.tsx
Signed-off-by: 杨家乐 <yjl516914@digital-engine.com>
Signed-off-by: 杨家乐 <yjl516914@digital-engine.com>
@codecov
Copy link

codecov bot commented Mar 9, 2026

Codecov Report

❌ Patch coverage is 75.96439% with 81 lines in your changes missing coverage. Please review.
✅ Project coverage is 95.26%. Comparing base (d7b755d) to head (c2c7933).

Files with missing lines Patch % Lines
src/components/clusters/blacklist.tsx 65.17% 75 Missing and 3 partials ⚠️
src/components/job/preheats/new.tsx 94.44% 2 Missing ⚠️
src/components/clusters/information.tsx 98.27% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #671      +/-   ##
==========================================
- Coverage   96.45%   95.26%   -1.19%     
==========================================
  Files          52       53       +1     
  Lines        5783     6064     +281     
  Branches     1824     1982     +158     
==========================================
+ Hits         5578     5777     +199     
- Misses        190      269      +79     
- Partials       15       18       +3     
Files with missing lines Coverage Δ
src/components/clusters/edit.tsx 99.55% <100.00%> (+<0.01%) ⬆️
src/components/clusters/new.tsx 100.00% <100.00%> (ø)
src/lib/api.ts 91.88% <ø> (+0.36%) ⬆️
src/components/clusters/information.tsx 95.45% <98.27%> (-1.97%) ⬇️
src/components/job/preheats/new.tsx 97.07% <94.44%> (+<0.01%) ⬆️
src/components/clusters/blacklist.tsx 65.17% <65.17%> (ø)

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

1 participant