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
58491002	tryton-env: Use record and field properties of widget instead of method arguments (Closed)	nicoe	rietveld-bot, ced, reviewbot	15	12 hours, 50 minutes
49721002	account_statement_aeb43: Ensure n43.txt file gets closed in test scenario (Closed)	dave	rietveld-bot, ced, reviewbot	4	1 day, 15 hours
45891002	trytond: Add search and read RPC access to ir.calendar* (Closed)	ced	pokoli, reviewbot		1 day, 15 hours
54431002	stock_lot_unit: rename inventory count field quantity_added to quantity_to_add (Closed)	dave	reviewbot		5 days, 7 hours
49661002	stock: rename field quantity_added to quantity (Closed)	dave	udono, ced, reviewbot	4	5 days, 7 hours
53521002	trytond: Make Cache transactional (Closed)	ced	rietveld-bot, reviewbot		5 days, 17 hours
62391002	trytond: Allow to search and order on Dict key value (Closed)	ced	dave, reviewbot, rietveld-bot, jeremy, pokoli	15	5 days, 18 hours
48691003	tryton-env: Allow to redirect bus request (Closed)	ced	dave, rietveld-bot, nicoe, reviewbot	13	5 days, 18 hours
54461002	sao: Add support for plugins and add translate view as example (Closed)	ced	rietveld-bot, reviewbot		5 days, 19 hours
64521002	trytond: Compute depends on-demand instead of for all callable (Closed)	ced	rietveld-bot, reviewbot		6 days, 5 hours
45851002	tryton-env: Initial commit (Closed)	ced	dave, reviewbot, rietveld-bot, pokoli	39	6 days, 12 hours
50661002	tryton-env: Add extras modules to ModuleTestCase (Closed)	ced	pokoli, rietveld-bot, reviewbot	10	6 days, 12 hours
51591002	account_payment: Add date to Pay Lines wizard (Closed)	ced	pokoli, reviewbot, rietveld-bot, dave	9	6 days, 12 hours
45871002	sao: Add minimal height for graph (Closed)	ced	rietveld-bot, reviewbot		6 days, 12 hours
49691002	account_fr: Use stable order for FEC lines (Closed)	ced	rietveld-bot, reviewbot		6 days, 12 hours
56531002	account_fr: Change accounts 400 and 410 into view (Closed)	ced	rietveld-bot, reviewbot		6 days, 12 hours
45861002	trytond: Remove actual stopped self from the transactions stack (Closed)	ced	pokoli, rietveld-bot, reviewbot	2	6 days, 12 hours
48701002	stock_lot_sled: Search queries inside Select and add different lot instances (Closed)	ced	rietveld-bot, reviewbot		6 days, 12 hours
52591002	trytond: Correctly rename column with sqlite (Closed)	ced	rietveld-bot, reviewbot		1 week, 1 day
50691002	trytond: Add timeout to Report.convert (Closed)	ced	pokoli, rietveld-bot, reviewbot		1 week, 2 days

1 - 20 Older ›