Implement new single Server State/Status display #97

Closed
opened 2026-02-05 17:08:04 +03:00 by OVERLORD · 2 comments
Owner

Originally created by @ItsNeil17 on GitHub (Jun 10, 2024).

We currently have 2 separate enums for server state and server status.
Server state refers to our own state such as reinstall, transfer, etc.
Server status refers to the docker container's status such as running, stopped, etc.

If the current server state exists, then show it. If it's empty, then we should show the current container status.

This also includes the labelling in the Edit Server page.

Originally created by @ItsNeil17 on GitHub (Jun 10, 2024). We currently have 2 separate enums for server state and server status. Server state refers to our own state such as reinstall, transfer, etc. Server status refers to the docker container's status such as running, stopped, etc. If the current server state exists, then show it. If it's empty, then we should show the current container status. This also includes the labelling in the Edit Server page.
OVERLORD added the 🟢 easy label 2026-02-05 17:08:04 +03:00
Author
Owner

@iamkubi commented on GitHub (Jun 12, 2024):

I'm fairly certain this is intentional. You don't want to have to query the docker status of the server every time you want to render a server in the UI.

@iamkubi commented on GitHub (Jun 12, 2024): I'm fairly certain this is intentional. You don't want to have to query the docker status of the server every time you want to render a server in the UI.
Author
Owner

@iamkubi commented on GitHub (Jun 12, 2024):

Although the way the API represents the state could be improved, it is important for performance reasons not to combine these 2 fields.

@iamkubi commented on GitHub (Jun 12, 2024): Although the way the API represents the state could be improved, it is important for performance reasons not to combine these 2 fields.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel-pelican-dev#97