-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
chore: remove documentation for user_external #11828
Conversation
Users expect, because the app is mentioned in our admin documentation, that there should be a compatible release and are disappointed if not. We do not maintain or support user_external, and therefore the mention in our documention should go. Signed-off-by: Daniel Kesselberg <[email protected]>
Woah, this goes way back. It's been around forever. (EDIT: to clarify: the "woah" was a reaction to seeing the age of the original addition; not about removing it). I'm okay with this, though it brings up a larger discussion of whether it's okay to mention things like this in the docs but somehow make it clearer they're community apps/not fully supported/etc. Sometimes we're going to want to do this I imagine. That said, the focus of the Admin Manual should be on shipped plus... featured?I guess. |
Backports desired? |
/backport to stable29 |
/backport to stable28 |
/backport to stable27 |
True. I guess it's okay and also wanted to have hints about apps in our documentation, but those apps should be maintained and not abandoned. ;) As reference: |
I think the thing here is that user_external was once created by a core dev and was part of a repo with multiple apps which were (I think) shipped with Nextcloud. That bundled repo was then abandoned and not integrated any more. Then user_external was split out of this and I'm since trying to maintain it in my free time. However my workload for university is currently so high that I don't find time to properly test and feel confident about creating releases for this app, sorry :/ |
Thanks @violoncelloCH, for the update and maintaining user_external 👍 I think it's fine and understandable that university is your top priority. |
@kesselb Is this something which should get back added in to the core, or maybe at least parts of it? Especially things like IMAP, HTTP BasicAuth and SSH which is very close to the functionality of nextcloud.
Maybe the IMAP part could be integrated back in to nextcloud with the email overhaul? |
I don't think so. |
☑️ Resolves
Users expect, because the app is mentioned in our admin documentation, that there should be a compatible release and are disappointed if not.
We do not maintain or support user_external, and therefore the mention in our documention should go.
🖼️ Screenshots