OCA / l10n-chile

Odoo Chilean Localization

6 stars 36 forks source link

[RFC] l10n_cl_etd_stock #49

Closed max3903 closed 4 years ago

max3903 commented 4 years ago

This change will allow the picking to use the folio from the sequence and comply with the changes listed in #48.

stock.picking

[x] Set the SII Document field as a read-only field
[x] Extend the create method to:
check the partner and its invoicing policy
if invoicing policy is "Electronic Guide"
- Set the SII Document field to "Guia de Despacho Electronica"
- Use the folio on the sequence to generate the document number
otherwise set the SII Document to False

nikitavaghela commented 4 years ago

hello @max3903, can you help me with this two-line what the actual functionality you want by this "Set the SII Document field to "Guia de Despacho Electronica" and "Use the folio on the sequence to generate the document number" for now, I searched the SII document by this name "Guia de Despacho Electronica" and set in class_id field of stock_picking. please correct me if it's wrong here.

max3903 commented 4 years ago

@nikitavaghela

We want to automatically set the "SII Document" field to a specific value "Guia de despacho electronica" if the invoicing_policy of the partner of the picking is "Electronic Guide"
With the same conditions, we want to use the folio on the sequence instead of the Odoo normal sequence.
You are correct.

CC: @nelsonramirezs

nikitavaghela commented 4 years ago

Hello @max3903, I checked the sequence folio functionality. when picking is creating at that time in sequence we have to configure is_dte is True and need to upload CAF file to generate folio sequence. so as per my knowledge these things are dynamic and depend on configuration. please correct me if I'm wrong. Can you please send me the reference or demo CAF file for testing?

max3903 commented 4 years ago

@nikitavaghela You are correct.

@nelsonramirezs Can you provide a demo CAF file?

max3903 commented 4 years ago

@nikitavaghela I sent the demo CAF file to @murtuzasaleh and @JayVora-SerpentCS this morning.

nikitavaghela commented 4 years ago

Hello @max3903, I received a demo CAF file. I tried to upload that file but it raised some traceback like Traceback I attached that screenshot it's an error of l10n_cl_sii_folio module. So what I suggest the folio sequence is configuration part so other than that I already committed code. so can you please review that code?

max3903 commented 4 years ago

@nikitavaghela Please review and merge https://github.com/nikitavaghela/l10n-chile/pull/1