I think we shouldn't set the tracking number readonly once the picking is done.
My rationale: if it makes sense for carriers which get the tracking number from the API, one may still want to handle some carrier manually and would still need to edit the field afterwards.
pdf_list is not used, and a semantic detail: not only pdf could be returned (zpl for instance):
pdf_list = []
for pick in pickings:
pdf = pick.generate_single_label() pdf_list.append(pdf)
In the access rights files, shouldn't it be stock.group_stock_user and stock.group_stock_manager instead of sales user / manager?
Thanks!
I pushed some change regarding the pep-8 and pep-257 (docstrings).
On this change:
I think we shouldn't set the tracking number readonly once the picking is done.
My rationale: if it makes sense for carriers which get the tracking number from the API, one may still want to handle some carrier manually and would still need to edit the field afterwards.
pdf_list is not used, and a semantic detail: not only pdf could be returned (zpl for instance):
pdf_list = [] single_ label()
pdf_ list.append( pdf)
for pick in pickings:
pdf = pick.generate_
In the access rights files, shouldn't it be stock.group_ stock_user and stock.group_ stock_manager instead of sales user / manager?