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
431161003	tryton-tweet: Update broken links (Closed)	acaubet	rietveld-bot, ced, reviewbot	3	1 day, 4 hours
427221003	party: Avoid duplicated emails in party and contact mechanism default email (Closed)	pokoli	rietveld-bot, ced, reviewbot	7	1 day, 21 hours
433161003	trytond: Do not copy avatars nor uuid (Closed)	acaubet	rietveld-bot, ced, reviewbot		1 day, 21 hours
370091002	tryton-env: Use field's description instead of name in header when exporting from the print toolbar (Closed)	Davidoff	pokoli, ced, reviewbot	31	1 day, 21 hours
427161003	stock_supply: Only delete draft requests when generating new ones (Closed)	pokoli	acaubet, rietveld-bot, ced, reviewbot	2	1 week
423201003	sao: Ensure more-cell always have a colspan attribute (Closed)	raphael42	rietveld-bot, ced, reviewbot		1 week
394211002	account_move_line_grouping: Hide empty grouped lines on move (Closed)	ced	rietveld-bot, reviewbot		1 week
403451002	account: Add miscellaneous journal (Closed)	ced	rietveld-bot, reviewbot		1 week
405311003	trytond: Add cache on list icons (Closed)	ced	pokoli, rietveld-bot, albert, reviewbot		1 week
378791003	trytond: Do not depend on the context for get_language cache (Closed)	ced	rietveld-bot, albert, reviewbot		1 week
356441002	account_es_sii: Remove uppercase to invoice in help text (Closed)	ced	pokoli, rietveld-bot, reviewbot		1 week
413141003	stock_supply: Skip product supplier without lead time to compute supply dates (Closed)	ced	pokoli, rietveld-bot, reviewbot		1 week
449081003	trytond: Ignore invalid Reference field when reading related fields (Closed)	ced	rietveld-bot, reviewbot		1 week
443091003	trytond: Do not log HTTPException in with_transaction (Closed)	ced	rietveld-bot, reviewbot		1 week
421151003	tryton-env: Correctly complete after opening OR-ed parenthesis (Closed)	ced	rietveld-bot, reviewbot		1 week
411171004	tryton-env: Test for sub-domain instead of not leaf in domain parser (Closed)	ced	rietveld-bot, reviewbot		1 week
443071003	sao: Set loaded value without triggering modification (Closed)	ced	rietveld-bot, reviewbot		1 week, 5 days
447071003	stock: Rename inventory count button action (Closed)	EdbO	rietveld-bot, ced, reviewbot		1 week, 5 days
427091003	product_image: Manage index argument to construct image URL (Closed)	ced	rietveld-bot, reviewbot		1 week, 5 days
427131003	account_stock_landed_cost: Clear invoice lines when copying landed cost (Closed)	mrichez	rietveld-bot, ced, reviewbot		1 week, 5 days

1 - 20 Older ›