Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

"Mark as read" in the conversation list does not remove the notification

Aperta Adatta ai principianti
  • Qualcosa non va?
#929 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
83/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
android, kotlin
Ambito
mobile

Direzione di ricerca

Inizia da app/src/main/kotlin/org/fossify/messages/adapters/ConversationsAdapter.kt, in corrispondenza di ConversationsAdapter.markAsRead(), e confrontane il comportamento con quello dell’archiviazione/eliminazione e di MarkAsReadReceiver. Riproduci il problema contrassegnando come letta una conversazione dall’elenco mentre è presente la relativa notifica SMS. Il lavoro è completato quando questa azione rimuove anche la notifica, oltre a contrassegnare la conversazione come letta.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

bug needs triage
Checklist
  • I can reproduce the bug with the latest version given here.
  • I made sure that there are no existing issues - open or closed - to which I could contribute my information.
  • I made sure that there are no existing discussions - open or closed - to which I could contribute my information.
  • I have read the FAQs inside the app (Menu -> About -> FAQs) and my problem isn't listed.
  • I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
  • This issue contains only one bug.
  • I have read and understood the contribution guidelines.
Affected app version

1.10.1

Affected Android/Custom ROM version

Android 16 (One UI 8.5)

Affected device model

Samsung Galaxy A54 (SM-A546E), also reproduced on the Android Studio emulator (API 36)

How did you install the app?

No response

Steps to reproduce the bug
  1. Receive an SMS while the app is closed (a notification is shown).
  2. Open the app, long-press that conversation, open the menu and choose Mark as read.
Expected behavior

The notification of that conversation is removed, like when you open the conversation or press "Mark as read" in the notification.

Actual behavior

The conversation is marked as read, but its notification stays. On Samsung the launcher badge counts notifications, so the icon still shows 1 unread.

Screenshots/Screen recordings

No response

Additional information

ConversationsAdapter.markAsRead() does not cancel the notification, while archive, delete and MarkAsReadReceiver do:
https://github.com/FossifyOrg/Messages/blob/cf045ab5bdd36814be2b3d287cc294ce3f2b3fc7/app/src/main/kotlin/org/fossify/messages/adapters/ConversationsAdapter.kt#L239-L253

Suggested fix:

conversationsMarkedAsRead.filter { conversation -> !conversation.read }.forEach {
    activity.markThreadMessagesRead(it.threadId)
    activity.notificationManager.cancel(it.threadId.hashCode())
}

Optional extra safety (what I did in my fork): when the main screen opens, cancel the message notifications (shortcutId = thread id, not bubbles) of threads that have no unread SMS/MMS in the provider. This also cleans notifications left by other paths, for example a thread read by another app.

Found and tested in a personal fork built from 1.10.1 (only unrelated features added). The code involved is unchanged on main (cf045ab). Fix in the fork, for reference: https://github.com/fatehiman/peppa-ai-messages/commit/44d2d3876

Lingua principale
Kotlin
Stelle
1.6k
Fork
187
Merge medio
1g 16h
PR unite (30g)
20

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di FossifyOrg/Messages

Tutte le issue di FossifyOrg/Messages

Issue simili

Altre issue su Kotlin

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.