Skip to content

web: replace raw YAML editor for DNS handler configs with structured UI#2182

Merged
BeryJu merged 5 commits intomainfrom
claude/sharp-lehmann-78731d
May 3, 2026
Merged

web: replace raw YAML editor for DNS handler configs with structured UI#2182
BeryJu merged 5 commits intomainfrom
claude/sharp-lehmann-78731d

Conversation

@BeryJu
Copy link
Copy Markdown
Owner

@BeryJu BeryJu commented May 3, 2026

image

thanks mr claude

Replaces the CodeMirror YAML textarea in the DNS zone form with a new
`gravity-dns-handler-config-editor` Lit component that renders each
handler as a draggable card with type-specific form fields.

- Drag-and-drop reordering via HTML5 DnD with visual drop-target feedback
- Handler-specific fields: forward_ip/forward_blocky show resolver
  addresses, cache TTL, network protocol; forward_blocky also exposes
  allowlists and blocklists; memory/etcd require no configuration
- Add/remove handlers via a type-selector dropdown and per-card remove button
- PatternFly v4 CSS with full dark-mode support via CSS variables
- Integrates with existing form serialisation: reflected `name` attribute
  and `value` getter return the HandlerConfig[] array directly
- HorizontalFormElement updated to propagate `name` to the new element tag

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented May 3, 2026

Deploy Preview for beryju-gravity canceled.

Name Link
🔨 Latest commit 275dd88
🔍 Latest deploy log https://app.netlify.com/projects/beryju-gravity/deploys/69f785c5092bea00087367a6

@codecov
Copy link
Copy Markdown

codecov Bot commented May 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.41%. Comparing base (e344c2c) to head (275dd88).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2182      +/-   ##
==========================================
+ Coverage   69.39%   69.41%   +0.02%     
==========================================
  Files         138      138              
  Lines        7583     7583              
==========================================
+ Hits         5262     5264       +2     
+ Misses       1787     1786       -1     
+ Partials      534      533       -1     
Flag Coverage Δ
bench 12.11% <ø> (ø)
e2e 45.30% <ø> (ø)
unittest 55.40% <ø> (+0.23%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@BeryJu BeryJu merged commit 9718283 into main May 3, 2026
24 checks passed
@BeryJu BeryJu deleted the claude/sharp-lehmann-78731d branch May 3, 2026 17:41
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