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][l10n_br_nfe] #2874

Merged
merged 1,166 commits into from
Jul 24, 2024
Merged

[15.0][MIG][l10n_br_nfe] #2874

merged 1,166 commits into from
Jul 24, 2024

Conversation

rvalyi
Copy link
Member

@rvalyi rvalyi commented Jan 17, 2024

@rvalyi rvalyi marked this pull request as draft January 17, 2024 18:32
@rvalyi rvalyi force-pushed the 15.0-mig-l10n_br_nfe branch 13 times, most recently from 9fb961f to 5318550 Compare January 19, 2024 14:54
@renatonlima renatonlima added this to the 15.0 milestone Feb 1, 2024
@rvalyi rvalyi force-pushed the 15.0-mig-l10n_br_nfe branch 6 times, most recently from 1f74442 to 9d1a345 Compare May 23, 2024 12:04
@rvalyi rvalyi mentioned this pull request May 23, 2024
2 tasks
@rvalyi rvalyi force-pushed the 15.0-mig-l10n_br_nfe branch 3 times, most recently from 01755f8 to a8a10ad Compare May 23, 2024 15:24
@rvalyi rvalyi force-pushed the 15.0-mig-l10n_br_nfe branch 3 times, most recently from 1649f36 to 362a42e Compare June 11, 2024 15:39
@renatonlima
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 15.0-ocabot-merge-pr-2874-by-renatonlima-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Jul 24, 2024
Signed-off-by renatonlima
@OCA-git-bot
Copy link
Contributor

@renatonlima your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-2874-by-renatonlima-bump-nobump.

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.

@rvalyi
Copy link
Member Author

rvalyi commented Jul 24, 2024

afff deu ruim na geraçao do .pot:

Traceback (most recent call last):
  File "/opt/odoo-venv/lib/python3.8/site-packages/click_odoo/env_options.py", line 191, in _invoke
    with self.environment_manager(
  File "/usr/lib/python3.8/contextlib.py", line 113, in __enter__
    return next(self.gen)
  File "/opt/odoo-venv/lib/python3.8/site-packages/click_odoo/env.py", line 15, in OdooEnvironment
    registry = odoo.registry(database)
  File "/opt/odoo/odoo/__init__.py", line 126, in registry
    return modules.registry.Registry(database_name)
  File "/opt/odoo/odoo/modules/registry.py", line 63, in __new__
    return cls.new(db_name)
  File "/opt/odoo/odoo/modules/registry.py", line 87, in new
    odoo.modules.load_modules(registry, force_demo, status, update_module)
  File "/opt/odoo/odoo/modules/loading.py", line 479, in load_modules
    registry.setup_models(cr)
  File "/opt/odoo/odoo/modules/registry.py", line 279, in setup_models
    model._prepare_setup()
  File "/opt/odoo/odoo/models.py", line 2983, in _prepare_setup
    cls.__bases__ = cls.__base_classes
TypeError: Cannot create a consistent method resolution
order (MRO) for bases BaseModel, base, NfeSpecMixin
Error: Cannot create a consistent method resolution
order (MRO) for bases BaseModel, base, NfeSpecMixin

@rvalyi rvalyi force-pushed the 15.0-mig-l10n_br_nfe branch from 81be3ab to 7cabfe5 Compare July 24, 2024 01:48
@rvalyi
Copy link
Member Author

rvalyi commented Jul 24, 2024

Com esse novo commit, eu estou tentando reverter em parte o commit que eu tinha feito semana passada aqui 8609ffa#diff-4970abfd213d7bb8083244ff7e51d9e73b4f3ec22a14018ed9b1d28bccbe20c6R32 para ver se eh isso que cria problema com a geraçao dos arquivos .pot

@rvalyi
Copy link
Member Author

rvalyi commented Jul 24, 2024

Com esse novo commit, eu estou tentando reverter em parte o commit que eu tinha feito semana passada aqui 8609ffa#diff-4970abfd213d7bb8083244ff7e51d9e73b4f3ec22a14018ed9b1d28bccbe20c6R32 para ver se eh isso que cria problema com a geraçao dos arquivos .pot

Nao eh isso nao. Eh algo que eu mudei recentemente na migracao do spec_driven_model. Louco que passava todos testes, mas bloca o Runboat tb. Tou resolvendo...

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@rvalyi rvalyi force-pushed the 15.0-mig-l10n_br_nfe branch from 659ac98 to a24ecfb Compare July 24, 2024 07:37
hint: in NFe35200159594315000157550010000000012062777161.xml the E-COM11
product got a price change in v15; also replaced spaces by tabs.

[TMP] attempt to fix v15 issue
@rvalyi rvalyi force-pushed the 15.0-mig-l10n_br_nfe branch from a24ecfb to 1ea6d37 Compare July 24, 2024 07:51
@rvalyi
Copy link
Member Author

rvalyi commented Jul 24, 2024

o problema foi resolvido com 48dea75

@rvalyi
Copy link
Member Author

rvalyi commented Jul 24, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit bd7ac77 into OCA:15.0 Jul 24, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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