[MIG] l10n_it_account_stamp: Migrazione a 18.0 #4373
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[FIX] clean [FIX] clean [FIX] pep8 [IMP] use tax obj instead of tax code [FIX] icon path [FIX] get product with lang context [FIX] translation [FIX] product name [FIX] compute after unlink of invoice line
…OCA#818) This is needed as stamp line should be under VAT exemption
Currently translated at 100.0% (29 of 29 strings) Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_account_stamp Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_account_stamp/it/
[FIX] extra blank line [FIX] typos in source strings [IMP] readme, adapt to v.12.0 and add italian section [IMP] __manifest__.py, set ITA name and summary
Currently translated at 100.0% (29 of 29 strings) Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_account_stamp Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_account_stamp/it/
Currently translated at 100.0% (29 of 29 strings) Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_account_stamp Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_account_stamp/it/
[FIX] 2 euro image file
[IMP] image on invoice to display stamp application
and revert checkbox to previous position
[FIX] show button in draft edit mode only
Currently translated at 100.0% (30 of 30 strings) Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_account_stamp Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_account_stamp/it/
Currently translated at 100.0% (30 of 30 strings) Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_account_stamp Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_account_stamp/it/
Currently translated at 100.0% (30 of 30 strings) Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_account_stamp Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_account_stamp/it/
Test case: - Create a product setting VAT "fuori campo" (price 1) - Configure "Tax Stamp 2 euro" product to apply to "fuori campo" - Create an invoice, add a line selecting the previously created product and setting unit price = 100 tax_stamp field stays false
This reverts commit 112146d.
Currently translated at 100.0% (31 of 31 strings) Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_account_stamp Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_account_stamp/it/
Currently translated at 100.0% (30 of 30 strings) Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_account_stamp Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_account_stamp/it/
…lement '<xpath expr=//p[@t-if='o.fiscal_position_id.note']>' cannot be located in parent view
Currently translated at 100.0% (31 of 31 strings) Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_account_stamp Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_account_stamp/it/
…singleton After odoo/odoo@28004b7#diff-a97352a94802dc006fe1dad7587741ccc6e8d9edf5eb70110a1087c52dde2759R97-R109 test_amount_total_changing_currency fails with the following error: Traceback (most recent call last): File "/__w/l10n-italy/l10n-italy/l10n_it_account_stamp/tests/test_account_stamp_invoicing.py", line 93, in test_amount_total_changing_currency invoice_form = Form(invoice) File "/opt/odoo/odoo/tests/common.py", line 1977, in __init__ assert recordp['id'], "editing unstored records is not supported" File "/opt/odoo/odoo/models.py", line 5897, in __getitem__ return self._fields[key].__get__(self, type(self)) File "/opt/odoo/odoo/fields.py", line 5065, in __get__ raise ValueError("Expected singleton: %s" % record) ValueError: Expected singleton: account.move(230, 236) and test_keep_lines_description fails with the following error: Traceback (most recent call last): File "/__w/l10n-italy/l10n-italy/l10n_it_account_stamp/tests/test_account_stamp_invoicing.py", line 73, in test_keep_lines_description self.assertEqual(len(invoice), 1) AssertionError: 2 != 1 and test_post_invoicing fails with the following error: Traceback (most recent call last): File "/__w/l10n-italy/l10n-italy/l10n_it_account_stamp/tests/test_account_stamp_invoicing.py", line 55, in test_post_invoicing self.assertEqual(len(invoice_ids), 1) AssertionError: 2 != 1
Steps: - Configure 2 companies - Company 1 - No tax stamp applied in the settings - Company 2 - Tax stamp applied in the settings - Activate "Synchronize invoices/bills" under "Inter-Company Transactions" in "General Settings" (enterprise feature) - From Company 1, create an invoice to Company 2 and try to "Confirm" the invoice Get "Missing tax stamp product in company settings" Expected behavior for Company 1 (invoice): - Since no taxstamp exists in settings for Company 1 and "Apply tax stamp" is FALSE on the invoice itself - there should be no error "Missing tax stamp". Expected behavior for Company 2 (bill): - Since taxstamp applied in the settings, it should be correctly processed.
If stamp line can be added (invoice in draft): show button to add stamp line. If stamp line can't be added (invoice not in draft): show message explaining why it can't be added. If stamp line has already been added: show message saying it has already been added (useful if invoice has many lines). Button and messages update live while updating the invoice. Better messages to user for charging stamp to customer Co-authored-by: Simone Rubino <[email protected]>
Currently translated at 100.0% (35 of 35 strings) Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_account_stamp Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_account_stamp/it/
|
@SirAionTech
|
This was referenced Sep 13, 2024
primes2h
force-pushed
the
18.0-mig-l10n_it_account_stamp
branch
from
d8aa8f9
to
de646a5
Compare
Aggiunti in de646a5 (da testare). |
Se aspettiamo di poter testare gli script di migrazione per mergiare diventiamo vecchi perché potrebbero volerci mesi prima che Odoo o OpenUpgrade abbiano la migrazione per |
primes2h
force-pushed
the
18.0-mig-l10n_it_account_stamp
branch
from
de646a5
to
7f24e90
Compare
Chiaro, il "da testare" l'ho messo solo come promemoria, per me possiamo fare il merge anche subito dopo la review. |
primes2h
force-pushed
the
18.0-mig-l10n_it_account_stamp
branch
from
7f24e90
to
e3a7d0e
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.