-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Avoid the error when there're no panels to show under one section on settings #27518
Comments
For context, some panels choose not to be rendered based on current config or other settings, and we only know this on render time. But Additional is different since it also contains legacy calls, if none are registered then we could hide the section, however it is possible for apps to use the new method and still request to be in additional - hmmm |
Hey, this issue has been closed because the label |
Hey, this issue has been closed because the label (This is an automated comment from GitMate.io.) |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Server version:
ownCloud version: 10.0 beta1
Reproduction step:
e.g. In current master go to
Settings > Personal > Additional
Expected behavior
Maybe the section can be hidden from the sidebar until at least there's one panel to show.
Actual behavior
Error is displayed:
The text was updated successfully, but these errors were encountered: