-
Notifications
You must be signed in to change notification settings - Fork 711
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
containers-schema: allow to specify ui-secret and show in aio interface #5945
Conversation
d81e046
to
1d4aeb2
Compare
Signed-off-by: Simon L. <szaimen@e.mail.de>
1d4aeb2
to
04ef456
Compare
@docjyJ I think this is probably also interesting for you. Please review :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good. Somme sugest:
- Update LLDAP json, Stalwart Readme/Json, NocoDB Readme/Json
- Add field with hide/show (Ex: https://codepen.io/piyush-tapaniya/pen/PvWKKj)
- Add copy button (Ex: https://codepen.io/foxbeefly/pen/dyWbQgJ)
I have not enough time to do that now...
Signed-off-by: Simon L. <szaimen@e.mail.de>
Thanks for the review @docjyJ! :)
I have adjusted all except the stalwart readme since that is in your repo...
Can we please do this in a follow-up? I've created #5954 for that :) |
This is now released with v10.5.0 Beta. Testing and feedback is welcome! See https://github.com/nextcloud/all-in-one#how-to-switch-the-channel |
Close #5938