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

BBB-116 - Only admin user can enable recordings when configuring a me… #73

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

Conversation

danielmerino
Copy link

…eting. New patch that use new configuration options.

…eting. New patch that use new configuration options.
@danielmerino
Copy link
Author

Hi @jfederico I have adapted my patch for the new properties. For example I have tested it with these ones and it works in my test server:

bbb.recording.enabled=true
bbb.recording.editable=false
bbb.recording.default=false
bbb.admin.recording.enabled=true

I have had several conflicts when modifying the code, so I have deleted the old branch and I have done this brand new one.

Please, could you tell me if you think this patch is fine or if I should do anything else?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant