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

feat: Add setup check to verify whiteboard backend server version and connectivity #297

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Dec 6, 2024

This should help us detect a few pitfalls with the setup

  • Detect if the Nextcloud app and backend server versions are not in sync
  • Detect if the Nextcloud server can properly connect to the whiteboard server
  • Detect if the whiteboard server can connect back to the Nextcloud server

The setup check is indicating problems now in the admin overview as well as when running occ setupcheck and when going to the whiteboard admin settings.

Fix #144

Adds some checks for parts of #238

@hweihwang
Copy link
Contributor

When I tested a bit, it kept saying failed to connect to whiteboard server even the whiteboard working fine. Seems the checks doing a curl from nextcloud backend to whiteboard server, not entirely sure but should it be a check from frontend to whiteboard server @juliusknorr (Maybe one check/call from FE => status endpoint of whiteboard server)

@juliusknorr
Copy link
Member Author

  • Need to check connectivity issues with docker when the node process is running outside of it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Check backend version
2 participants