Browse Source

add usage info

abhishek9686 1 year ago
parent
commit
1ffc1f220b
2 changed files with 12 additions and 25 deletions
  1. 6 14
      cli/cmd/user/groups.go
  2. 6 11
      cli/cmd/user/roles.go

+ 6 - 14
cli/cmd/user/groups.go

@@ -18,10 +18,6 @@ var userGroupCmd = &cobra.Command{
 	Long:  `Manage User Groups`,
 	Long:  `Manage User Groups`,
 }
 }
 
 
-// List Roles
-var (
-	groupID string
-)
 var userGroupListCmd = &cobra.Command{
 var userGroupListCmd = &cobra.Command{
 	Use:   "list",
 	Use:   "list",
 	Args:  cobra.NoArgs,
 	Args:  cobra.NoArgs,
@@ -65,12 +61,12 @@ var userGroupCreateCmd = &cobra.Command{
 }
 }
 
 
 var userGroupDeleteCmd = &cobra.Command{
 var userGroupDeleteCmd = &cobra.Command{
-	Use:   "delete",
-	Args:  cobra.NoArgs,
+	Use:   "delete [groupID]",
+	Args:  cobra.ExactArgs(1),
 	Short: "delete user group",
 	Short: "delete user group",
 	Long:  `delete user group`,
 	Long:  `delete user group`,
 	Run: func(cmd *cobra.Command, args []string) {
 	Run: func(cmd *cobra.Command, args []string) {
-		resp := functions.DeleteUserGrp(groupID)
+		resp := functions.DeleteUserGrp(args[0])
 		if resp != nil {
 		if resp != nil {
 			fmt.Println(resp.Message)
 			fmt.Println(resp.Message)
 		}
 		}
@@ -78,12 +74,12 @@ var userGroupDeleteCmd = &cobra.Command{
 }
 }
 
 
 var userGroupGetCmd = &cobra.Command{
 var userGroupGetCmd = &cobra.Command{
-	Use:   "get",
-	Args:  cobra.NoArgs,
+	Use:   "get [groupID]",
+	Args:  cobra.ExactArgs(1),
 	Short: "get user group",
 	Short: "get user group",
 	Long:  `get user group`,
 	Long:  `get user group`,
 	Run: func(cmd *cobra.Command, args []string) {
 	Run: func(cmd *cobra.Command, args []string) {
-		data := functions.GetUserGrp(groupID)
+		data := functions.GetUserGrp(args[0])
 		switch commons.OutputFormat {
 		switch commons.OutputFormat {
 		case commons.JsonOutput:
 		case commons.JsonOutput:
 			functions.PrettyPrint(data)
 			functions.PrettyPrint(data)
@@ -115,12 +111,8 @@ func init() {
 	userGroupCmd.AddCommand(userGroupCreateCmd)
 	userGroupCmd.AddCommand(userGroupCreateCmd)
 
 
 	// delete role cmd
 	// delete role cmd
-	userGroupDeleteCmd.Flags().StringVar(&groupID, "group-id", "", "user role ID")
-	userGroupDeleteCmd.MarkFlagRequired("role-id")
 	userGroupCmd.AddCommand(userGroupDeleteCmd)
 	userGroupCmd.AddCommand(userGroupDeleteCmd)
 
 
 	// Get Role
 	// Get Role
-	userGroupGetCmd.Flags().StringVar(&groupID, "group-id", "", "user role ID")
-	userGroupGetCmd.MarkFlagRequired("group-id")
 	userGroupCmd.AddCommand(userGroupGetCmd)
 	userGroupCmd.AddCommand(userGroupGetCmd)
 }
 }

+ 6 - 11
cli/cmd/user/roles.go

@@ -21,7 +21,6 @@ var userRoleCmd = &cobra.Command{
 // List Roles
 // List Roles
 var (
 var (
 	platformRoles bool
 	platformRoles bool
-	roleID        string
 )
 )
 var userRoleListCmd = &cobra.Command{
 var userRoleListCmd = &cobra.Command{
 	Use:   "list",
 	Use:   "list",
@@ -64,12 +63,12 @@ var userRoleCreateCmd = &cobra.Command{
 }
 }
 
 
 var userRoleDeleteCmd = &cobra.Command{
 var userRoleDeleteCmd = &cobra.Command{
-	Use:   "delete",
-	Args:  cobra.NoArgs,
+	Use:   "delete [roleID]",
+	Args:  cobra.ExactArgs(1),
 	Short: "delete user role",
 	Short: "delete user role",
 	Long:  `delete user role`,
 	Long:  `delete user role`,
 	Run: func(cmd *cobra.Command, args []string) {
 	Run: func(cmd *cobra.Command, args []string) {
-		resp := functions.DeleteUserRole(roleID)
+		resp := functions.DeleteUserRole(args[0])
 		if resp != nil {
 		if resp != nil {
 			fmt.Println(resp.Message)
 			fmt.Println(resp.Message)
 		}
 		}
@@ -77,12 +76,12 @@ var userRoleDeleteCmd = &cobra.Command{
 }
 }
 
 
 var userRoleGetCmd = &cobra.Command{
 var userRoleGetCmd = &cobra.Command{
-	Use:   "get",
-	Args:  cobra.NoArgs,
+	Use:   "get [roleID]",
+	Args:  cobra.ExactArgs(1),
 	Short: "get user role",
 	Short: "get user role",
 	Long:  `get user role`,
 	Long:  `get user role`,
 	Run: func(cmd *cobra.Command, args []string) {
 	Run: func(cmd *cobra.Command, args []string) {
-		d := functions.GetUserRole(roleID)
+		d := functions.GetUserRole(args[0])
 		switch commons.OutputFormat {
 		switch commons.OutputFormat {
 		case commons.JsonOutput:
 		case commons.JsonOutput:
 			functions.PrettyPrint(d)
 			functions.PrettyPrint(d)
@@ -115,12 +114,8 @@ func init() {
 	userRoleCmd.AddCommand(userRoleCreateCmd)
 	userRoleCmd.AddCommand(userRoleCreateCmd)
 
 
 	// delete role cmd
 	// delete role cmd
-	userRoleDeleteCmd.Flags().StringVar(&roleID, "role-id", "", "user role ID")
-	userRoleDeleteCmd.MarkFlagRequired("role-id")
 	userRoleCmd.AddCommand(userRoleDeleteCmd)
 	userRoleCmd.AddCommand(userRoleDeleteCmd)
 
 
 	// Get Role
 	// Get Role
-	userRoleGetCmd.Flags().StringVar(&roleID, "role-id", "", "user role ID")
-	userRoleGetCmd.MarkFlagRequired("role-id")
 	userRoleCmd.AddCommand(userRoleGetCmd)
 	userRoleCmd.AddCommand(userRoleGetCmd)
 }
 }