-
Notifications
You must be signed in to change notification settings - Fork 167
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
Added hooks for setting the name of the user and the editor permission #115
base: master
Are you sure you want to change the base?
Conversation
Hello @Peterede, thank you for your interest in improving ONLYOFFICE. Could you please describe the aim to be achieved by implementing this feature list. We would ask you to describe the usage scenario. |
In the nextcloud talk app we invite a number of guests to a meeting then share a link to a document with them. We do not want the guests to appear as anonymous, we want to use the name set in the call so this is where the name hook is used. For the editing privileges we do not want guests to be able to edit, only view, so this is where the editable hook is used. |
@alexanderonlyoffice is there anything else needed to merge this pull request into master? |
As discussed at the Nextcloud conference we should aim for a unified server UI to set the guest name across different apps. @Peterede Feel free to join the discussion in nextcloud/server#17147 |
The hooks implemented in this pull request could be used by the unified server UI to set the guest name. This will then still allow other apps to implement the hooks and extend the behaviour. |
|
Allow other apps to supply the name and editing permissions