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
46801002	tryton: Avoid comparison between tuple and int in one2many position (Closed)	ced	rietveld-bot, reviewbot		3 days, 21 hours
58541002	tryton: Try to use native time zone of Windows (Closed)	ced	rietveld-bot, reviewbot		3 days, 21 hours
47861002	tryton: Allow to click button on readonly record in tree view (Closed)	ced	pokoli, rietveld-bot, reviewbot		3 days, 21 hours
70591002	sao: Evaluate DateTime PYSON in UTC (Closed)	ced	rietveld-bot, reviewbot		3 days, 21 hours
62441002	sao: Polish global search (Closed)	ced	pokoli, rietveld-bot		3 days, 21 hours
45961002	tryton-env: Simplify xmlrpc marshallers for Python3 (Closed)	ced	pokoli, rietveld-bot, reviewbot	2	3 days, 21 hours
50791002	sale: Do not fill pygal chart with None value (Closed)	ced	pokoli, rietveld-bot, reviewbot		3 days, 21 hours
259091002	account_stock_anglo_saxon: Round cost with company instead of invoice currency (Closed)	prakash.pandey	rietveld-bot, ced, reviewbot		3 days, 21 hours
48741002	account_dunning_letter: Update state test from done to waiting (Closed)	smn	rietveld-bot, ced, reviewbot	2	3 days, 21 hours
54551002	account: Do not recreate models that have template override is True (Closed)	ced	pokoli, rietveld-bot, reviewbot	4	3 days, 21 hours
269141003	trytond: Wait up to 10 seconds for cache sync in tests (Closed)	ced	rietveld-bot, reviewbot		4 days, 5 hours
52631002	tryton-docker: Add pygal dependency (Closed)	pokoli	rietveld-bot, ced	3	5 days, 2 hours
72421002	account_invoice: Use selection widget from payment method and writeoff on pay wizard (Closed)	pokoli	rietveld-bot, ced, reviewbot		5 days, 2 hours
58531002	stock_supply: Use OR clauses on location searcher (Closed)	pokoli	rietveld-bot, ced, reviewbot	2	5 days, 2 hours
50781002	stock_supply: Follow other modules design on order point rec_name (Closed)	pokoli	rietveld-bot, ced, reviewbot		5 days, 2 hours
255111002	account_es: Fix typo in non subjected operations tax code (Closed)	pokoli	rietveld-bot, ced, reviewbot		5 days, 3 hours
68451002	product_price_list_parent: Add parent_unit_price to help field of formula (Closed)	ced	rietveld-bot, reviewbot		5 days, 7 hours
46731002	trytond: Allow to extend Field string and help (Closed)	ced	rietveld-bot, albert, reviewbot		5 days, 7 hours
273061002	tryton: Add shortcuts for switch and remove on One2Many (Closed)	ced	rietveld-bot, reviewbot		5 days, 7 hours
56641003	tryton: Resize wizard dialog on show (Closed)	ced	rietveld-bot, reviewbot		5 days, 7 hours

1 - 20 Older ›