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
36211002	tryton: Implement workaround for GTK bug with InfoBar and revealer (Closed)	ced	resteve, rietveld-bot, reviewbot		1 hour, 37 minutes
45391002	sao: Test the length of the Array before applying map/reduce (Closed)	ced	rietveld-bot, reviewbot		1 hour, 40 minutes
50271002	analytic_invoice: Add asset.xml file on tryton.cfg (Closed)	pokoli	rietveld-bot, ced, reviewbot		10 hours, 36 minutes
47331002	sao: Move max-width CSS Integer to apply only on form (Closed)	ced	pokoli, rietveld-bot, reviewbot	1	1 day, 2 hours
36191002	sao: Replace Model.find call in Screen.search_filter by direct call to search (Closed)	ced	pokoli, rietveld-bot, reviewbot		1 day, 2 hours
36171002	account_stock_anglo_saxon: Fill party if account has party required (Closed)	ced	pokoli, rietveld-bot, reviewbot		1 day, 2 hours
36161002	sao: Use ISO-8601 format for toString call on Date and DateTime (Closed)	ced	rietveld-bot, reviewbot		1 day, 2 hours
41251002	analytic_account: Quantize to number of digits instead of rounding computed amount (Closed)	ced	pokoli, rietveld-bot, reviewbot		1 day, 16 hours
37251002	account: Quantize to number of digits instead of rounding computed amount (Closed)	ced	pokoli, rietveld-bot, reviewbot		1 day, 16 hours
42091002	account_payment: Add payment_blocked to move_line_list (Closed)	ced	pokoli, rietveld-bot, reviewbot		1 day, 16 hours
49261002	sao: Synchronize the Date Dict widget behavior with the Date widget (Closed)	ced	pokoli, rietveld-bot, reviewbot		1 day, 16 hours
47341002	trytond: Fix class name for ModelSQL.search documentation (Closed)	ced	rietveld-bot, lukio, reviewbot		1 day, 16 hours
44971002	proteus: Support Decimal in PYSONEncoder (Closed)	ced	pokoli, rietveld-bot, reviewbot		2 days, 8 hours
41241002	tryton: Support Decimal in PYSONEncoder (Closed)	ced	pokoli, rietveld-bot, reviewbot		2 days, 8 hours
38161002	account: Ensure None are before 0 (Closed)	ced	pokoli, rietveld-bot, reviewbot	3	2 days, 8 hours
44961002	stock: Add missing space on pending shipment error message (Closed)	pokoli	rietveld-bot, ced, reviewbot		5 days, 9 hours
45371002	trytond: Add Persian (Closed)	ced	rietveld-bot, reviewbot		5 days, 9 hours
40181002	account_stock_landed_cost: Skip move with no quantity to allocate cost (Closed)	prakashpp.pandey	rietveld-bot, ced, reviewbot	1	6 days, 6 hours
45271002	currency: Re-enable RPC to compute method (Closed)	prakashpp.pandey	rietveld-bot, ced, reviewbot	1	6 days, 6 hours
38111002	sao: Disable button if parent is changed and ensure states is boolean (Closed)	ced	rietveld-bot, reviewbot		6 days, 6 hours

1 - 20 Older ›