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

Improve HPB warning scenarios #14018

Closed
nickvergessen opened this issue Dec 18, 2024 · 6 comments · Fixed by #14086
Closed

Improve HPB warning scenarios #14018

nickvergessen opened this issue Dec 18, 2024 · 6 comments · Fixed by #14086
Assignees
Labels
1. to develop bug feature: signaling 📶 Internal and external signaling backends

Comments

@nickvergessen
Copy link
Member

How to use GitHub

  • Please use the 👍 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

  • Warn in all calls if HPB is not present (the 4 person warning atm seems to have disappeared?)
  • Warn in admin settings and setupcheck about incomplete setup if HPB is not present

Warning

Nextcloud Talk setup not complete

Install the High Performance Backend to ensure calls with multiple people work seamlessly.

@nickvergessen nickvergessen added 1. to develop bug feature: signaling 📶 Internal and external signaling backends labels Dec 18, 2024
@nickvergessen nickvergessen added this to the 🖤 Next Major (31) milestone Dec 18, 2024
@nickvergessen nickvergessen self-assigned this Dec 18, 2024
@github-project-automation github-project-automation bot moved this to 🧭 Planning evaluation (don't pick) in 💬 Talk team Dec 18, 2024
@nickvergessen nickvergessen moved this from 🧭 Planning evaluation (don't pick) to 📄 To do (~10 entries) in 💬 Talk team Dec 18, 2024
@Antreesy
Copy link
Contributor

Antreesy commented Dec 18, 2024

  • Warn in admin settings and setupcheck about incomplete setup if HPB is not present

In which place?
Image
Image

  • Warn in all calls if HPB is not present (the 4 person warning atm seems to have disappeared?)

Don't see it in the code, will try to restore

@nickvergessen
Copy link
Member Author

-Warn in admin settings and setupcheck about incomplete setup if HPB is not present

In which place?

I will take care of that part for now and ping you later if needed

@nimishavijay
Copy link
Member

Great idea! We should also add a call to action button in the note card as an obvious next step, possibly with a link to the relevant documentation or something else that would help the set up process

@nickvergessen
Copy link
Member Author

nickvergessen commented Jan 2, 2025

Frontend Call Warning history

@Antreesy
Copy link
Contributor

Antreesy commented Jan 8, 2025

From PR discussion (QualityWarningTooltip):

  • we can use an icon in the call view, I'd even say use the same connection icon entry, just add another block of text in the popover?

@nickvergessen
Copy link
Member Author

nickvergessen commented Jan 9, 2025

@github-project-automation github-project-automation bot moved this from 📄 To do (~10 entries) to ☑️ Done in 💬 Talk team Jan 9, 2025
@github-project-automation github-project-automation bot moved this from 🏗️ At engineering to 🎉 Done in 🖍 Design team Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop bug feature: signaling 📶 Internal and external signaling backends
Projects
Status: ☑️ Done
Status: 🎉 Done
Development

Successfully merging a pull request may close this issue.

3 participants