Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incoming call is shown for old notifications #961

Closed
SystemKeeper opened this issue Dec 4, 2024 · 2 comments · Fixed by #967
Closed

Incoming call is shown for old notifications #961

SystemKeeper opened this issue Dec 4, 2024 · 2 comments · Fixed by #967
Assignees
Labels
1. to develop bug Something isn't working

Comments

@SystemKeeper
Copy link

How to use GitHub

  • Please use the 👍 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Steps to reproduce

  1. Have an old/missed call notification in your notifications
  2. Have enough "new" notifications, that the call notification is not received on the /notifications endpoint
  3. Make sure that the call notification gets back to the first 25 elements and therefore returned by the server (it's not happening all the time, not sure if we track the last notification id or the like)

Expected behaviour

No call notification should be triggered by an old call notification

Actual behaviour

Image

Image

Diagnosis and logs

Diagnosis report

Diagnosis report

Nextcloud Talk Desktop
Version v1.0.0
Built-in Talk version v20.1.0
Operating system Darwin 24.1.0 (Darwin Kernel Version 24.1.0: Thu Oct 10 21:03:15 PDT 2024; root:xnu-11215.41.3~2/RELEASE_ARM64_T6000)
Executable Path /Applications/Nextcloud Talk.app/Contents/MacOS/Nextcloud Talk
Connected to -
Nextcloud server version 31.0.0 dev
Nextcloud Talk version 21.0.0-dev.3
notifications app enabled ✅ yes
notify_push app enabled ✅ yes

Application config

{
  "theme": "default",
  "systemTitleBar": false,
  "monochromeTrayIcon": true,
  "zoomFactor": 1,
  "playSoundChat": "respect-dnd",
  "playSoundCall": "respect-dnd",
  "enableCallbox": "respect-dnd"
}
@SystemKeeper SystemKeeper added 0. Needs triage bug Something isn't working labels Dec 4, 2024
@nickvergessen
Copy link
Member

Web fix back then nextcloud/notifications#1943

@nickvergessen
Copy link
Member

@github-project-automation github-project-automation bot moved this to 🧭 Planning evaluation (don't pick) in 💬 Talk team Dec 9, 2024
@nickvergessen nickvergessen moved this from 🧭 Planning evaluation (don't pick) to 📄 To do (~10 entries) in 💬 Talk team Dec 9, 2024
@github-project-automation github-project-automation bot moved this from 📄 To do (~10 entries) to ☑️ Done in 💬 Talk team Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop bug Something isn't working
Projects
Status: ☑️ Done
Development

Successfully merging a pull request may close this issue.

3 participants