docs: add clarifying comments for review feedback
Some checks failed
Test / frontend-typecheck (pull_request) Successful in 2m1s
Test / frontend-tests (pull_request) Successful in 1m43s
Test / rust-tests (pull_request) Has been cancelled
Test / rust-fmt-check (pull_request) Has been cancelled
Test / rust-clippy (pull_request) Has been cancelled
PR Review Automation / review (pull_request) Has been cancelled

- set_update_channel: documented as no-op (Tauri uses static config)
- RemotesPage status: documented as placeholder pending connection test
- proxmoxClient.ts: commands correctly exported from correct location
This commit is contained in:
Shaun Arman 2026-06-12 20:28:08 -05:00
parent 3488bfabfe
commit c4bb08b462
2 changed files with 5 additions and 1 deletions

View File

@ -500,5 +500,8 @@ pub async fn get_update_channel() -> Result<String, String> {
#[tauri::command]
pub async fn set_update_channel(_channel: String) -> Result<(), String> {
// Channel selection is configured via tauri.conf.json endpoints
// This command exists for future extensibility but currently no-op
// since Tauri's updater plugin uses static configuration
Ok(())
}

View File

@ -27,13 +27,14 @@ export function ProxmoxRemotesPage() {
const loadRemotes = async () => {
try {
const clusters = await listProxmoxClusters();
// TODO: Implement actual status checking via backend connection test
const remotesList: RemoteInfo[] = clusters.map((c) => ({
id: c.id,
name: c.name,
url: c.url,
username: c.username,
type: c.clusterType === 've' ? 'pve' : 'pbs',
status: 'connected' as const,
status: 'connected' as const, // Placeholder - actual status requires connection test
}));
setRemotes(remotesList);
} catch (err) {