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

[14.0][IMP] Paulistana Status Consulta #3527

Draft
wants to merge 2 commits into
base: 14.0
Choose a base branch
from
Draft

[14.0][IMP] Paulistana Status Consulta #3527

wants to merge 2 commits into from

Conversation

mileo
Copy link
Member

@mileo mileo commented Dec 5, 2024

Apesar de buscar o retorno do status da consulta o método não atualizava a nota fiscal.

@OCA-git-bot
Copy link
Contributor

Hi @rvalyi, @luismalta, @renatonlima, @gabrielcardoso21,
some modules you are maintaining are being modified, check this out!

@mileo mileo force-pushed the 14.0-imp-paulistana-status-consulta branch from ce3d562 to 138ad37 Compare December 5, 2024 04:56
rvalyi
rvalyi approved these changes Dec 5, 2024
View reviewed changes
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.

LGTM

antoniospneto
antoniospneto approved these changes Dec 6, 2024
View reviewed changes
Copy link
Contributor

@antoniospneto antoniospneto left a comment

Choose a reason for hiding this comment

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

por mim ok tbm, mas ver que o coverage nessa etapa tá bem baixo

@mileo mileo added the bug label Dec 6, 2024
@mileo mileo force-pushed the 14.0-imp-paulistana-status-consulta branch 4 times, most recently from 2a0c6a3 to fa45018 Compare December 6, 2024 14:27
@mileo mileo force-pushed the 14.0-imp-paulistana-status-consulta branch from fa45018 to f71dbad Compare December 6, 2024 19:17
@mileo mileo marked this pull request as draft December 6, 2024 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@antoniospneto antoniospneto antoniospneto approved these changes

@renatonlima renatonlima renatonlima approved these changes

@rvalyi rvalyi rvalyi approved these changes

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

Successfully merging this pull request may close these issues.
5 participants
@mileo @OCA-git-bot @rvalyi @renatonlima @antoniospneto