Skip to content

Commit

Permalink
Remove --verified required argument from CLI register-user command
Browse files Browse the repository at this point in the history
  • Loading branch information
tcompa committed Dec 21, 2023
1 parent d55ef8b commit 34d32a6
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 8 deletions.
1 change: 0 additions & 1 deletion fractal_client/cmd/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -340,7 +340,6 @@ async def user(
"slurm_user",
"username",
"superuser",
"verified",
]
function_kwargs = get_kwargs(parameters, kwargs)
iface = await user_register(client, batch=batch, **function_kwargs)
Expand Down
2 changes: 1 addition & 1 deletion fractal_client/cmd/_user.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ async def user_register(
cache_dir: Optional[str] = None,
username: Optional[str] = None,
superuser: bool = False,
verified: bool = False,
verified: bool = True, # TODO: this is not currently exposed in the CLI
batch: bool = False,
) -> Union[RichJsonInterface, PrintInterface]:

Expand Down
6 changes: 0 additions & 6 deletions fractal_client/parser.py
Original file line number Diff line number Diff line change
Expand Up @@ -852,12 +852,6 @@
action="store_true",
required=False,
)
user_register_parser.add_argument(
"--verified",
help="Make the new user verified.",
action="store_true",
required=True,
)

# user list
user_list_parser = user_subparsers.add_parser(
Expand Down

0 comments on commit 34d32a6

Please sign in to comment.