From 0b3627218f0ea021a7a5a8f62a7e397a0f81090d Mon Sep 17 00:00:00 2001 From: Julien Maupetit Date: Mon, 16 Dec 2024 16:50:57 +0100 Subject: [PATCH] =?UTF-8?q?=F0=9F=92=84(api:cli)=20update=20groups=20/=20o?= =?UTF-8?q?perational=20units=20ordering?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit When using the `list-groups` command it's more user-friendly to sort things alphabetically. --- src/api/CHANGELOG.md | 2 ++ src/api/qualicharge/cli.py | 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/api/CHANGELOG.md b/src/api/CHANGELOG.md index adeaa949..9499b8da 100644 --- a/src/api/CHANGELOG.md +++ b/src/api/CHANGELOG.md @@ -19,6 +19,8 @@ and this project adheres to - Improve bulk endpoints permissions checking - Cache logged user object for `API_GET_USER_CACHE_TTL` seconds to decrease the number of database queries +- CLI: sort groups and operational units alphabetically in the `list-groups` + command ## [0.16.0] - 2024-12-12 diff --git a/src/api/qualicharge/cli.py b/src/api/qualicharge/cli.py index c7386535..8b021447 100644 --- a/src/api/qualicharge/cli.py +++ b/src/api/qualicharge/cli.py @@ -38,7 +38,7 @@ def list_groups(ctx: typer.Context): """List API groups.""" session: SMSession = ctx.obj - groups = session.exec(select(Group)).all() + groups = session.exec(select(Group).order_by(Group.name)).all() table = Table(title="QualiCharge API groups") table.add_column("Name", justify="right", style="cyan", no_wrap=True) @@ -49,7 +49,7 @@ def list_groups(ctx: typer.Context): table.add_row( group.name, ",".join(sorted(user.username for user in group.users)), - ",".join(ou.code for ou in group.operational_units), + ",".join(sorted(ou.code for ou in group.operational_units)), ) console.print(table)