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
31191002	purchase_request: Set process_request wrapper before Workflow.transition wrapper (Closed)	mrichez	pokoli, ced, reviewbot		20 hours, 29 minutes
43191002	sao: Remove unnecessary placeholder (Closed)	albert	ced, reviewbot		1 week, 2 days
40011002	trytond: Do not register tests module on non test pools (Closed)	pokoli	timitos3000, ced, reviewbot	5	1 week, 2 days
367501002	tryton-env: Lock Period Journal table when checking their existence (Closed)	nicoe	ced, reviewbot	5	1 week, 4 days
445351003	timesheet_cost: Add salary scale to employees (Closed)	pokoli	reviewbot		1 week, 5 days
345601002	account_payment: Add window domain on payment group (Closed)	acaubet	reviewbot		1 week, 6 days
382201002	tryton-env: Remove migrations previous to 5.0 (Closed)	acaubet	reviewbot		1 week, 6 days
359611002	sao: Use search input for search boxes (Closed)	acaubet	reviewbot		1 week, 6 days
423251003	production: Reschedule planned date for late productions (Closed)	acaubet	pokoli, ced, reviewbot	2	2 weeks
437511020	account_es: Add intracomunitary services on intracomunitary iva devengado (Closed)	acaubet	pokoli, rietveld-bot, reviewbot		2 weeks
338111002	tryton-env: New Romanian chart of accounts module (Closed)	dotbit	pokoli, dave, reviewbot	11	2 weeks, 1 day
370771002	account: Clear fiscalyear when changing reporing context company (Closed)	pokoli	reviewbot		2 weeks, 2 days
366591002	production: Support Phantom BoM (Closed)	pokoli	ced, reviewbot	17	2 weeks, 2 days
387961002	trytond: Allow to use date format syntax in prefixes and suffixes of sequences (Closed)	Davidoff	pokoli, ced, reviewbot	16	2 weeks, 2 days
328331003	stock: Add help_selection to states (Closed)	dave	pokoli, reviewbot	6	2 weeks, 4 days
287741002	tryton-env: Add account_uk module (Closed)	dave	pokoli, reviewbot	15	2 weeks, 4 days
328891002	trytond: Add res module documentation (Closed)	dave	udono, ced, reviewbot	28	2 weeks, 5 days
421981003	www.tryton.org: Add buchkontor teltow and Inmedio Berlin success stories (Closed)	pokoli	ced, reviewbot		2 weeks, 6 days
19181002	account_invoice_history:Add scenario test (Closed)	udono	pokoli, reviewbot	13	3 weeks, 2 days
58351002	trytond: Add help text (Closed)	udono	pokoli, dave, reviewbot	34	3 weeks, 2 days

1 - 20 Older ›