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.

Sign up for GitHub

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

Jump to bottom

Update Cockpit applications metadata #717

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

Update Cockpit applications metadata #717

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions _data/applications.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -176,6 +176,15 @@ navigator:
prerelease: false
description: |
A Featureful File Browser for Cockpit.

identities:
title: Cockpit Identities
source: https://github.com/45drives/cockpit-identities
issues: https://github.com/45Drives/cockpit-identities/issues
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 •
edited
Loading

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.


benchmark:
title: Benchmark
Expand Down