Skip to content

Conversation

@david-crespo
Copy link
Collaborator

@david-crespo david-crespo commented Oct 14, 2025

Closes #2903
Closes #2905

We have read-only stuff and setting target release. I had trouble with upload, so it is in the first commit but it is removed in a subsequent commit.

Outstanding questions

  • How to display "suspended" (Just saying True/False is kinda ugly)
    • Went with Yes/No
  • Should we disable the set target release button in the UI in more conditions?
image

@david-crespo david-crespo added this to the 17 milestone Oct 14, 2025
@vercel
Copy link

vercel bot commented Oct 14, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
console Ready Ready Preview Oct 15, 2025 2:25am

"sourceMap": true,
"strict": true,
"target": "es2022",
"target": "es2023",
Copy link
Collaborator Author

@david-crespo david-crespo Oct 14, 2025

Choose a reason for hiding this comment

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

Needed to do this in order to get roundingMode on the type of the args to Intl.NumberFormat. es2023 is currently the default in TS anyway, so we should be on it anyway.

https://www.typescriptlang.org/tsconfig/#target

systemIpPoolsDocs: 'https://docs.oxide.computer/guides/operator/ip-pool-management',
systemMetricsDocs: 'https://docs.oxide.computer/guides/operator/system-metrics',
systemSiloDocs: 'https://docs.oxide.computer/guides/operator/silo-management',
systemUpdateDocs: 'https://docs.oxide.computer/guides/operator/system-update',
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@david-crespo david-crespo enabled auto-merge (squash) October 15, 2025 02:28
@david-crespo david-crespo merged commit 74c2060 into main Oct 15, 2025
7 checks passed
@david-crespo david-crespo deleted the system-update-ui branch October 15, 2025 02:31
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.

Update: set target release Update: status and repo list page

2 participants