-
Notifications
You must be signed in to change notification settings - Fork 16
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
Feature/sponsorship #613
Feature/sponsorship #613
Conversation
Oh, I totally didn't see that, my bad 🤦♂️. Should be fixed now! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🐉 💀
🗡🏇
@juramos-2020 habilitei as contribuições pra essa comunidade (por hora as comunidades vão precisar nos pedir pra habilitar) |
⚠ Ao acessar, no Dashboard, o modal para realizar a contribuição não abriu. 😉👌 Os outros fluxos funcionaram perfeitamente e o layout estava de acordo ❓ Uma dúvida: As "Minhas contribuições" só apareceram depois de atualizar a página (clicando no icone para atualizar). Seria isso mesmo? Ou deveria aparecer automaticamente após a 1º contribuição sem precisar "forçar" a atualização da tela? |
Ju só mostramos as que já foram aprovadas pelo PayPal, pode demorar bastante tempo a depender do contexto (tipo de pagamento, país etc). Se você achar que devemos mostrar as pendentes, precisamos de um design pra mostrar essas diferenças de status |
Resolvido! Tinha um problema onde o modal de contribuição não aparecia quando o usuário ainda não tinha contatos registrados. Como esse modal só aparece uma vez por usuário (como descrito no figma), uma opção pra testar esse fluxo é usar o app em uma janela anônima do browser. Poderia testar de novo e ver se era esse o problema mesmo? |
Funcionou 👌😉
Já que pode demorar bastante tempo, é melhor apresentarmos um status para não correr o risco do usuário pensar que sua doação não foi processada e ficar refazendo essa doação várias vezes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚗 ✨
@juramos-2020 Adicionamos as tags de status da contribuição, acho que estamos prontos 💪. Poderia dar uma última olhada nessa parte das tags? |
Ficou show 👌😉✨ |
@lucca65 lembro que falamos de deixar esse PR aberto por enquanto. Ainda vamos deixar aberto ou posso mergear? |
ping @lucca65 |
ué jurava que tinha respondido... pode mandar bala no merge! |
What issue does this PR close
Closes #599
Changes Proposed ( a list of new changes introduced by this PR)
PAYPAL_CLIENT_ID
, which should contain the PayPal client idbeforeUpdate
"hook" onInput
, which allows us to treat the inputString
before sending the update msgHow to test ( a list of instructions on how to test this PR)
Note: you can use this sandbox account to test with PayPal:
SPONSOR FLOW:
CHECKING SPONSORS FLOW: