ADD stock_picking_report_by_partner

[angelmoya]

We add new module to asign picking report template to partner, in order to allow set diferents templates for print picking report.

[pedrobaeza]

Same remarks as in the invoice module. But one thing: are you sure you want to limit the print to one record? Picking waves makes a batch printing for example, or you can select multiple documents to print from list view.

[angelmoya]

This method only is used when click button in form view, like in invoices.

[pedrobaeza]

OK, I see. But in any case, if someone wants to call it for multiple reports, why limit it? Is there any real reason for that?

[pedrobaeza]

You should note that the template is used only when you are clicking on the print button on the picking form, not when you click on meny Print > Picking List

[angelmoya]

I only could use this method for one report, if I have more than one picking, with more than one partner, with more than one template... then I can't use this method.

Maybe I could set template in one "master" template, in order to print default in other way... Now I'm re-coding this...

[angelmoya]

No, I can't do in other way, if I could have template like invoices I could inherit this to set de partner template, but in picking I think is not a god idea.

Finally I think this is the better way.

[coveralls]

Coverage remained the same at 66.67% when pulling 0ec9261 on AngelMoya-Domatix:stock_picking_report_by_partner into 9ee1b04 on OCA:8.0.

[pedrobaeza]

OK, thanks for your explanations. Please check travis error.

[laetitia-gangloff]

Can you add Odoo Community Association (OCA) as author ?

[laetitia-gangloff]

Hello,

I don't understand the position of the field picking_report_id.
Why should have pricelist group to be able to set a picking_report ? (Or I missed something ?)

[tafaRU]

Could you please add 'Try me on Runbot' button and 'Bug Tracker' section?
See https://github.com/OCA/maintainer-tools/blob/master/template/module/README.rst as reference.

[angelmoya]

README was updated.

@laetitia-gangloff picking_report_id is after pricelist group, not inside

[laetitia-gangloff]

Other question, I create a report directly in odoo (qweb view and report) and reference it on a supplier.
Then, I try to print this report an I get an error (QWebTemplateNotFound: External ID not found in the system: stock.report_picking_tst) there is something wrong in my configuration ?

[angelmoya]

@laetitia-gangloff sorry I don't know about your error, I was trying on my server with report created on module and works right. I will try it on runbot when starts

[laetitia-gangloff]

@amoya-dx "picking_report_id is after pricelist group, not inside"

I add screen capture :
Without group "Manage pricelist per supplier"

-> no picking_report_id

With group "Manage pricelist per supplier"

-> picking_report_id is available

[angelmoya]

Ok, thanks for this capture, I will fix ASAP

[angelmoya]

Fixed the position in the view.

I was testing this error in runbot, this error appear because no external_id found in the template view, I don't know about create report in web client, but if report is in module it's not a problem.

[laetitia-gangloff]

It seems good now, thank you.
Too bad that is not possible to create report on the fly.

👍

[gurneyalex]

s/diferent/different/
s/report/reports/

[gurneyalex]

small typos in the README; When fixed, I'm 👍

[angelmoya]

Sorry for delay, finally it is fixed

[coveralls]

Coverage decreased (-13.8%) to 86.207% when pulling ae2cdb3 on angelmoya:stock_picking_report_by_partner into 11c4fd5 on OCA:8.0.

[pedrobaeza]

Can you please add tests?

[angelmoya]

Yes, I will do next week

El 16/9/2016 17:29, "Pedro M. Baeza" notifications@github.com escribió:

Can you please add tests?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#16 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGcAhW8JuFIewawWKleO3NJeBPZ9bjObks5qqrXagaJpZM4DiwXL
.

[pedrobaeza]

Closing as no further development. If you want to continue, please reopen it.