Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: cms/translations/xtm/cli.py

Issue 29969555: Issue 7039 - [XTM Integration] Make parameter naming consistent with the API docs
Patch Set: Created Dec. 28, 2018, 2:51 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | cms/translations/xtm/constants.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cms/translations/xtm/cli.py
diff --git a/cms/translations/xtm/cli.py b/cms/translations/xtm/cli.py
index 9bade4323d56be2a85360ea696297c07c45b50ca..c3f7ea602b310bd25baa59b9e280f53580dca38d 100644
--- a/cms/translations/xtm/cli.py
+++ b/cms/translations/xtm/cli.py
@@ -43,9 +43,9 @@ def handle_projects(args):
def generate_token(args):
"""Generate an API token from username and password."""
- username = input_fn('Username: ')
+ username = input_fn('Client name: ')
Vasily Kuznetsov 2019/01/04 20:21:14 Do you think it will be more clear to say "Client
Tudor Avram 2019/01/11 13:03:42 That was why I used `username` initially, but Wins
user_id = input_fn('User ID: ')
- password = getpass.getpass(prompt='Pasword: ')
+ password = getpass.getpass(prompt='Password: ')
logging.info(const.InfoMessages.GENERATING_TOKEN.format(username, user_id))
try:
@@ -96,8 +96,8 @@ def parse_args():
)
project_create_parser.add_argument(
- '--client-id', required=True, type=int,
- help=const.ArgumentsHelp.ProjectCreate.CLIENT_ID,
+ '--customer-id', required=True, type=int,
+ help=const.ArgumentsHelp.ProjectCreate.CUSTOMER_ID,
)
project_create_parser.add_argument(
'--ref-id', required=True,
« no previous file with comments | « no previous file | cms/translations/xtm/constants.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld