-
Notifications
You must be signed in to change notification settings - Fork 5
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
payment_event vs order.set_status #3
Comments
Eventos? |
Sim, eventos! Ao ler o sinal do pagseguro, ao invés de "order.set_status('Aguardando pagamento')", usar um add_payment_event('Aguardando pagamento'). No dashboard é possível também criar um evento de pagamento, mas o mais intuitivo dele alterar é o estado do pedido. Veja bem, só estou tentando entender a pipeline, na forma atual:
Aparentemente não há problemas, mas olhando os eventos de pagamento, o único que existe até então é um "pre-auth" de quando o pagseguro está aprovando a transação. Será que não seria melhor estes sinais do pagseguro serem eventos de pagamento e o admin do dashboard ficar de olho no estado do pedido e alterá-los quando necessário? |
Ah entendi Eu realmente não me atentei ao funcionamento dos eventos do Oscar, o pouco que olhei não resolveu meu problema, acabei abandonando. Quando tiver um tempo dou uma olhada, ou se quiser, faça um pull-request 👍 |
OK, qualquer novidades postamos aqui |
Tudo jóia @daneoshiga?
Queria saber se existe alguma razão de os sinais da oscar-pagseguro setarem o
status do pedido ao invés de emitirem eventos de pagamento com os estados do
pagseguro ('aguardando pagemento', 'pago', etc).
Pensei nisto por que apenas o pagseguro pode mudar estes estados no fluxo
normal, se fosse alterado diretamente pelo dashboard da aplicação o pagseguro
não reconheceria, não é uma comunicação de 2 vias certo?
Se o oscar-pagseguro fosse implementado paga emitir sinais de pagamento, o
estado do pedido poderia ser alterado automaticamente via EventHandlers, mas
também manualmente pelo dashboard da aplicação no caso de algum problema com o
pagseguro/correios/etc.
O que acha?
The text was updated successfully, but these errors were encountered: