Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactor Domain/Zone Code #1601

Open
AgentTNT opened this issue Jun 25, 2023 · 1 comment
Open

Refactor Domain/Zone Code #1601

AgentTNT opened this issue Jun 25, 2023 · 1 comment
Assignees
@AgentTNT
Labels
mod / accepted This request has been accepted mod / change-request Used by internal developers to indicate a change-request. mod / reviewed This request has been reviewed
Milestone
v0.4.3

Comments

@AgentTNT
Copy link
Collaborator

Proposed Changes

Decoupling the domain/zone pages and routes will ease the burden for future coders on current code base.

Currently functions were moved into an AJAX call on page load, and then the rendering will complete on the client side after the JSON payload is acquired. Moving forward with server side CSRF should also help us here. This will also allow the app the be broken out to a frontend and backend separate deployments like a microservice if we choose to; it would be up to the individual deploying the software.

Justification

Please see above
@AgentTNT AgentTNT added the mod / change-request Used by internal developers to indicate a change-request. label Jun 25, 2023
@AgentTNT AgentTNT mentioned this issue Jun 25, 2023
Closed
@AzorianMatt AzorianMatt added mod / accepted This request has been accepted mod / reviewed This request has been reviewed labels Jun 25, 2023
@AzorianMatt AzorianMatt linked a pull request Jun 25, 2023 that will close this issue
(WIP) 1601 - Refactor Domain routes #1602
Closed
@AzorianMatt AzorianMatt added this to the v0.4.3 milestone Jan 31, 2024
@marcgweg
Copy link

Could you please give a status update on the v0.4.3 Milestone?

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

@AgentTNT AgentTNT

Labels
mod / accepted This request has been accepted mod / change-request Used by internal developers to indicate a change-request. mod / reviewed This request has been reviewed
Projects
None yet
Milestone
v0.4.3
Development

Successfully merging a pull request may close this issue.

(WIP) 1601 - Refactor Domain routes PowerDNS-Admin/PowerDNS-Admin
3 participants
@marcgweg @AgentTNT @AzorianMatt