OCA / l10n-france

France Localization for Odoo
GNU Affero General Public License v3.0
42 stars 114 forks source link

[17.0][WIP] account_factoring_receivable_balance #555

Open dreispt opened 3 months ago

dreispt commented 3 months ago

Port from https://github.com/akretion/odoo-factoring/tree/16.0 and migration to 17.0 Related to https://github.com/akretion/odoo-factoring/issues/20

max3903 commented 2 months ago

@Murtaza-OSI @dreispt I was able to get move lines from invoices and generate the txt file with 101 lines.

I didn't get the move lines from credit notes, so no 102 lines.

Murtaza-SerpentCS commented 2 months ago

@Murtaza-OSI @dreispt I was able to get move lines from invoices and generate the txt file with 101 lines.

I didn't get the move lines from credit notes, so no 102 lines.

@max3903 file have 101 and 102 lines in .TXT file image

max3903 commented 2 months ago

@Murtaza-SerpentCS do you get those lines in the form view after clicking on "Calculate"?

Murtaza-OSI commented 2 months ago

@Murtaza-SerpentCS do you get those lines in the form view after clicking on "Calculate"?

Yes getting line fact

max3903 commented 2 months ago

@Murtaza-OSI Smart button is fixed and lines are correct in line_ids:

Screenshot-from-2024-06-13-20-02-41

CC: @dreispt

bealdav commented 2 months ago

Thanks for porting this module. Nice Improvements

I asked myself if this module, is not generic enough to move an account OCA repository ?

Happy to hear there

bealdav commented 1 month ago

@dreispt as I proposed account_factoring_receivable_balance can move to a non localization repo and other modules could stay here . What do you think ?

But it can be done at the end of your process here

dreispt commented 1 month ago

@bealdav I agree in principle, but in the immediate term it is easier to work and refactor if they are in the same repo. Once this is stabilized and before a merge, we can introduce those changes.