Skip to content

Feat/Node cloning ability#5562

Closed
KroZen-Dev wants to merge 9 commits intopterodactyl:1.0-developfrom
KroZen-Dev:feat/node-cloning-ability
Closed

Feat/Node cloning ability#5562
KroZen-Dev wants to merge 9 commits intopterodactyl:1.0-developfrom
KroZen-Dev:feat/node-cloning-ability

Conversation

@KroZen-Dev
Copy link

This pull request introduces a new feature that allows administrators to clone existing nodes through the admin interface. The main changes include backend logic for cloning nodes, a new form request for validation, updates to the controller, and enhancements to the UI to support the cloning workflow.

Node Cloning Feature:

  • Added a new NodeCloneService that handles the logic for cloning nodes, leveraging the existing NodeCreationService to create a new node based on the source node's attributes with user-provided overrides.
  • Introduced a NodeCloneRequest form request to validate input data when cloning a node, ensuring required fields and proper formats.
  • Updated the NodesController (now under Admin\Nodes namespace) to inject the clone service, add a clone method for handling clone requests, and register the new POST route for node cloning. [1] [2] [3] [4] [5]

User Interface Enhancements:

  • Modified the nodes index Blade view to add a "Clone" button for each node, which opens a modal form for cloning. The modal collects new node details and submits them to the clone endpoint. [1] [2]
  • Added JavaScript to pre-fill the clone modal with the selected node's name and set the form action dynamically.

Other UI Adjustments:

  • Improved the error icon and tooltip handling in the node ping section for better user feedback

🔗 Linked Issues: #3725

@DaneEveritt
Copy link
Member

The use of AI in this software requires explicit acknowledgement, and you are expected to have tested your work before opening a PR.

@DaneEveritt DaneEveritt closed this Feb 8, 2026
@KroZen-Dev
Copy link
Author

The use of AI in this software requires explicit acknowledgement, and you are expected to have tested your work before opening a PR.

You're very funny. This is a work in progress, not the final version 😂

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.

2 participants