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 Open Issues

1 - 20 Older ›

	Subject	Owner	Reviewers	Comments	Last updated
439151003	tryton-env: Add header parameter on export data	pokoli	reviewbot		1 day, 21 hours
433171003	stock: Set default party currency when creating manual move on shipment	mrichez	reviewbot		1 day, 23 hours
421211005	notification_email: Format email addresses with encoding in message header	ced	reviewbot		2 days, 4 hours
423251003	production: Reschedule planned date for late productions	acaubet	pokoli, reviewbot	1	2 days, 5 hours
417221003	tryton-env: Extend also the simple address form	ced	reviewbot		3 days, 3 hours
445091003	bank: Test country code and bank/branch code when guessed BIC is different	ced	reviewbot		4 days, 1 hour
431141003	tryton-env: Round actual quantity after the sum	ced	pokoli, reviewbot	1	1 week
433141003	tryton-env: Select the clicked line when activating a button in list view	nicoe	ced, reviewbot	1	1 week
346041008	tryton-env: Add account_consolidation	jared.esparza	pokoli, ced, dave, reviewbot	193	1 week, 1 day
425191003	account: Remove read access to non account group of computed reporting fields	ced	reviewbot		1 week, 1 day
445081003	account: Enforce statement coherence between children and parent of account type	ced	reviewbot		1 week, 1 day
443121003	tryton-env: Use context model for charts	ced	pokoli, reviewbot	13	1 week, 2 days
421171003	sao: Test if value is not null before using its keys in set_on_change	ced	reviewbot		1 week, 2 days
423091003	tryton-env: Add timezone on cronjobs	nicoe	pokoli, ced, reviewbot	129	1 week, 3 days
419181003	sao: Replace usage of arguments in arrow function	ced	reviewbot		1 week, 3 days
419171003	account: Invert SQL expression of domain_active	ced	reviewbot		1 week, 3 days
427201003	production_outsourcing: Add relate to find purchases of production	acaubet	pokoli, ced, reviewbot	2	1 week, 3 days
447091003	trytond: Update documentation to note priority of attributes over states	acaubet	pokoli, ced, reviewbot	4	1 week, 3 days
382201002	tryton-env: Remove migrations previous to 5.2	acaubet	reviewbot		1 week, 4 days
431111003	bank: Do not fail to compute number record name without bank	ced	reviewbot		1 week, 5 days

1 - 20 Older ›