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

fix: replace removed success and complete callbacks #589

Merged
merged 1 commit into from
Apr 18, 2024

Conversation

kesselb
Copy link
Collaborator

@kesselb kesselb commented Apr 18, 2024

Fix #588

also reduced the polling intervals a bit to decrease load

How to test

  • Open the monitoring app
  • See graphs for cpu and memory (that's broken on master due to a jquery update in server)

also reduced the polling intervals a bit to decrease load

Signed-off-by: Daniel Kesselberg <[email protected]>
@kesselb kesselb added this to the Nextcloud 30 milestone Apr 18, 2024
@kesselb kesselb self-assigned this Apr 18, 2024
@kesselb
Copy link
Collaborator Author

kesselb commented Apr 18, 2024

/backport to stable29

@tobiasKaminsky
Copy link
Member

Works.
Thanks for the test instructions, so now me as a client dev could test it 👏

@kesselb kesselb merged commit 7e23ee8 into master Apr 18, 2024
31 checks passed
@delete-merged-branch delete-merged-branch bot deleted the bug/noid/fix-cpu-memory-chart branch April 18, 2024 12:32
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.

NC29RC4 no CPU and no Memory graphs shown
3 participants