Skip to content
This repository was archived by the owner on Oct 22, 2024. It is now read-only.

Conversation

@michelts
Copy link
Contributor

De acordo com a documentação, número e série da nota são obrigatórios
para PAC, por isso, abri a possibilidade de especificá-los.

De acordo com a documentação, número e série da nota são obrigatórios
para PAC, por isso, abri a possibilidade de especificá-los.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 60.494% when pulling 6f801d2 on michelts:dados-da-nota into 394ba58 on mstuttgart:develop.

@mstuttgart
Copy link
Owner

mstuttgart commented Jan 18, 2017

@michelts será que ha alguma maneira de deixar isso dinâmico? Assim como você comentou, o número e série da Nota Fiscal são obrigatórios apenas para PAC, porém para outros tipos de pacotes será necessário informá-los mesmo que não seja obrigatórios.

@michelts
Copy link
Contributor Author

@mstuttgart Vi que mesclou agora. Se não me engano o jinja vai colocar uma string vazia no lugar. Eu vou escrever testes de qualquer maneira, só não o fiz por que a suite de testes precisa de uma revisão mais ampla.

@michelts
Copy link
Contributor Author

Obrigado ;)

@mstuttgart
Copy link
Owner

@michelts entendi. Sem problema. Obrigado pela contribuição!

@michelts michelts deleted the dados-da-nota branch January 18, 2017 13:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants