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.

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

Adapt for erpbrasil.edoc integration #103

Merged
merged 1 commit into from
May 15, 2024

Conversation

antoniospneto
Copy link
Contributor

Adapted the current library to ensure compatibility with the new erpbrasil.edoc library that is pending release. This change is on a feature branch to support the upcoming integration.

See the related PR: erpbrasil/erpbrasil.edoc#77

@marcelsavegnago
Copy link

@antoniospneto essa fica como rascunho ainda ?

@antoniospneto
Copy link
Contributor Author

@antoniospneto essa fica como rascunho ainda ?

É que esse só pode dar merge, depois do merge no erpbrasil, aí deixei em rascunho por segurança ☺️

@rvalyi
Copy link
Member

rvalyi commented May 15, 2024

@antoniospneto essa fica como rascunho ainda ?

É que esse só pode dar merge, depois do merge no erpbrasil, aí deixei em rascunho por segurança ☺️

hum, mas nesse tipo de codigo que não é da OCA, nem necessaramente a pessoa vai usar as boas versões juntas...

erpbrasil.base e nfelib não tem dependencia explicitas entre eles (até pela falta de modularidade na arquitetura do erpbrasil.base senão começaria a depender da terra inteira). Neste caso seria melhor algum teste que onde o codigo roda com as duas versões do erpbrasil.base, mesmo que a gente deixa isso apenas por alguns meses. E sim neste tipo de coisa a gente começã a ver o tipo de problema que a gente teria com essa arquitertura monolitica do erpbrasil.base. Eh o problema com essas libs monoliticas que pretendem engulir o mundo, nunca é bem future proof... Tudo bem dar um tapa por cima para remediar a urgencia, mas é bom abrir o olho tb.

Adapted the current library to ensure compatibility with the new erpbrasil.edoc library that is pending release. This change is on a feature branch to support the upcoming integration.

See the related PR: erpbrasil/erpbrasil.edoc#77
@antoniospneto antoniospneto marked this pull request as ready for review May 15, 2024 16:45
@antoniospneto
Copy link
Contributor Author

antoniospneto commented May 15, 2024

@rvalyi fato, é melhor garantir que o código funcione nas duas situações, atualizei o código e coloquei como pronto para revisão, obrigado!

@rvalyi rvalyi merged commit cda030e into akretion:master May 15, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants