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	Last updated
41611002	trytond: Remove support of Python 3.3 and add Python 3.6 (Closed)	ced	rietveld-bot, reviewbot	11 hours, 32 minutes
40741002	sale_promotion: Add rounding on new unit price (Closed)	ced	pokoli, rietveld-bot, reviewbot	1 day, 14 hours
39751002	trytond: Do not copy fields with using table_query ModelSQL (Closed)	ced	pokoli, rietveld-bot, reviewbot	1 day, 14 hours
35661002	purchase: Fill uom in context when there is not unit (Closed)	ced	rietveld-bot, reviewbot	1 day, 14 hours
39741002	sao: Skip column without field on next_column (Closed)	ced	rietveld-bot, reviewbot	1 day, 14 hours
44361002	sao: Unselect other rows if not ctrl or single selection (Closed)	ced	rietveld-bot, reviewbot	1 day, 14 hours
39731002	account_stock_anglo_saxon: Do not round anglo_saxon quantity (Closed)	ced	rietveld-bot, reviewbot	1 day, 14 hours
36661002	trytond: Enforce required Reference field to have valid record (Closed)	ced	pokoli, rietveld-bot, reviewbot	1 day, 15 hours
39821002	stock_package_shipping_ups: Limit size of fields according to the documentation (Closed)	ced	pokoli, rietveld-bot, reviewbot	1 day, 15 hours
35701002	sale_shipment_cost: Reset carrier to None only when there is no available carriers (Closed)	ced	rietveld-bot, reviewbot	1 day, 15 hours
43611002	sao: Modify loaded record in set_client and set_on_change of O2M (Closed)	ced	rietveld-bot, reviewbot	1 day, 15 hours
39801002	tryton: Modify loaded record in set_client and set_on_change of O2M (Closed)	ced	rietveld-bot, reviewbot	1 day, 15 hours
36701002	trytond: Use the target model name to check access right for many2many field (Closed)	ced	rietveld-bot, reviewbot	1 day, 15 hours
38871002	sao: Test current_view before using it (Closed)	ced	pokoli, rietveld-bot, reviewbot	1 day, 15 hours
35711002	trytond: Use string formating on warn messages (Closed)	pokoli	rietveld-bot, ced, reviewbot	2 days, 11 hours
39831002	account_dunning_email: Catch KeyError on email_template default value (Closed)	pokoli	rietveld-bot, ced, reviewbot	2 days, 11 hours
36721002	sao: Prevent undefined errors on dashboard view (Closed)	pokoli	rietveld-bot, ced, reviewbot	2 days, 11 hours
39811002	tryton: Use dirname of executable for current dir (Closed)	ced	pokoli, rietveld-bot, reviewbot	4 days, 11 hours
38861002	account_fr: Switch kind of escompte (Closed)	ced	rietveld-bot, reviewbot	4 days, 11 hours
40751003	commission: Limit agent field on sale and invoice to the type 'agent' (Closed)	ced	pokoli, rietveld-bot, reviewbot	4 days, 11 hours

1 - 20 Older ›