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.

Sign up for GitHub

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

Jump to bottom

Feature/dfe mdfe #928

Merged
merged 33 commits into from
Sep 28, 2020
Merged

Feature/dfe mdfe #928

merged 33 commits into from
Sep 28, 2020

Conversation

gabrielcardoso21
Copy link
Contributor

Modelos e campos relacionados à DF-e e MDF-e sem a transmissão.

@gabrielcardoso21 gabrielcardoso21 force-pushed the feature/dfe_mdfe branch 3 times, most recently from 53de475 to 53f3b51 Compare September 1, 2020 18:34
rvalyi and others added 25 commits September 25, 2020 14:50
@rvalyi @gabrielcardoso21
usability: view_xml creates the XML file if not generated yet
79e2052
@hugouchoasborges @gabrielcardoso21
[ADD] dfe_mdfe views
a9e0f1d
@hugouchoasborges @gabrielcardoso21
[IMP] Propriedades dos campos pelo XML
91f3a26
@hugouchoasborges @gabrielcardoso21
[ADD] DF-e search documents
7e282cf
@gabrielcardoso21
[FIX] Baixa XML / CIência / Confirma / Nega / Desconhece Operação
372934a
@gabrielcardoso21
[FIX] Manage Manifestations Button
693fc4a
@hugouchoasborges @gabrielcardoso21
[FIX] Raise Errors
64c7721
@hugouchoasborges @gabrielcardoso21
[IMP] search_documents raise True
0cac7d7
@hugouchoasborges @gabrielcardoso21
[REF] Campos e métodos
6134496
@hugouchoasborges @gabrielcardoso21
[ADD] modelo l10n_br_fiscal.attachment
812f834
@hugouchoasborges @gabrielcardoso21
[IMP] res_company nfe_environment
54d0fb8
@hugouchoasborges @gabrielcardoso21
[ADD] Incomprehensible Return from DF-e webservice
424126e
@hugouchoasborges @gabrielcardoso21
[ADD] mdfe constants
9fb4295
@hugouchoasborges @gabrielcardoso21
[IMP] nfe_environment
207b06a
@hugouchoasborges @gabrielcardoso21
[REF] tmpFile -> BytesIO
5fa951e
@hugouchoasborges @gabrielcardoso21
[ADD] force_ambiente. Controle de ambiente da NF-e para testes
72d407d
@hugouchoasborges @gabrielcardoso21
[REF] Imported Documents
27862d8
@hugouchoasborges @gabrielcardoso21
[IMP] download_attachment return view
9490846
@hugouchoasborges @gabrielcardoso21
[FIX] SIT_MANIF constantes
8bf94bf
@hugouchoasborges @gabrielcardoso21
[IMP] action_salva_xml
1ed07b5
@hugouchoasborges @gabrielcardoso21
[ADD] Método parse_xml_document
15443ce
@hugouchoasborges @gabrielcardoso21
[FIX][REF] download_documents refatorado para novos modelos
615f453
@hugouchoasborges @gabrielcardoso21
[FIX][REF] action_import_document refatorado para novos modelos
1626277
@hugouchoasborges @gabrielcardoso21
[REF] action_download_all_xmls refatorado para novos modelos
f9c72c6
@gabrielcardoso21
[FIX] Rebase
600a8c5
@mileo mileo self-requested a review September 25, 2020 17:58
@mileo
Copy link
Member

mileo commented Sep 25, 2020

@gabrielcardoso21 poderia verificar os erros do travis e do runbot por gentileza?

@mileo
Copy link
Member

mileo commented Sep 28, 2020

@gabrielcardoso21 pode ver o que houve com o runbot?

@mileo
Copy link
Member

mileo commented Sep 28, 2020

@gabrielcardoso21 please fix:

2020-09-28 12:58:13,730 175 WARNING openerp_test odoo.fields: Field l10n_br_fiscal.dfe.display_name depends on itself; please fix its decorator @api.depends().

mileo
mileo requested changes Sep 28, 2020
View reviewed changes
Copy link
Member

@mileo mileo left a comment

Choose a reason for hiding this comment

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

Runbot Error

@gabrielcardoso21
Copy link
Contributor Author

@gabrielcardoso21 please fix:

2020-09-28 12:58:13,730 175 WARNING openerp_test odoo.fields: Field l10n_br_fiscal.dfe.display_name depends on itself; please fix its decorator @api.depends().

De acordo com essa issue odoo/odoo#41081, corrigir o problema não some com o erro. Tive que trocar a abordagem, não mais usando o campo display_name e sim a função name_get.

marcelsavegnago
marcelsavegnago reviewed Sep 28, 2020
View reviewed changes
Copy link
Member

@marcelsavegnago marcelsavegnago left a comment •
edited
Loading

Choose a reason for hiding this comment

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

image
Pode esquecer .. achei que era um bug e não tinha visto que já filtrava as empresas com certificado vinculado

marcelsavegnago
marcelsavegnago approved these changes Sep 28, 2020
View reviewed changes
Copy link
Member

@marcelsavegnago marcelsavegnago left a comment

Choose a reason for hiding this comment

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

Entendo que é uma área intermediária para buscar documentos eletrônicos emitidos contra a empresa me parece ok.

@mileo
Copy link
Member

mileo commented Sep 28, 2020

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 12.0-ocabot-merge-pr-928-by-mileo-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit c17037d into OCA:12.0 Sep 28, 2020
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 7b5b814. Thanks a lot for contributing to OCA. ❤️

This was referenced Sep 28, 2020
Closed
Closed
@mileo mileo deleted the feature/dfe_mdfe branch November 4, 2020 14:03
@rvalyi rvalyi mentioned this pull request Dec 28, 2020
Merged
mileo pushed a commit to kmee/l10n-brazil that referenced this pull request Mar 2, 2021
@OCA-git-bot
Merge PR OCA#928 into 12.0
c98f6bb 
Signed-off-by mileo
@rvalyi rvalyi mentioned this pull request Jun 4, 2021
Merged
@rvalyi rvalyi mentioned this pull request Aug 17, 2021
Merged
@rvalyi rvalyi mentioned this pull request Oct 13, 2022
Closed
@rvalyi rvalyi mentioned this pull request Jul 12, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marcelsavegnago marcelsavegnago marcelsavegnago approved these changes

@mileo mileo mileo approved these changes

@rvalyi rvalyi Awaiting requested review from rvalyi

Assignees
No one assigned
Labels
approved merged 🎉
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@gabrielcardoso21 @mileo @OCA-git-bot @marcelsavegnago @rvalyi @hugouchoasborges