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][FIX] l10n_br_fiscal: adapt ChaveEdoc method #2288

Closed
wants to merge 1 commit into from
Closed

[14.0][FIX] l10n_br_fiscal: adapt ChaveEdoc method #2288

wants to merge 1 commit into from

Conversation

renatonlima
Copy link
Member

Port do commit 1f523d9 do PR #2000

rvalyi
rvalyi approved these changes Jan 6, 2023
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.

Ok. So não podemos fazer merge ainda por conta dos outros problemas (erpbrasil.assinatura e account_payment_order).

@rvalyi
Copy link
Member

rvalyi commented Jan 6, 2023

a mudança é para acomodar com essa mudança no erpbrasil.base https://github.com/erpbrasil/erpbrasil.base/pull/33/files#diff-e59d3448effc7f1ed9f71ee2692b4f2ab6dbb0ae6182950c921647a5cfccfa7dR102

@rvalyi rvalyi mentioned this pull request Jan 6, 2023
Merged
3 tasks
@rvalyi
Copy link
Member

rvalyi commented Jan 6, 2023

fechando porque ja fiz um cherry-pick desse commit no #2289 que entrou

@rvalyi rvalyi closed this Jan 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rvalyi rvalyi rvalyi approved these changes

Assignees
No one assigned
Labels
14.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@renatonlima @rvalyi