Skip to content

[19.0][FIX] l10n_es_aeat: Pestaña AEAT en compañía invisible si el país no es España - #4963

Merged
OCA-git-bot merged 1 commit into
OCA:19.0from
moduon:19.0-fix-l10n_es_aeat-page_aeat_company
Apr 17, 2026
Merged

[19.0][FIX] l10n_es_aeat: Pestaña AEAT en compañía invisible si el país no es España#4963
OCA-git-bot merged 1 commit into
OCA:19.0from
moduon:19.0-fix-l10n_es_aeat-page_aeat_company

Conversation

@EmilioPascual

@EmilioPascual EmilioPascual commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

La pestaña AEAT en compañía sólo debe ser visible si el país es España.

Ahora está así
image

Con el cambio queda así:
image
image

@ArantxaSudon @OCA/local-spain-maintainers por favor revisad. Gracias

MT-14056 @moduon

@EmilioPascual
[FIX] l10n_es_aeat: Invisible page aeat in company if country is not …
4526d17 
…Spain

Page aeat must be visible only for counrty Spain

MT-14056 @moduon
@OCA-git-bot

OCA-git-bot commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

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

@pedrobaeza pedrobaeza changed the title [FIX] l10n_es_aeat: Pestaña AEAT en compañía invisible si el país no es España [19.0][FIX] l10n_es_aeat: Pestaña AEAT en compañía invisible si el país no es España Apr 17, 2026
@pedrobaeza pedrobaeza added this to the 19.0 milestone Apr 17, 2026
pedrobaeza
pedrobaeza requested changes Apr 17, 2026
View reviewed changes

@pedrobaeza pedrobaeza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Esto no es correcto. Tú puedes necesitar poner identificación AEAT independientemente del país del contacto. Lo que sí que se podría ocultar es si el país de la compañía no es "España".

@EmilioPascual

EmilioPascual commented Apr 17, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Esto no es correcto. Tú puedes necesitar poner identificación AEAT independientemente del país del contacto. Lo que sí que se podría ocultar es si el país de la compañía no es "España".

Es lo que hace el PR, oculta la pestaña AEAT en la compañía si el país de la compañía no es España

@pedrobaeza

pedrobaeza commented Apr 17, 2026

Copy link
Copy Markdown
Member

Me refiero a res.company.

@EmilioPascual

EmilioPascual commented Apr 17, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Me refiero a res.company.

Creo que no te estoy entendiendo. La vista que modifico es la de res.company. Si tu compañía tiene como país Francia no debería de verse la pestaña AEAT en el form de la compañia.

Creo que has pensado que lo he hecho en res.partner

@pedrobaeza

pedrobaeza commented Apr 17, 2026

Copy link
Copy Markdown
Member

Joer, disculpa, creía que era la de res.partner. Fallo mío.

pedrobaeza
pedrobaeza approved these changes Apr 17, 2026
View reviewed changes

@pedrobaeza pedrobaeza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/ocabot merge patch

@OCA-git-bot

OCA-git-bot commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 19.0-ocabot-merge-pr-4963-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit fe44007 into OCA:19.0 Apr 17, 2026
7 checks passed
@OCA-git-bot

OCA-git-bot commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

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

@EmilioPascual EmilioPascual mentioned this pull request Apr 17, 2026
Merged
@EmilioPascual
EmilioPascual deleted the 19.0-fix-l10n_es_aeat-page_aeat_company branch April 17, 2026 08:11
This was referenced Apr 17, 2026
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pedrobaeza pedrobaeza pedrobaeza approved these changes

Assignees

No one assigned

Labels

merged 🎉 mod:l10n_es_aeat Module l10n_es_aeat series:19.0

Projects

None yet

Milestone

19.0

Development

Successfully merging this pull request may close these issues.

3 participants

@EmilioPascual @OCA-git-bot @pedrobaeza