Skip to content

Support editing zone record TTL field#12

Merged
mrmelon54 merged 3 commits into
mainfrom
feature/edit-zone-record-ttl
Jan 23, 2026
Merged

Support editing zone record TTL field#12
mrmelon54 merged 3 commits into
mainfrom
feature/edit-zone-record-ttl

Conversation

@mrmelon54

Copy link
Copy Markdown
Member

Resolves #5

@mrmelon54 mrmelon54 added enhancement New feature or request Domains This issue concerns domains. labels Jan 21, 2026
@mrmelon54 mrmelon54 self-assigned this Jan 21, 2026
@mrmelon54 mrmelon54 requested a review from Captain-ALM January 21, 2026 09:15

@Captain-ALM Captain-ALM left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM Though it seems to be missing editing the default.

Comment thread src/components/TTLEditor.svelte Outdated
@mrmelon54

Copy link
Copy Markdown
Member Author

The default is set on the zone SOA, currently the code to modify those fields is missing server side. The default TTL does exist in the zone table and is used as the default when a record is set to TTL default.

@Captain-ALM

Copy link
Copy Markdown
Member

Any way of outputting the default vault to the interface?

@mrmelon54

mrmelon54 commented Jan 23, 2026

Copy link
Copy Markdown
Member Author

Yeah there is an API for SOA details already. I need to add SOA viewing and editing to the admin panel though.

@mrmelon54 mrmelon54 requested a review from Captain-ALM January 23, 2026 13:33

@Captain-ALM Captain-ALM left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@mrmelon54 mrmelon54 merged commit 9f92422 into main Jan 23, 2026
1 check passed
@mrmelon54 mrmelon54 deleted the feature/edit-zone-record-ttl branch January 23, 2026 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Domains This issue concerns domains. enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TTL Editing support

2 participants