nextcloud / richdocuments

📑 Collabora Online for Nextcloud
https://nextcloud.com/collaboraonline
350 stars 116 forks source link

Talk meetings on shared nc office files with guest cannot be performed #4105

Open stoneubi opened 3 weeks ago

stoneubi commented 3 weeks ago

Hi there,

my actual versions are:

grafik

Description / steps to reproduce:

The thing is that you want to discuss a presentation with a guest, you share the presentation but then you cannot use talk.

If you close the presentation, the chat view will be visible: grafik

and you can open the chat part -> you have to enter a name. Now you are in the chat part but cannot modify the presentation together with someone else.

How can the system be configured to allow talk meetings during a collaborate file creation / modification with guests = no nextcloud user.

Thank you, Stoneubi

Antreesy commented 3 weeks ago
nickvergessen commented 2 weeks ago

should be backported to stable28/29 / if nextcloud/vue version is 8.13.0 or higher

Can you check and confirm?

Antreesy commented 4 days ago

Fix is in there and restores access to chat sidebar from public link. But Office integration doesn't show a toggle for guests, it seems. so I'd check with Office team, if they can provide it

Guest User
Image Image
juliushaertl commented 3 days ago

Two questions:

Antreesy commented 3 days ago
  • How can we check from the richdocuments app

This returns a conversation token for public share: https://github.com/nextcloud/spreed/blob/1ccbf67da231c37801c1406b10000144125aec35/src/services/filesIntegrationServices.js#L28-L31

If response is successfull = Talk is available, and conversation can be joined

  • How can we trigger/toggle the sidebar in the frontend?

We already have a trigger mounted in DOM - in NC header (but it's behind the Office integration): https://github.com/nextcloud/spreed/blob/976f67921c09848e0780cd8c1b70b663cf587c50/src/mainPublicShareSidebar.js#L74 Image

Maybe just shrinking of Office integration would be enough (similar to how it's done for Files app?)

juliushaertl commented 3 days ago

This returns a conversation token for public share: https://github.com/nextcloud/spreed/blob/1ccbf67da231c37801c1406b10000144125aec35/src/services/filesIntegrationServices.js#L28-L31

If response is successfull = Talk is available, and conversation can be joined

We'd need a backend way but guess we can dig this up from the route then.

Thanks for the pointers