01a0adcb - Show inbox unread in the menu and home-screen badge - #255
Merged
Merged
Conversation
TaprootFreakAI
force-pushed
the
feat/01a0adcb-inbox-unread
branch
from
September 17, 2026 17:28
7e6de9d to
2d37eee
Compare
TaprootFreakAI
changed the title
TaprootFreakAI
force-pushed
the
feat/01a0adcb-inbox-unread
branch
from
September 17, 2026 19:58
2d37eee to
71ef0d4
Compare
Load conversation unread beside notifications, mark a thread read when it opens, and style unread inbox rows.
The badge is notifications plus inbox unread. Visiting /notifications does not clear remaining inbox unread from the icon.
Bump the badge epoch on logout, exclude the open thread from remaining inbox unread, and add SCREEN_VARIANTS for inbox-unread menu and list.
openConversation now returns Conversation with unread after the inbox-unread schema; the #240 thread fixture must set it.
Linux Chromium drifted 15 pixels on the mobile combos after the about-me photo screen landed on develop.
clearAuth bumps the badge epoch before zeroing. refreshUnreadAppBadge and the notifications inbox write skip a mismatched session. The e2e needle counts a notifications GET after the thread opens.
The qemu-linux refresh drifted 15 pixels from GitHub-hosted Chromium.
…base. Conversation rows keep lastSats and unread. Same-origin proxies keep POST invoice and POST read. Inbox list styling keeps unread inbound rows and gift-only last-sats previews.
TaprootFreakAI
force-pushed
the
feat/01a0adcb-inbox-unread
branch
from
September 17, 2026 21:42
3c1587d to
d78d65e
Compare
Opening a thread clears unread and refreshes the badge locally first. The mock conversations list now returns lastSats and unread together.
Unread inbound names are semibold with foreground last-text; read inbound last-text stays muted; outbound last-text stays the sent chip.
setUnreadAppBadge is not called from InboxLoader. unreadAppBadgeEpoch is also used by NotificationsLoader and refreshUnreadAppBadge. The conversations/[id] tree lists invoice then read.
The badge refresh after inbox mark-read reads the session token and GET /forum/notifications; handbook Used-by now names those callers.
Collaborator
Author
|
EN: DE: DetailsHead |
TaprootFreakAI
marked this pull request as ready for review
September 17, 2026 23:13
TaprootFreak
approved these changes
Sep 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
EN:
The signed-in Menu now shows an unread count on Messages, the same way Notifications already does.
Opening a thread marks it read; unread rows in the inbox list use a stronger name and last-text.
The home-screen PWA badge is notifications unread plus inbox unread; visiting
/notificationsdoes not clear remaining inbox from the icon.DE:
Das eingeloggte Menü zeigt beim Posteingang eine Ungelesen-Zahl, analog zu Benachrichtigungen.
Ein geöffneter Thread gilt als gelesen; ungelesene Zeilen in der Liste haben einen kräftigeren Namen und Last-Text.
Das Home-Screen-Badge ist Benachrichtigungen plus ungelesener Posteingang;
/notificationswischt den restlichen Posteingang nicht vom Icon.Details
nav.inboxUnreadplus a right-side count when inbox unread is greater than zeroGET /conversationsunread flags drive the count; missing API fields default to false/0/messages?c=POSTs/conversations/:id/readand clears that row locallyuseUnreadCountwaits for both fetches, then writes the sumNotificationsLoaderno longer forces badge 0; remaining inbox unread stays/messages?c=<id>in the service workerNeeds the matching api change for live unread data and conversation push payloads.