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

[16.0][NEW] l10n_br_crm_cnpj_search: busca de CNPJ através do Lead #3289

Draft
wants to merge 8 commits into
base: 16.0
Choose a base branch
from

Conversation

corredato
Copy link
Contributor

A combinação do l10n_br_cnpj_search e l10n_br_crm havia uma ponta solta: Caso necessário, não era possível realizar uma busca das informações de uma empresa através do CNPJ em um lead que não tinha partner_id preenchido, esse módulo faz com que esse problema não exista mais, além de fazer com que informações como CNAE primário, CNAE secundário, capital e natureza legal sejam passadas para o cadastro do parceiro quando um novo contato é criado a partir da conversão de Lead para Oportunidade.

@mileo @antoniospneto @rvalyi

@corredato corredato changed the title [16.0] [NEW] l10n_br_crm_cnpj_search: busca de CNPJ através do Lead [16.0][NEW] l10n_br_crm_cnpj_search: busca de CNPJ através do Lead Aug 16, 2024
Copy link
Member

@rvalyi rvalyi left a comment

Choose a reason for hiding this comment

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

@corredato, tem que ver os testes e inclusive adicionar testes com um mock semelhante ao que foi feito no módulo de busca. No repo precisa de 90% de test coverage. Até bater ter isso é bom botar o PR como rascunho.

@mileo mileo marked this pull request as draft August 22, 2024 13:11
l10n_br_crm_cnpj_search/models/crm_lead.py Outdated Show resolved Hide resolved
@corredato
Copy link
Contributor Author

Depends on #3306

@corredato corredato force-pushed the feature/cnpj-search-crm-lead branch 2 times, most recently from f184f12 to 191eae9 Compare August 29, 2024 18:52
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.

4 participants