Skip to content

[VW-0] DeviceWithImage model, Devices Tables shows image status & DeviceDetails view#38

Merged
trummelhadron merged 7 commits into
mainfrom
device-details-and-pending-status
Jul 7, 2026
Merged

[VW-0] DeviceWithImage model, Devices Tables shows image status & DeviceDetails view#38
trummelhadron merged 7 commits into
mainfrom
device-details-and-pending-status

Conversation

@trummelhadron

@trummelhadron trummelhadron commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Includes:

  • Get Device API endpoints use DeviceWithImage to bundle the image model with the device
  • Start of a proper Device Details page
  • Device list and detail will show image Pending Upload bubbles linking to /images if still pending
  • Lots of CSS touchups (now that I have a little extra time)

@trummelhadron trummelhadron requested review from hartmans and timrcm June 25, 2026 19:49
Comment thread layout/python/models.py Outdated
@@ -1,5 +1,5 @@
from functools import wraps
from typing import Any, Callable, Literal, Optional, TypeVar
from typing import Any, Callable, Literal, Optional, TypeVar, Union

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This appears to be unused

@@ -0,0 +1,140 @@
import { useQuery } from "@tanstack/react-query";

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is pretty shiny!

  • Can we show something other than 'Uploaded' for containers?
  • Also, when I add a VM that does not have an image yet, it still shows 'uploaded'; might want to check if vm_image_id is still null.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks great now, thanks!

@timrcm timrcm self-requested a review July 7, 2026 15:29

@timrcm timrcm left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This all looks solid, thanks!

@@ -0,0 +1,140 @@
import { useQuery } from "@tanstack/react-query";

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks great now, thanks!

@trummelhadron trummelhadron merged commit 9f091a2 into main Jul 7, 2026
7 checks passed
@trummelhadron trummelhadron deleted the device-details-and-pending-status branch July 7, 2026 16:16
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