OCA / l10n-romania

Romanian Localization for Odoo
GNU Affero General Public License v3.0
27 stars 39 forks source link

[14.0][FIX] account ed ubl: fix parametru filter mesaje spv #1034

Closed danila12 closed 3 weeks ago

OCA-git-bot commented 1 month ago

Hi @feketemihai, @dhongu, some modules you are maintaining are being modified, check this out!

danila12 commented 1 month ago

Stie cineva de unde vine fail-ul din test? Traceback (most recent call last): File "/__w/l10n-romania/l10n-romania/l10n_ro_stock_picking_valued_report/tests/test_purchase_stock_picking_valued.py", line 124, in test_06_po_currency_qty self.assertEqual(agg_lines, exp) AssertionError: {'333[165 chars],), 'hs_code': False, 'currency': 28, 'l10n_ro[170 chars] 28}} != {'333[165 chars],), 'currency': 28, 'l10n_ro_price_unit': 450.[152 chars] 28}} Diff is 732 characters long. Set self.maxDiff to None to see it. Daca facem assertEqual pe fiecare cheie din dict oare mai da eroarea?

feketemihai commented 1 month ago

'hs_code': False,

danila12 commented 1 month ago

'hs_code': False,

Ms.

dhongu commented 1 month ago

/ocabot merge minor

OCA-git-bot commented 1 month ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 14.0-ocabot-merge-pr-1034-by-dhongu-bump-minor, awaiting test results.

OCA-git-bot commented 1 month ago

@dhongu your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-1034-by-dhongu-bump-minor.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

dhongu commented 3 weeks ago

/ocabot merge minor

OCA-git-bot commented 3 weeks ago

On my way to merge this fine PR! Prepared branch 14.0-ocabot-merge-pr-1034-by-dhongu-bump-minor, awaiting test results.

OCA-git-bot commented 3 weeks ago

@dhongu your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-1034-by-dhongu-bump-minor.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.