zhongjiajie commented on code in PR #12381:
URL: https://github.com/apache/dolphinscheduler/pull/12381#discussion_r999129444
##########
dolphinscheduler-python/pydolphinscheduler/src/pydolphinscheduler/cli/commands.py:
##########
@@ -93,6 +94,206 @@ def config(getter, setter, init) -> None:
click.echo("Set configuration done.")
[email protected]()
[email protected](
+ "--set",
+ "-s",
+ "setter",
+ multiple=False,
+ type=click.Tuple([str, str, str, str, str, str, int]),
+ help="Set user to pydolphinscheduler."
+ "Use ``--set <NAME> <PASSWORD> <EMAIL> <PHONE> <TENANT> <QUEUE> <STATUS>``
options to set user",
Review Comment:
we currently use cli `user --set <NAME> <PASSWORD> <EMAIL> <PHONE> <TENANT>
<QUEUE> <STATUS>`, but how about using `user create --user-name <NAME>
--password <PASSWORD> --email <MAIL> ...` for more readable?
also for the update `user update <NAME> --email <MAIL> ...` for the part of
update?
##########
dolphinscheduler-python/pydolphinscheduler/src/pydolphinscheduler/cli/commands.py:
##########
@@ -93,6 +94,206 @@ def config(getter, setter, init) -> None:
click.echo("Set configuration done.")
[email protected]()
[email protected](
+ "--set",
+ "-s",
+ "setter",
+ multiple=False,
+ type=click.Tuple([str, str, str, str, str, str, int]),
+ help="Set user to pydolphinscheduler."
+ "Use ``--set <NAME> <PASSWORD> <EMAIL> <PHONE> <TENANT> <QUEUE> <STATUS>``
options to set user",
Review Comment:
Same as tenant and project
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]