astra/owpengram-server
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

added dry run for update button

Browse source
This commit is contained in:
onysd 2026-09-08 13:25:00 +03:00
parent 5c16371ae0
commit e240bfbbfa
3 changed files with 30 additions and 32 deletions
Download patch file Download diff file Expand all files Collapse all files

2
cmd/telesrv-admin/web/dist/assets/index-Cto7ZSSt.js → cmd/telesrv-admin/web/dist/assets/index-BdqgFh6n.js vendored
View file

File diff suppressed because one or more lines are too long

2
cmd/telesrv-admin/web/dist/index.html vendored
View file

@ -23,7 +23,7 @@
})(); })();
</script> </script>
<script type="module" crossorigin src="/assets/index-Cto7ZSSt.js"></script> <script type="module" crossorigin src="/assets/index-BdqgFh6n.js"></script>
<link rel="stylesheet" crossorigin href="/assets/index-pvHlBxfG.css"> <link rel="stylesheet" crossorigin href="/assets/index-pvHlBxfG.css">
</head> </head>
<body> <body>

56
cmd/telesrv-admin/web/src/pages/ServerSettingsPage.tsx
View file

@ -636,11 +636,13 @@ const LIVE_POLL_MS = 4000;
// UpdateButton is a two-state control: "Check updates" (a plain git fetch + // UpdateButton is a two-state control: "Check updates" (a plain git fetch +
// commit count, no side effects) until a check finds the branch behind its // commit count, no side effects) until a check finds the branch behind its
// upstream, at which point it becomes "Update (<N>)" -- a second click runs // upstream, at which point it becomes "Update (<N>)" -- an ActionButton
// the real git pull + rebuild + restart (via the existing update-server // running the real git pull + rebuild + restart through the same
// action, called directly with confirm:true since the check step already // reason/dry-run/confirm flow as Restart beside it. The check is not the
// serves as the "are you sure" gate). Auto-checks once on mount so the // gate: it only reports how far behind the branch is, and pulling somebody
// button reflects reality without an operator having to click twice. // else's commits and relaunching the deployment deserves the same recorded
// reason as every other action here. Auto-checks once on mount so the button
// reflects reality without an operator having to click twice.
function UpdateButton({ onUpdateStarted }: { onUpdateStarted: (overlayLabel: string) => void }) { function UpdateButton({ onUpdateStarted }: { onUpdateStarted: (overlayLabel: string) => void }) {
const [behind, setBehind] = useState<number | null>(null); const [behind, setBehind] = useState<number | null>(null);
const [message, setMessage] = useState(""); const [message, setMessage] = useState("");
@ -666,39 +668,35 @@ function UpdateButton({ onUpdateStarted }: { onUpdateStarted: (overlayLabel: str
useEffect(() => { void check(); }, [check]); useEffect(() => { void check(); }, [check]);
async function runUpdate() {
const commits = behind ?? 0; const commits = behind ?? 0;
if (!window.confirm(`Pull ${commits} commit(s), rebuild, and restart owpengram-server and the admin panel?`)) { const hasUpdates = commits > 0;
return;
}
setBusy(true);
setError("");
try {
const result = await api.action("/api/actions/update-server", { command_id: "", reason: "Update via Services tab", confirm: true });
if (result.error) {
setError(result.error);
setBusy(false);
return;
}
onUpdateStarted(`Pulled ${commits} commit${commits === 1 ? "" : "s"} -- rebuilding and restarting owpengram-server and the admin panel...`);
} catch (err) {
setError(errorMessage(err));
setBusy(false);
}
}
const hasUpdates = (behind ?? 0) > 0; if (hasUpdates) {
return (
<ActionButton
compact
tone="danger"
icon={<Download size={15} />}
label={`Update (${commits})`}
path="/api/actions/update-server"
payload={() => ({})}
onDone={() => {
onUpdateStarted(`Pulled ${commits} commit${commits === 1 ? "" : "s"} -- rebuilding and restarting owpengram-server and the admin panel...`);
}}
/>
);
}
return ( return (
<button <button
className={`btn compact-btn icon-text ${hasUpdates ? "danger" : ""}`} className="btn compact-btn icon-text"
type="button" type="button"
disabled={busy} disabled={busy}
title={error || message || undefined} title={error || message || undefined}
onClick={() => void (hasUpdates ? runUpdate() : check())} onClick={() => void check()}
> >
{busy ? <Loader2 className="spin" size={15} /> : hasUpdates ? <Download size={15} /> : <RefreshCw size={15} />} {busy ? <Loader2 className="spin" size={15} /> : <RefreshCw size={15} />}
{hasUpdates ? `Update (${behind})` : "Check updates"} {"Check updates"}
</button> </button>
); );
} }