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

Update Cockpit applications metadata #717

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

Conversation

MrGrymReaper
Copy link

@MrGrymReaper MrGrymReaper commented Mar 22, 2024

I recently found out an Cockpit application which is currently released and in active development, though not listed on the official website's applications list. Due to it being excluded from this list people are more likely to not know about this application, in order to fix this issue. I decided to contribute the necessary information to the project.

Fixes #718

Added an application which was missing from the list of applications, but only linked to on the 45Drives GitHub repository for the File Sharing application.
@MrGrymReaper MrGrymReaper changed the title Update applications.yml Update Cockpit applications metadata Mar 22, 2024
Copy link
Member

@martinpitt martinpitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

official: false
prerelease: false
description: |
A Cockpit plugin to easily manage users and groups along with associated items.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please make the description a bit more concrete? The builtin Accounts page can do all the listed features except for setting the Samba password. I.e. please drop "easily", if possible avoid "manage", and replace the vague "associated items" with something more specific. I.e. it should answer the question "why and when would I want to install this?" Unfortunately the upstream project itself isn't very verbose in that regard.

Copy link
Member

@martinpitt martinpitt Mar 24, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be more specific -- the 45drives apps are not very reliable. They have zero tests, do, well, "interesting" things with shell commands, don't fit the general Cockpit UI look and guidelines, are not translatable/translated, etc. Presumably they did these for a specific product, but Cockpit is supported across a wide range of operating systems.

In other words, we don't want to recommend this page for standard Unix user management. If there's some "killer feature" which the Accounts page doesn't have, it should be pointed out as such.

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

Successfully merging this pull request may close these issues.

Unlisted application - Cockpit Identities
2 participants