[16.0][IMP] Adicionado cadastro de Natureza Jurídica #3786
Merged
+274
−56
Conversation
This file contains hidden or 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
|
Hi @rvalyi, |
rvalyi
force-pushed
the
16.0-l10n_br_fiscal-legal-nature
branch
from
57314cb to
abdb0cb
Compare
renatonlima
force-pushed
the
16.0-l10n_br_fiscal-legal-nature
branch
4 times, most recently
from
eec90b0 to
aa734c2
Compare
mileo
approved these changes
May 13, 2025
| @@ -1,6 +1,6 @@ | |||
| <?xml version="1.0" encoding="UTF-8" ?> | |||
| <odoo> | |||
| <record model="ir.ui.view" id="crm_lead_cnpj_search_form_view"> | |||
| <record model="ir.ui.view" id="crm_idad_cnpj_search_form_view"> | |||
There was a problem hiding this comment.
Esta certo alterar o ID da view?
There was a problem hiding this comment.
@renatonlima faltou arrumar aqui tbm
l10n_br_fiscal/models/res_partner.py
Outdated
| column1="company_id", | ||
| column2="cnae_id", | ||
| domain="[('internal_type', '=', 'normal'), ('id', '!=', cnae_main_id)]", | ||
| string="Secondary CNAE", |
There was a problem hiding this comment.
Suggested change
| string="Secondary CNAE", | |
| string="Secondary CNAEs", |
por ser um many2many acho que pode ser no plural aqui
renatonlima
force-pushed
the
16.0-l10n_br_fiscal-legal-nature
branch
from
aa734c2 to
f57216a
Compare
|
Eu fiz o rebase da branch agora só tem os commits no PR |
| @@ -1,6 +1,6 @@ | |||
| <?xml version="1.0" encoding="UTF-8" ?> | |||
| <odoo> | |||
| <record model="ir.ui.view" id="crm_lead_cnpj_search_form_view"> | |||
| <record model="ir.ui.view" id="crm_idad_cnpj_search_form_view"> | |||
There was a problem hiding this comment.
@renatonlima faltou arrumar aqui tbm
renatonlima
force-pushed
the
16.0-l10n_br_fiscal-legal-nature
branch
from
f57216a to
c089ed4
Compare
antoniospneto
approved these changes
May 13, 2025
CristianoMafraJunior
approved these changes
May 13, 2025
marcelsavegnago
approved these changes
May 13, 2025
rvalyi
approved these changes
May 13, 2025
|
/ocabot merge minor |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at 13979dd. 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
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.
Esse PR implementa do cadastro de natureza jurídica e também faz algumas mudanças na visão da empresa e adapta o módulo l10n_br_crm_cnpj_search substituindo o campo legal_nature pelo campo relacional legal_nature_id
No cadastro de empresa eu coloquei o grupo do CNAE e a natureza jurídica acima, pois essas seriam informações que deveriam ser preenchida antes dos campos de regime tributário e impostos pois futuramente podemos deduzir ou filtrar o preenchimento de outros campos, como no simples nacional.