Payments #35
Payments #35
Conversation
| end | ||
| end | ||
|
|
||
| def validate_total_amount_vs_parts!(total_amount:, subtotal:, tax:, shipping:, discount:) |
There was a problem hiding this comment.
Penso que as validações na classe podem ajudar a encontrar erros antes de fazer a requisição para a Meta. Mas me parece que essas duas novas classes estão fazendo muitas validações e muito específicas.
Talvez a gem devesse se manter fazendo apenas as validações mais básicas, pois dessa forma fica muito suscetível a erros, e fica muito amarrado à regras de negócio da Meta.
Acho válido ver o que o Zé acha, se faz sentido tantas validações na gem.
There was a problem hiding this comment.
Percebi que da maneira atual, a classe fica um pouco dificil de fazer manutenção, pois as validações e estrutura não fica tão clara. Penso se existe alguma maneira de fazer as validações, mas de forma que fique intuitivo ler e entender a estrutura dela.
Talvez tenha uma forma de validar que o payload construido segue um padrão esperado. Algo que facilite a manutenção e e evolução da classe com o tempo.
WIP da feature de pagamentos
criação do resource de order_details e order_status para fazer o dispatch