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

Adicionando externalId na chamada à API - evitar duplicidade de nfse #83

Merged
merged 2 commits into from
Dec 6, 2023

Conversation

luiznfeio
Copy link
Contributor

No description provided.

@andrekutianski
Copy link
Collaborator

andrekutianski commented Dec 5, 2023

@luiznfeio acredito que seja necessário observar a questão do atributo externalId possuir unicidade na API. Nos deparamos com esta questão no módulo do WHMCS nfe/whmcs-addon#110 onde implementamos uma lógica que gera um código único que não conflitaria com qualquer outro sistema ou integração que o cliente possa ter com a NFe.io https://github.com/nfe/whmcs-addon/blob/main/modules/addons/NFEioServiceInvoices/lib/NFEio/Nfe.php#L232C29-L232C29

Seria pertinente termos um processo similar para o Woocommerce?

Referencia: #68

@luiznfeio
Copy link
Contributor Author

bom ponto @andrekutianski , vou implementar semelhante ao que fizemos para o WHMCS

@luiznfeio
Copy link
Contributor Author

@andrekutianski analisei um pouco mais a fundo o funcionamento do plugin da NFE.io. Hoje não temos as mesmas regras que existem no plugin do WHMCS, ou seja, não há emissão de múltiplas notas por código de serviço, nem cálculo de descontos. Sempre a emissão é única por pedido (order) e mesmo que haja diversos itens no pedido, eles serão somados e contemplados numa única nota. Sendo assim, coloquei apenas o prefixo "WOO-NFE-" para identificar nos logs da API da NFE.io que a solicitação veio do plugin. Pode por favor revisar novamente?

@andrekutianski andrekutianski self-requested a review December 6, 2023 02:56
@luiznfeio luiznfeio merged commit 004e60a into master Dec 6, 2023
@luiznfeio luiznfeio deleted the evitando-duplicidade-nfse-via-externalId branch December 6, 2023 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants