Skip to content

Add Machine Page - #98

Open
TheArchons wants to merge 9 commits into
mainfrom
web-interface-machines
Open

Add Machine Page#98
TheArchons wants to merge 9 commits into
mainfrom
web-interface-machines

Conversation

@TheArchons

Copy link
Copy Markdown
Collaborator

No description provided.

CopilotAI review requested due to automatic review settings April 17, 2026 14:11

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a new Machines page and refactors shared “usage chart”/data types to support machine + job resource history displays in the web UI.

Changes:

  • Introduces Machine/Job TypeScript types (including UsageData) and updates the Jobs route to use them.
  • Implements a /machines route that renders machine cards with job lists and usage charts (sample-data loader for now).
  • Refactors Chart to accept an array of UsageData and manage component pagination internally; adds date-fns for date formatting.

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 8 comments.

Show a summary per file
FileDescription
web-interface/src/types/machine.tsAdds machine type definitions including machine-level usageHistory and job list.
web-interface/src/types/job.tsAdds job type definitions and centralizes UsageData for charts.
web-interface/src/routes/machines.tsxImplements Machines page UI and sample loader; displays machine/job info and charts.
web-interface/src/routes/jobs.tsxConverts Jobs page to loader-driven sample data using new types; uses date-fns for formatting; updates chart usage.
web-interface/src/components/Chart.tsxChanges chart API to accept UsageData[] and adds internal pagination state/controls.
web-interface/src/components/Buttons.tsxMinor button class adjustment (adds pointer cursor).
web-interface/package.jsonAdds date-fns dependency.
web-interface/bun.lockLocks date-fns dependency resolution.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment threadweb-interface/src/routes/jobs.tsx
Comment threadweb-interface/src/routes/jobs.tsx
Comment threadweb-interface/src/components/Chart.tsx
Comment threadweb-interface/src/components/Chart.tsx
Comment threadweb-interface/src/routes/machines.tsx
Comment threadweb-interface/src/routes/machines.tsx Outdated
Comment threadweb-interface/src/routes/machines.tsx Outdated
Comment threadweb-interface/src/routes/machines.tsx Outdated
@TheArchons
TheArchons requested a review from CopilotApril 17, 2026 14:37
@TheArchonsTheArchons self-assigned this Apr 17, 2026
@TheArchonsTheArchons linked an issue Apr 17, 2026 that may be closed by this pull request

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 5 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment threadweb-interface/src/routes/machines.tsx
Comment threadweb-interface/src/types/machine.ts
Comment threadweb-interface/src/types/job.ts
Comment threadweb-interface/src/components/Chart.tsx
Comment threadweb-interface/src/components/Buttons.tsx
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Interface] Machine Registry

2 participants

@TheArchons