-
-
Notifications
You must be signed in to change notification settings - Fork 251
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
[REF] document_type_id move required attribute from object to view #1062
[REF] document_type_id move required attribute from object to view #1062
Conversation
(o erro do Travis novamente é do teste do payment_cielo da KMEE e nao tem a ver com o PR) |
é legal ser aprovado, mas nao sei se é legal fazer merge com o problema no teste da gateway cielo. Na vdd o teste sempre falharia a gente fazendo o merge ou nao, so nao seria legal passar a mensagem que a gente nao liga para a CI estar OK no projeto... |
@gabrielcardoso21 Você consegue ver esse erro https://travis-ci.com/github/OCA/l10n-brazil/jobs/468217433#L2954 no teste do modulo payment_cielo? Porque seria bom fazer os merges com o travis verde... |
O @DiegoParadeda fez um PR de correção do travis: #1064 |
ace164f
to
3e8215e
Compare
88fcd7b
to
bfe4567
Compare
3e8215e
to
71df066
Compare
7b2f6a6
to
6100848
Compare
71df066
to
e65f344
Compare
e65f344
to
b7b15fd
Compare
/ocabot merge nobump |
On my way to merge this fine PR! |
Congratulations, your PR was merged at bb3df06. Thanks a lot for contributing to OCA. ❤️ |
Movido o atributo requerido do campo document_type_id do objeto para a visão para não impactar a criação de faturas (account.invoice) sem um documento fiscal (apenas relacionada com o l10n_br_fiscal.fiscal_document_dummy).