pagseguro / pagseguro-sdk-dotnet

Biblioteca de integração em .NET
55 stars 88 forks source link

Update PaymentMethodCode and PaymentMethodType #43

Closed LucasArgate closed 4 years ago

LucasArgate commented 4 years ago

Este relatório deve ser usado APENAS para reportar bugs

Comportamento esperado

No resultado da API do PagSeguro, hoje vem PaymentMethodCode 803 com o valor e PaymentMethodType com o valor 8.

Comportamento atual

Não estão mapeadas no Constants com PaymentMethodType.cs e PaymentMethodCode.cs com os valores atualizados.

Ambiente (produção, sandbox)

Ambos

Passos para reproduzir o fato

https://github.com/pagseguro/pagseguro-sdk-dotnet/blob/master/source/Uol.PagSeguro/Constants/PaymentMethodType.cs https://github.com/pagseguro/pagseguro-sdk-dotnet/blob/master/source/Uol.PagSeguro/Constants/PaymentMethodCode.cs

  1. Instancie uma classe do serviço que retorne qualquer transação de Cartão de Débito na moderninha.
  2. var transaction = NotificationService.CheckTransaction(credentials, codigoNotificacao);
  3. var codigo = transaction.PaymentMethod.PaymentMethodCode; //803 = Não mapeados!
  4. var tipo = transaction.PaymentMethod.PaymentMethodType; //8 = Não mapeados!

Descrição Detalhada

Propor adicionar esses arquivos as constants mapeadas para os próximos consumidores da Biblioteca não tenham que redescobrir algo que já tivemos problemas. Atualizar a documentação também é muito importante, ainda mais nesse momento (aceleração da transformação digital) que haverá mais procura por integrações online.

Possível solução

Uol.PagSeguro.Domain.PaymentMethodType.cs public const int OfflineDebit = 8; //Sugestões de nomes: DebitMachine, DebitOffline, DevidCard.

Uol.PagSeguro.Domain.PaymentMethodCode.cs public const int DebitoMaquina= 803; //Sugestões de nomes: DebitMachine, DebitOffline, DevidCard.

Evidências:

Valores retornados pela API e não atribuídos no SDK: image

Documentação desatualizada faltando os dados: image

Lucas Argate