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] intrastat_product: Migration to 15.0 #187

Merged
merged 82 commits into from
Sep 12, 2022

Conversation

victoralmau
Copy link
Member

@victoralmau victoralmau commented Aug 3, 2022

Migration to 15.0

Locked by:

Superseed #165

@Tecnativa TT36959

@pedrobaeza
Copy link
Member

/ocabot migration intrastat_product

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Aug 3, 2022
@OCA-git-bot OCA-git-bot mentioned this pull request Aug 3, 2022
5 tasks
@pedrobaeza
Copy link
Member

Please rebase

@victoralmau
Copy link
Member Author

Rebase done, it's ready to review

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Please update at least Spanish translation

@victoralmau victoralmau force-pushed the 15.0-mig-intrastat_product branch 2 times, most recently from d1423cd to fe66d7b Compare August 12, 2022 08:10
@victoralmau
Copy link
Member Author

Spanish translation updated.

@pedrobaeza
Copy link
Member

@luc-demeyer @alexis-via is it ok for you?

Alexis de Lattre and others added 17 commits September 12, 2022 12:31
Rename report.intrastat.product to l10n.fr.report.intrastat.product (same for service and for lines) + migration scripts

Rename field country_id to origin_country_id (TODO : write mig script) and move this field from l10n_fr_intrastat_product to intrastat_product

Move intrastat_code and uom_id of report.intrastat.code from l10n_fr_intrastat_product to intrastat_product, as discussed with Luc de Meyer

Use display_name instead of an inherit of name_get (better use of new API) Inherit write and create of intrastat codes to remove spaces (patch by Luc De Meyer from Noviat) Add O2M fields from intrastat code to products and product categs
…cant, for instance, import files with name_search
* Update README.rst and translation files
* Add index=True on some fields on which we often search, as suggested by Yannick
* Remove description in __openerp__.py
* Move ACL from l10n_fr_intrastat_product module to intrastat_product module
* Large code re-engineering following the Akretion-Noviat code sprint on intrastat
  This code has been written both by Luc de Meyer and myself.
* Add first draft of code to generate decl lines
* Add product_origin_country_id on declaration/computation lines

  Copy incoterms and destination country from SO to invoice when invoicing from SO
  We need weight even when supplementary units is used
  Small cleanups and enhancements

* Add support for accessory costs

  Add default values for intrastat transaction on company
  Code cleanup

* If rounded weight is 0, put 1

  Take into account the taxes for B2C
  Small code cleanup

* Remove field exclude_from_intrastat

  Re-organise view of intrastat.product.declaration

* Add option intrastat_accessory_costs on company

  Set more fields as invisible (localisation should put them visible if they need it)
  Fix handling of suppl. units when hs_code is empty on invoice line (but set on product)
  Small usability enhancements

* Include selection of type of invoice in search, for better perf

  Isolate domain in a _prepare method, for easier inheritance
  WARNING: I changed the default selection of invoice type ; adapt it in your localization m
  odule if necessary
  Add intrastat_transaction_in_refund for company (not needed for France, but may be needed elsewhere)
  Add a log when an invoice line is skipped

* Include product code in warning msg on weight

* Inspired by the PR akretion/account-financial-reporting#8 of Luc de Meyer
* Improve logs and messages
* total_amount is a sum of integers, so it should be an integer
* Add transport mode in computation tree view
* FIX intrastat_country for invoices without src_dest_country_id
* FIX wrong model for seach method
* Use stock_picking_invoice_link for a better identification of the intrastat region

  With this commit, we now support the following scenario: I order to my supplier a quantity of 50 kg and he delivers/invoices 52kg ; odoo will create an additional invoice line of 2kg which is linked to the stock move, but not to any PO line.
* Modularise a piece of code
* Add ACL on hs.code to financial manager (I can't do it in product_harmonized_system because it doesn't depend on account)
* Handle scenario where an invoice has products with 0 value (samples for example) and shipping costs (accessory costs) with value > 0.
Because we can have intrastat unit that will never be mapped to an odoo unit (will always be set manually)
intrastat_base: Move company view params to account config page
rousseldenis and others added 22 commits September 12, 2022 12:35
As Sale and Purchase users have access to invoices, we need to allow them
to read intrastat move lines as present on invoices.
…_origin_country_code to avoid breaking the localisation modules
[UPD] Update intrastat_product.pot

intrastat_product 14.0.1.6.0

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: intrastat-extrastat-14.0/intrastat-extrastat-14.0-intrastat_product
Translate-URL: https://translation.odoo-community.org/projects/intrastat-extrastat-14-0/intrastat-extrastat-14-0-intrastat_product/
[UPD] Update intrastat_product.pot

intrastat_product 14.0.1.6.1

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: intrastat-extrastat-14.0/intrastat-extrastat-14.0-intrastat_product
Translate-URL: https://translation.odoo-community.org/projects/intrastat-extrastat-14-0/intrastat-extrastat-14-0-intrastat_product/
Currently translated at 0.7% (2 of 261 strings)

Translation: intrastat-extrastat-14.0/intrastat-extrastat-14.0-intrastat_product
Translate-URL: https://translation.odoo-community.org/projects/intrastat-extrastat-14-0/intrastat-extrastat-14-0-intrastat_product/es/
intrastat_product 14.0.1.6.2
TT37168

[UPD] Update intrastat_product.pot

[UPD] README.rst

intrastat_product 14.0.2.0.0

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: intrastat-extrastat-14.0/intrastat-extrastat-14.0-intrastat_product
Translate-URL: https://translation.odoo-community.org/projects/intrastat-extrastat-14-0/intrastat-extrastat-14-0-intrastat_product/
Accidentally removed in OCA@2646dc4

TT38922

intrastat_product 14.0.2.0.1
@pedrobaeza
Copy link
Member

Let's merge this finally for unblocking other dependent things:

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-187-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit f434465 into OCA:15.0 Sep 12, 2022
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 092d3bb. 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.

None yet