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

[Bug]: Empty DND user statuses listed on dashboard #43089

Closed
4 of 8 tasks
ChristophWurst opened this issue Jan 24, 2024 · 3 comments · Fixed by #43464
Closed
4 of 8 tasks

[Bug]: Empty DND user statuses listed on dashboard #43089

ChristophWurst opened this issue Jan 24, 2024 · 3 comments · Fixed by #43464

Comments

@ChristophWurst
Copy link
Member

⚠️ This issue respects the following points: ⚠️

Bug description

The empty user statues ", seconds ago" are back.

This was fixed in #41512. https://github.com/nextcloud/server/pull/41714/files#r1404002684 reverted the fix.

Steps to reproduce

  1. Create two users
  2. Set availability for one. Make sure the user is unavailable at the time of testing
  3. Flush cron jobs
  4. Log in as the other user and look at the dashboard

Expected behavior

No recent user status

Installation method

None

Nextcloud Server version

28

Operating system

None

PHP engine version

None

Web server

None

Database engine version

None

Is this bug present after an update or on a fresh install?

None

Are you using the Nextcloud Server Encryption module?

None

What user-backends are you using?

  • Default user-backend (database)
  • LDAP/ Active Directory
  • SSO - SAML
  • Other

Configuration report

No response

List of activated Apps

No response

Nextcloud Signing status

No response

Nextcloud Logs

No response

Additional info

No response

@ChristophWurst
Copy link
Member Author

Bringing back the 0 timestamp in \OCA\UserStatus\Service\StatusService::setUserStatus seems to do the trick. Didn't test for any other side effects of this

@miaulalala
Copy link
Contributor

Bringing back the 0 timestamp in \OCA\UserStatus\Service\StatusService::setUserStatus seems to do the trick. Didn't test for any other side effects of this

I don't understand what 0 timetamp you're referring to?

@ChristophWurst
Copy link
Member Author

#41512

@ChristophWurst ChristophWurst moved this from 📄 To do to 🏗️ In progress in 💌 📅 👥 Groupware team Feb 12, 2024
@ChristophWurst ChristophWurst added 3. to review Waiting for reviews and removed 1. to develop Accepted and waiting to be taken care of labels Feb 12, 2024
@github-project-automation github-project-automation bot moved this from 🏗️ In progress to ☑️ Done in 💌 📅 👥 Groupware team Feb 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

2 participants