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

Ajustes nas Aplicações de Demonstração, devido alteração do Enum Credito em Loja das Informações de Pagamento #18

Merged
merged 1 commit into from
May 31, 2024

Conversation

adrbarros
Copy link
Collaborator

Ajustes nas Aplicações de Demonstração, devido alteração do Enum Credito em Loja das Informações de Pagamento

@adrbarros
Copy link
Collaborator Author

@robertorp @marcosgerene @danilobreda , houve uma alteração na Descrição do Enum Credito em Loja, avaliar se mantem essa alteração, ou voltar o Enum para a Descrição inicialmente definida

@marcosgerene marcosgerene merged commit eff3fbd into Hercules-NET:master May 31, 2024
2 checks passed
@danilobreda
Copy link
Collaborator

Foi por conta do pull #17 acho que pode manter pois estão corretos

@adrbarros
Copy link
Collaborator Author

adrbarros commented May 31, 2024

Sim foi por causa do Pull #17, imaginei o seguinte: como mudou de fpCreditoLoja para fpCreditoEmLoja, muitos ja utilizam o Enum Inicial (fpCreditoLoja), e assim quando atualizaram suas Libs pelo Nuget, vão ter que corrigir os pontos que possivelmente utilizam fpCreditoLoja para fpCreditoEmLoja. (No meu Caso tive que ajustar PDV e Retaguarda)

@danilobreda
Copy link
Collaborator

@adrbarros é realmente, não devíamos ter alterado o nome do enum, e agora a cagada ja ta feita sera?

@danilobreda
Copy link
Collaborator

@adrbarros acho que está tranquilo, se o sefaz alterou e adicionou novos, fica complicado. Vou lançar uma notícia no canal do discord sobre, notificando a galera.

@adrbarros
Copy link
Collaborator Author

@danilobreda também não vejo problemas nas alterações, no meu caso por exemplo só precisei fazer alteração em 2 pontos ma minha implementação, porquê na compilação deu erro devido a alteração do nome do ENUM, e é claro que for atualizar os pacotes ficarem atentos para só começar a utilizar as novas formas de pagamento quanto estrarem em vigor.

Valmir-Cunha pushed a commit to Valmir-Cunha/ZeusFiscal that referenced this pull request Jul 4, 2024
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