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

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

Issue 29908581: Issue #7036 - [XTM Integration] Add support for providing workflow name (Closed)
Patch Set: Addressed comments from Patch Set 1 Created Oct. 18, 2018, 4:23 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
Index: cms/translations/xtm/projects_handler.py
diff --git a/cms/translations/xtm/projects_handler.py b/cms/translations/xtm/projects_handler.py
index 1e5c718d95121cd4bcdca16c44ad96899cc5d48d..95bb86620bc77978916cf338aeae16c00abdbc2b 100644
--- a/cms/translations/xtm/projects_handler.py
+++ b/cms/translations/xtm/projects_handler.py
@@ -71,11 +71,11 @@ def create_project(args, api, source):
reference_id=args.ref_id,
target_languages=target_langs,
customer_id=args.client_id,
- workflow_id=args.workflow_id,
+ workflow_id=utils.extract_workflow_id(api, args),
source_language=args.source_lang,
files=files_to_upload,
)
- except XTMCloudException as err:
+ except Exception as err:
sys.exit(err)
logging.info(const.InfoMessages.PROJECT_CREATED.format(project_id))
« no previous file with comments | « cms/translations/xtm/constants.py ('k') | cms/translations/xtm/utils.py » ('j') | tests/utils.py » ('J')

Powered by Google App Engine
This is Rietveld