Ctrlpanel-gg / panel

CtrlPanel offers an easy-to-use and free billing solution for all starting and experienced hosting providers that seamlessly integrates with the Pterodactyl panel.
https://ctrlpanel.gg/
MIT License
408 stars 146 forks source link

Move transferusers #976

Closed jameskitt616 closed 3 months ago

jameskitt616 commented 3 months ago

Move transferusers file for #952 also Updated docs see: https://github.com/Ctrlpanel-gg/docs/pull/117

IceToast commented 3 months ago

It is not part of the App (which is the laravel app) imo. Actually the transfer users script should ultimately not exist and be a laravel command (app/kernel/commands)

jameskitt616 commented 3 months ago

So delete the script and only rely on this? https://market.ctrlpanel.gg/resources/resource/35-transfer-users-from-ptero-to-cpgg/

IceToast commented 3 months ago

Both options are not ideal

jameskitt616 commented 3 months ago

I do agree, a laravel command would make the most sense. No clue how to write one tho.

jameskitt616 commented 3 months ago

So what happens next? Do i close the PR and trash it? & Remove the Docs PR.

S0ly commented 3 months ago

@IceToast

IceToast commented 3 months ago

I would not move it. I would keep it as is (a temporary solution) that gets replaced with a laravel command later on