Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[15.0][MIG] DES #339

Merged
merged 109 commits into from
Dec 3, 2022
Merged

[15.0][MIG] DES #339

merged 109 commits into from
Dec 3, 2022

Conversation

alexis-via
Copy link
Contributor

No description provided.

Copy link
Contributor

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Diff review.
LGTM.
Thanks.

@legalsylvain
Copy link
Contributor

Hi. For your information CI is red because the module instrastat_base is not available. (OCA/intrastat-extrastat#157 is not merged)
OCA is removing Travis and now uses github actions.

  • OCA merged modules are now automatically found without any actions, via pypi (that's great!)
  • OCA unmerged modules like instrastat_base can be referenced temporary in the test-requirements.txt file in the root folder.

Detail of the syntax available here : OCA/oca-ci#23 (comment)

regards.

@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label May 29, 2022
@alexis-via
Copy link
Contributor Author

If someone knows how we can force Travis to install a decent version of stdnum, it would allow to merge this PR, which would be great...

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jun 5, 2022
@github-actions
Copy link

github-actions bot commented Oct 9, 2022

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Oct 9, 2022
@alexis-via
Copy link
Contributor Author

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

@alexis-via The rebase process failed, because command git push --force akretion tmp-pr-339:15-mig-des failed with output:

remote: Permission to akretion/l10n-france.git denied to OCA-git-bot.
fatal: unable to access 'https://github.com/akretion/l10n-france/': The requested URL returned error: 403

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Oct 16, 2022
Alexis de Lattre added 18 commits November 23, 2022 21:40
… Check that company currency is EUR when generating service lines. Do the rounding work when generating the service lines, not when generating the XML. Remove unused code.
…rst factorisation of common functionnal code between the product and service module (intrastat_common.py) - to be continued. Some code clean-up in the service module.
…nsport type" and partial support for the department. More factorisation between product & service module. Add date_done field. More checks.
… (fields that are not in the official declaration or never subject to change) Add copy of department from warehouse to invoice. Import DEB don't have the partner_vat field Department can be 99 for Monaco. Copy tranport and department from report_intrastat_product in XML file First attempt at adapting the display of fields depending on the type of declaration.
…tat_product/service object is stored in database, we don't need the legal_intrastat state on invoices any more, because we will be able to create entry lines for DEB for a repair of equipment under warranty without using a legal_intrastat. Report_intrastat_type is now only required for DEB -> object moved to module l10n_fr_intrastat_product. For DEB, the button "regenerate lines" only regenerate DEB lines related to invoices.

Renamed intrastat base module, because it doesn't have France-specific parameters any more.

Add demo data.
…rocedure_code, for example some info don't need to be declared for some procedure codes. Ability to enter DEB lines manually Stat value doesn't exist any more Tree vue of DEB lines is now identical to Cerfa
…port_intrastat_type table, restarting OpenERP and re-creating intrastat types. Moved intrastat departments from stock.warehouse to stock.location. Dropped SQL queries ; replaced by traditionnal python code logic. No more need to have one rate per day for invoices with foreign currency. Add total fiscal value for DEB More code factorization. Prepare translation work.
… Moved configuration about taxes from company form to tax form Some modifications to ease v5 -> v6 migration : - object report_intrastat_code now belong to group account manager - button functions now return True Tried to implement the following feature : open attachement form when the XML file as been generated : works on v6, but make client crash en v5 -> code has been commented DEB lines with procedure code = 25 are now deducted from the fiscal total. Round invoice total.
… l10n_fr_intrastat_product to intrastat_base, because it should also be used in the module l10n_fr_intrastat_service. Take this field into account in the generation of DEB lines (module l10n_fr_intrastat_service).
@alexis-via
Copy link
Contributor Author

rebased

@alexis-via
Copy link
Contributor Author

Seems to be caused by the fact that requirements.txt from odoo v15 asks for python-stdnum 1.13 and 1.13 doesn't have vatin :-( I see no other reason.

@alexis-via
Copy link
Contributor Author

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-339-by-alexis-via-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 054879b into OCA:15.0 Dec 3, 2022
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at fa39399. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants