Change CLI oauth2 command from set-display-name to set-displayname for consistency. (#3212)

* Change CLI domain command from set-display-name to set-displayname for consistency.

* Also fix CLI.
This commit is contained in:
George Wu 2024-11-18 20:28:07 -08:00 committed by GitHub
parent 155225c0f9
commit ed38e92448
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 6 additions and 6 deletions

View file

@ -7,7 +7,7 @@ use std::fs::read;
impl DomainOpt { impl DomainOpt {
pub fn debug(&self) -> bool { pub fn debug(&self) -> bool {
match self { match self {
DomainOpt::SetDisplayName(copt) => copt.copt.debug, DomainOpt::SetDisplayname(copt) => copt.copt.debug,
DomainOpt::SetLdapBasedn { copt, .. } DomainOpt::SetLdapBasedn { copt, .. }
| DomainOpt::SetImage { copt, .. } | DomainOpt::SetImage { copt, .. }
| DomainOpt::RemoveImage { copt } | DomainOpt::RemoveImage { copt }
@ -19,7 +19,7 @@ impl DomainOpt {
pub async fn exec(&self) { pub async fn exec(&self) {
match self { match self {
DomainOpt::SetDisplayName(opt) => { DomainOpt::SetDisplayname(opt) => {
eprintln!( eprintln!(
"Attempting to set the domain's display name to: {:?}", "Attempting to set the domain's display name to: {:?}",
opt.new_display_name opt.new_display_name

View file

@ -1078,7 +1078,7 @@ pub enum Oauth2Opt {
#[clap(name = "delete")] #[clap(name = "delete")]
/// Delete a oauth2 client /// Delete a oauth2 client
Delete(Named), Delete(Named),
/// Set a new displayname for a client /// Set a new display name for a client
#[clap(name = "set-displayname")] #[clap(name = "set-displayname")]
SetDisplayname(Oauth2SetDisplayname), SetDisplayname(Oauth2SetDisplayname),
/// Set a new name for this client. You may need to update /// Set a new name for this client. You may need to update
@ -1197,7 +1197,7 @@ pub enum Oauth2Opt {
} }
#[derive(Args, Debug)] #[derive(Args, Debug)]
pub struct OptSetDomainDisplayName { pub struct OptSetDomainDisplayname {
#[clap(flatten)] #[clap(flatten)]
copt: CommonOpt, copt: CommonOpt,
#[clap(name = "new-display-name")] #[clap(name = "new-display-name")]
@ -1260,9 +1260,9 @@ pub enum DeniedNamesOpt {
#[derive(Debug, Subcommand)] #[derive(Debug, Subcommand)]
pub enum DomainOpt { pub enum DomainOpt {
#[clap[name = "set-display-name"]] #[clap[name = "set-displayname"]]
/// Set the domain display name /// Set the domain display name
SetDisplayName(OptSetDomainDisplayName), SetDisplayname(OptSetDomainDisplayname),
#[clap[name = "set-ldap-basedn"]] #[clap[name = "set-ldap-basedn"]]
/// Change the basedn of this server. Takes effect after a server restart. /// Change the basedn of this server. Takes effect after a server restart.
/// Examples are `o=organisation` or `dc=domain,dc=name`. Must be a valid ldap /// Examples are `o=organisation` or `dc=domain,dc=name`. Must be a valid ldap