Code Review

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

Recent Closed Issues

1 - 20 Older ›

	Subject	Owner	Reviewers	Comments	Last updated
56381002	tryton-env: Allow to pass parent_context when calling get_context (Closed)	ced	pokoli, rietveld-bot, reviewbot	2	22 hours, 34 minutes
64371002	account: Move description of line after the taxes (Closed)	ced	rietveld-bot, reviewbot		22 hours, 36 minutes
66391002	tryton-env: Use add_box as tryton-create icon (Closed)	ced	resteve, rietveld-bot, reviewbot		22 hours, 37 minutes
48611002	trytond: Clean install requirements (Closed)	udono	pokoli, rietveld-bot, ced, reviewbot	2	22 hours, 39 minutes
66381002	tryton: Remove usage of Clipboard set_with_data (Closed)	ced	rietveld-bot, reviewbot		3 days
66371002	tryton: Replace do_start_editing by 'editing-started' signal (Closed)	ced	rietveld-bot, reviewbot		3 days
74341002	tryton-env: Replace self by cls in classmethod (Closed)	ced	pokoli, rietveld-bot, reviewbot		3 days, 9 hours
70371002	trytond: Merge fs_values when extending record from other module (Closed)	ced	pokoli, rietveld-bot, reviewbot		3 days, 9 hours
49571002	tryton-env: Add quick return when evaluate simple expression (Closed)	ced	pokoli, rietveld-bot, albert, reviewbot		3 days, 9 hours
50581002	account_fr: Use date instance instead of string for start and end date (Closed)	ced	pokoli, rietveld-bot, reviewbot		3 days, 9 hours
51531002	sao: Ignore undefined icon name (Closed)	ced	pokoli, rietveld-bot, reviewbot		3 days, 9 hours
56371002	account_invoice: Avoid negative zero as amount to pay (Closed)	ced	rietveld-bot, reviewbot		4 days, 15 hours
52471002	account_invoice: Compare amount in invoice currency against amount_to_pay (Closed)	ced	pokoli, rietveld-bot, reviewbot	1	4 days, 15 hours
45721002	stock: Prevent create move at the closed period date (Closed)	ced	pokoli, rietveld-bot, reviewbot		4 days, 15 hours
72311002	account_fr: Skip line with debit and credit equals to zero in FEC (Closed)	ced	pokoli, rietveld-bot, reviewbot		4 days, 15 hours
52481002	trytond: Update documentation to use "activate-dependencies" instead of "install-dependencies" (Closed)	pokoli	rietveld-bot, ced, reviewbot		4 days, 23 hours
49561002	account_statement: Test if move is set before reading its lines (Closed)	pokoli	rietveld-bot, ced, reviewbot		4 days, 23 hours
42221002	stock: Check stock move if planned_date is equal to stock period date (Closed)	perilla	ced, reviewbot	1	6 days, 14 hours
51511002	tryton: Do not set text property on CellRendererToggle (Closed)	ced	rietveld-bot, reviewbot		6 days, 15 hours
74331002	stock_package_shipping: Add migration for XML id rename of create_shipping button (Closed)	ced	pokoli, rietveld-bot, reviewbot		6 days, 15 hours

1 - 20 Older ›