rouggy/OpsLog
0
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 67 Wiki Activity

fix(qsl): clearer QRZ download summary (confirmed vs total)

Browse Source 
QRZ hands back the whole logbook; a confirmation download only acts on the
QSOs QRZ marks confirmed. The old "(of 7950 returned)" read as if QRZ had
returned only a fraction of a 26k-QSO logbook. Now it states "N of M are
confirmed — the rest aren't confirmed yet", so nothing looks lost.
This commit is contained in:
rouggy
2026-08-04 19:35:47 +02:00
parent dfe3afbf2d
commit d93c40ebb3
2 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app.go
+5 -1
View File
@@ -10579,7 +10579,11 @@ func (a *App) runDownloadConfirmations(ctx context.Context, svc extsvc.Service,
}
sort.Strings(keys)
emit(fmt.Sprintf("Parsed %d record(s). Fields seen: %s", parsed, strings.Join(keys, ", ")))
emit(fmt.Sprintf("Confirmed %d, added %d (of %d returned)", matched, added, total))
// Spell out the funnel: QRZ hands back the WHOLE logbook, but a confirmation
// download only acts on the ones QRZ marks confirmed — the "26165 but only
// 7950?" question. Nothing is lost; the rest simply aren't confirmed yet.
emit(fmt.Sprintf("%d of your %d QRZ QSOs are confirmed by the other station — the rest aren't confirmed yet, so there's nothing to import for them.", total, parsed))
emit(fmt.Sprintf("Of those %d confirmed: %d newly marked, %d added.", total, matched, added))
if unconfirmed > 0 {
// Worth its own line and not a footnote: these QSOs were showing as
// confirmed a moment ago, and an award total may move because of it.