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

[stable24] fix(Polling): Perform computations in milliseconds #3774

Merged
merged 2 commits into from
Feb 8, 2023

Conversation

max-nextcloud
Copy link
Collaborator

@max-nextcloud max-nextcloud commented Feb 7, 2023

The collaborator disconnect timeout is in ms as are all intervals. So we can directly subtract it from Date.now() - which also is in ms. The only time that is not expressed in ms is the last contact timestamp.

Backport of #3769

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests are missing as this is hard to test.
  • Documentation is not required

@max-nextcloud
Copy link
Collaborator Author

/compile

@cypress
Copy link

cypress bot commented Feb 7, 2023

Passing run #8536 ↗︎

0 51 0 0 Flakiness 0

Details:

Merge 4f71345 into 1d42608...
Project: Text Commit: 0a23c2b5f7 ℹ️
Status: Passed Duration: 02:19 💡
Started: Feb 8, 2023 7:35 AM Ended: Feb 8, 2023 7:38 AM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@max-nextcloud max-nextcloud force-pushed the backport/3769/stable24 branch from 85f9442 to be1f559 Compare February 7, 2023 09:42
@max-nextcloud
Copy link
Collaborator Author

/compile

@max-nextcloud max-nextcloud changed the title fix(Polling): Perform computations in milliseconds [stable24] fix(Polling): Perform computations in milliseconds Feb 7, 2023
max-nextcloud and others added 2 commits February 8, 2023 08:33
The collaborator disconnect timeout is in ms as are all intervals.
So we can directly subtract it from `Date.now()` - which also is in ms.
The only time that is not expressed in ms is the last contact timestamp.

Backport of #3769

Signed-off-by: Max <max@nextcloud.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@max-nextcloud max-nextcloud force-pushed the backport/3769/stable24 branch from 8b4fcb8 to 4f71345 Compare February 8, 2023 07:33
@juliusknorr juliusknorr merged commit 46d71bb into stable24 Feb 8, 2023
@delete-merged-branch delete-merged-branch bot deleted the backport/3769/stable24 branch February 8, 2023 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants