Follow-up delta to merged PR #224: the details grid got the attachment size, but the summary pill still used humanSize(len(row.Content)) which reads 0 B for file pastes (text content is empty). Verified locally: image paste summary now shows the attachment size; text paste unchanged; raw view still serves the image.
Ref #221
Follow-up delta to merged PR #224: the details grid got the attachment size, but the summary pill still used humanSize(len(row.Content)) which reads 0 B for file pastes (text content is empty). Verified locally: image paste summary now shows the attachment size; text paste unchanged; raw view still serves the image.
The merged #224 fix covered the details grid but left the summary
pill using humanSize(len(row.Content)), which reads 0 for attachment
pastes (text content is empty). Use the attachment size there too.
fen
merged commit e589428315 into dev2026-09-10 19:33:51 +00:00
PR #228 head d18bbb7 is an ancestor of dev (merge commit e589428), CI test + docker both green on dev tip.
Live check on https://palette-dev.archfox.org: created a 4000-byte image paste via multipart; the paste page summary pill renders text · 3.9 KB (not 0 B), image renders via /f/ attachment URL, and /raw/w78rg2 serves the PNG (200, image/png, 4000 bytes).
The fix path (summary size from attachment.Size for file pastes) is exercised end to end on the deployed server; no deploy restart was needed (pod already serving dev-tip behavior).
Branch fix-221-r2: already auto-deleted by the merge (confirmed via GET /branches).
QA PASS (verified live on palette-dev):
- PR #228 head d18bbb7 is an ancestor of dev (merge commit e589428), CI test + docker both green on dev tip.
- Live check on https://palette-dev.archfox.org: created a 4000-byte image paste via multipart; the paste page summary pill renders `text · 3.9 KB` (not `0 B`), image renders via /f/ attachment URL, and /raw/w78rg2 serves the PNG (200, image/png, 4000 bytes).
- The fix path (summary size from attachment.Size for file pastes) is exercised end to end on the deployed server; no deploy restart was needed (pod already serving dev-tip behavior).
Branch fix-221-r2: already auto-deleted by the merge (confirmed via GET /branches).
fen
added spent time 15 minutes2026-09-10 19:36:33 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Ref #221
Follow-up delta to merged PR #224: the details grid got the attachment size, but the summary pill still used humanSize(len(row.Content)) which reads 0 B for file pastes (text content is empty). Verified locally: image paste summary now shows the attachment size; text paste unchanged; raw view still serves the image.
QA PASS (verified live on palette-dev):
d18bbb7is an ancestor of dev (merge commite589428), CI test + docker both green on dev tip.text · 3.9 KB(not0 B), image renders via /f/ attachment URL, and /raw/w78rg2 serves the PNG (200, image/png, 4000 bytes).Branch fix-221-r2: already auto-deleted by the merge (confirmed via GET /branches).
QA PASS (PR #228, verified live on palette-dev against dev merge
e589428):