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

Regression: comments on a card are not being synced properly, with consequences for Client Push #6233

Open
LostinSpacetime opened this issue Aug 21, 2024 · 1 comment

Comments

@LostinSpacetime
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.

Describe the bug
When I add a new comment on a card, the counter on the card is not being updated. In combination with the Client Push app, this results in comments not being updated on other devices until a page reload. Interestingly, if the commenter reloads the page, the content (including comments and the counter on the card) gets updated on all devices.

This is a regression which came with some recent update of NC or Deck.

To Reproduce
Steps to reproduce the behavior:

  1. Go to a deck
  2. Click on a card
  3. Add a comment to the card

Expected behavior
The counter of comments on the card should change (+1).

Actual behavior
The counter doesn't change until a page reload.

Client details:

  • OS: Fedora 40
  • Browser Chromium and Firefox
  • Device: desktop, laptop
Server details

Operating system:
Ubuntu 22.04.4

Web server:
Apache

Database:
MariaDB 10.6.18

PHP version:
8.2.22

Nextcloud version:
29.0.5

Where did you install Nextcloud from:

Signing status:

No errors have been found.

List of activated apps:

  • activity: 2.21.1
  • admin_audit: 1.19.0
  • bruteforcesettings: 2.9.0
  • calendar: 4.7.16
  • cloud_federation_api: 1.12.0
  • comments: 1.19.0
  • contacts: 6.0.0
  • contactsinteraction: 1.10.0
  • dav: 1.30.1
  • deck: 1.13.1
  • drop_account: 2.6.0
  • federatedfilesharing: 1.19.0
  • federation: 1.19.0
  • files: 2.1.1
  • files_downloadlimit: 2.0.0
  • files_pdfviewer: 2.10.0
  • files_reminders: 1.2.0
  • files_sharing: 1.21.0
  • files_trashbin: 1.19.0
  • files_versions: 1.22.0
  • firstrunwizard: 2.18.0
  • group_everyone: 0.1.16
  • groupfolders: 17.0.2
  • logreader: 2.14.0
  • lookup_server_connector: 1.17.0
  • mail: 3.7.7
  • nextcloud_announcements: 1.18.0
  • notes: 4.10.1
  • notifications: 2.17.0
  • notify_push: 0.7.0
  • oauth2: 1.17.0
  • onlyoffice: 9.3.0
  • password_policy: 1.19.0
  • photos: 2.5.0
  • polls: 7.2.0
  • privacy: 1.13.0
  • provisioning_api: 1.19.0
  • qownnotesapi: 24.4.0
  • quota_warning: 1.19.0
  • recommendations: 2.1.0
  • registration: 2.4.0
  • related_resources: 1.4.0
  • serverinfo: 1.19.0
  • settings: 1.12.0
  • sharebymail: 1.19.0
  • side_menu: 3.13.1
  • support: 1.12.0
  • survey_client: 1.17.0
  • suspicious_login: 7.0.0
  • systemtags: 1.19.0
  • tasks: 0.16.1
  • text: 3.10.1
  • theming: 2.4.0
  • twofactor_backupcodes: 1.18.0
  • updatenotification: 1.19.1
  • user_status: 1.9.0
  • viewer: 2.3.0
  • weather_status: 1.9.0
  • workflowengine: 2.11.0

Nextcloud configuration:

{
"system": {
"passwordsalt": "REMOVED SENSITIVE VALUE",
"secret": "REMOVED SENSITIVE VALUE",
"trusted_domains": [
"REMOVED SENSITIVE VALUE",
"localhost",
"REMOVED SENSITIVE VALUE"
],
"datadirectory": "REMOVED SENSITIVE VALUE",
"dbtype": "mysql",
"version": "29.0.5.1",
"overwrite.cli.url": "https://cloud.schloss-schule.wien/",
"dbname": "REMOVED SENSITIVE VALUE",
"dbhost": "REMOVED SENSITIVE VALUE",
"dbport": "",
"dbtableprefix": "oc_",
"dbuser": "REMOVED SENSITIVE VALUE",
"dbpassword": "REMOVED SENSITIVE VALUE",
"installed": true,
"instanceid": "REMOVED SENSITIVE VALUE",
"mail_smtpmode": "smtp",
"mail_smtpsecure": "tls",
"mail_sendmailmode": "smtp",
"mail_from_address": "REMOVED SENSITIVE VALUE",
"mail_domain": "REMOVED SENSITIVE VALUE",
"mail_smtphost": "REMOVED SENSITIVE VALUE",
"mail_smtpauthtype": "LOGIN",
"simpleSignUpLink.shown": false,
"maintenance": false,
"maintenance_window_start": 1,
"mysql.utf8mb4": true,
"default_language": "de_DE",
"default_locale": "de_AT",
"default_phone_region": "AT",
"app_install_overwrite": [
"auto_mail_accounts",
"whiteboard",
"discoursesso"
],
"theme": "",
"loglevel": 2,
"memcache.local": "\OC\Memcache\Redis",
"memcache.locking": "\OC\Memcache\Redis",
"memcache.distributed": "\OC\Memcache\Redis",
"redis": {
"host": "REMOVED SENSITIVE VALUE",
"port": 6379
},
"updater.release.channel": "stable",
"ldapIgnoreNamingRules": false,
"overwritehost": "cloud.schloss-schule.wien",
"auth.bruteforce.protection.enabled": true,
"app.mail.verify-tls-peer": false,
"encryption.legacy_format_support": false,
"encryption.key_storage_migrated": false,
"mail_smtpauth": 1,
"mail_smtpname": "REMOVED SENSITIVE VALUE",
"mail_smtppassword": "REMOVED SENSITIVE VALUE",
"allow_local_remote_servers": true,
"trusted_proxies": "REMOVED SENSITIVE VALUE"
}
}

Are you using an external user-backend, if yes which one:

No

Nextcloud log (data/nextcloud.log)

nextcloud.log

Browser log

viewer-main.mjs?v=4ce083d9-28:18309 [WARN] viewer: Some mimes were ignored because they are not enabled in the server previews config Object
log @ viewer-main.mjs?v=4ce083d9-28:18309
listener.js:184 [Deprecation] Listener added for a 'DOMNodeRemoved' mutation event. This event type is deprecated, and will be removed from this browser VERY soon. Usage of this event listener will cause performance issues today, and represents a large risk of imminent site breakage. Consider using MutationObserver instead. See https://chromestatus.com/feature/5083947249172480 for more information.
(anonymous) @ listener.js:184
session-heartbeat.js:103 session heartbeat polling started

@LostinSpacetime LostinSpacetime changed the title comments on a card are not being synced properly Regression: comments on a card are not being synced properly, with consequences for Client Push Aug 22, 2024
@LostinSpacetime
Copy link
Author

I tested some more, and it's not necessary to reload the page. It's enough to switch to another tab and back to the NC-tab to update the comments counter on the deck and (I assume) consequently the comments on any other machines when using the Client Push app. This behavior is consistent on different Browsers (Firefox, Chromium) and OSs (Fedora 40, Windows 11).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant