Skip to content

Conversation

@max-nextcloud
Copy link
Collaborator

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

2 flaky tests on run #8538 ↗︎

0 99 0 0 Flakiness 2

Details:

Merge c348ef4 into a9ceb40...
Project: Text Commit: d3ad9ced05 ℹ️
Status: Passed Duration: 05:15 💡
Started: Feb 8, 2023 7:43 AM Ended: Feb 8, 2023 7:48 AM
Flakiness  share.spec.js • 1 flaky test

View Output Video

Test
Open test.md in viewer > Shares a folder as a public read only link Screenshot
Flakiness  links.spec.js • 1 flaky test

View Output Video

Test
test link marks > link preview > shows a link preview Screenshot

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

@max-nextcloud max-nextcloud changed the title fix(Polling): Perform computations in milliseconds [stable25] fix(Polling): Perform computations in milliseconds Feb 7, 2023
@max-nextcloud max-nextcloud force-pushed the backport/3769/stable25 branch from 1274f85 to ff703ea Compare February 7, 2023 09:43
@max-nextcloud
Copy link
Collaborator Author

/compile

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 <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>
@juliusknorr juliusknorr merged commit 0ba87db into stable25 Feb 8, 2023
@juliusknorr juliusknorr deleted the backport/3769/stable25 branch February 8, 2023 12:46
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