Side by Side Diff
Use n/p to move between diff chunks; N/P to move between comments.
Keyboard Shortcuts
File
u
:
up to issue
m
:
publish + mail comments
M
:
edit review message
j
/
k
:
jump to file after / before current file
J
/
K
:
jump to next file with a comment after / before current file
Side-by-side diff
i
:
toggle intra-line diffs
e
:
expand all comments
c
:
collapse all comments
s
:
toggle showing all comments
n
/
p
:
next / previous diff chunk or comment
N
/
P
:
next / previous comment
<Up>
/
<Down>
:
next / previous line
<Enter>
:
respond to / edit current comment
d
:
mark current comment as done
Issue
u
:
up to list of issues
m
:
publish + mail comments
j
/
k
:
jump to patch after / before current patch
o
/
<Enter>
:
open current patch in side-by-side view
i
:
open current patch in unified diff view
Issue List
j
/
k
:
jump to issue after / before current issue
o
/
<Enter>
:
open current issue
#
:
close issue
Comment/message editing
<Ctrl>
+
s
or
<Ctrl>
+
Enter
:
save comment
<Esc>
:
cancel edit
Rietveld
Code Review Tool
Help
|
Bug tracker
|
Discussion group
|
Source code
Issues
All Issues
Side by Side Diff: cms/translations/xtm/__init__.py
Issue
29886648
:
Issue #6942 - Add XTM integration in CMS (Closed)
Patch Set: Addressed comments from patch set #3
Created Oct. 5, 2018, 12:30 p.m.
Left:
Base
Patch Set 1: None
Patch Set 2: Addressed initial comments
Patch Set 3: Addressed comments on patch set #2
Patch Set 4: Addressed comments from patch set #3
Patch Set 5: Addressed comments from Patch Set #4
Right:
Patch Set 1: None
Patch Set 2: Addressed initial comments
Patch Set 3: Addressed comments on patch set #2
Patch Set 4: Addressed comments from patch set #3
Patch Set 5: Addressed comments from Patch Set #4
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
.gitignore
.hgignore
cms/bin/xtm_translations.py
cms/sources.py
cms/translations/__init__.py
cms/translations/xtm/__init__.py
cms/translations/xtm/cli.py
cms/translations/xtm/constants.py
cms/translations/xtm/projects_handler.py
cms/translations/xtm/utils.py
cms/translations/xtm/xtm_api.py
docs/usage/xml-sync.md
tests/conftest.py
tests/test_site/settings.ini
tests/test_xtm_api.py
tests/test_xtm_translate.py
tests/test_xtm_translations_utils.py
tests/utils.py
tests/xtm_conftest.py
tests/xtm_mock_api.py
tox.ini
View unified diff
|
Download patch
« cms/bin/xtm_translations.py
('K') |
« cms/translations/__init__.py
('k') |
cms/translations/xtm/cli.py »
('j') |
cms/translations/xtm/utils.py »
('J')
Toggle Intra-line Diffs
('i') |
Expand Comments
('e') |
Collapse Comments
('c') |
Show Comments
Hide Comments
('s')
OLD
NEW
(Empty)
(Empty)
OLD
NEW
« cms/bin/xtm_translations.py
('K') |
« cms/translations/__init__.py
('k') |
cms/translations/xtm/cli.py »
('j') |
cms/translations/xtm/utils.py »
('J')
Edit Message
Issue 29886648: Issue #6942 - Add XTM integration in CMS (Closed)
Created Sept. 20, 2018, 4:24 p.m. by Tudor Avram
Modified Oct. 10, 2018, 3:09 p.m.
Reviewers: Vasily Kuznetsov, wspee
Base URL:
Comments: 142
This is Rietveld