kbevers
commented
1 year ago En mulighed kunne være at lade --db uden argument returnere hvilken database der er sat som default. Eller bare spytte det ud i noget logging (som endnu ikke er implementeret). Men måske er problemet ikke så stort i den virkelige verden, da applikationen fortæller hvilken database man arbejder på hver gang man er i færd med at ændre på indholdet i databasen.
Generelt er min anbefaling at man som "almindelig" bruger har default sat til prod og ellers tilføjer --db test hvis der er brug for testdatabasen.
Forslag til forbedring.
Som det er nu, skal man orientere sig i ini-filen eller specifikt eksplicificere hvilken database man vil arbejde i. Nedenstående tilføjer kommandoen "visdb" og returnerer information om hvorvidt det enten er test- eller produktionsdatabasen, som er i brug (koden kan indsættes i bunden af /fire/main.py).
Det kønneste ville være en kommando med -- foran, fx --visdb, men det ligger udenfor mine evner for nuværende :-)
@fire.command def visdb(): """ Vis aktuel databaseforbindelse (test/prod) """ from fire.api.configuration import get_configuration myparser = get_configuration() click.echo("Du arbejder på "+myparser.get("general","default_connection").upper()+"-databasen")