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

Feature/sponsorship #613

Merged
merged 73 commits into from
Oct 21, 2021
Merged

Feature/sponsorship #613

merged 73 commits into from
Oct 21, 2021

Conversation

henriquecbuss
Copy link
Member

@henriquecbuss henriquecbuss commented Aug 6, 2021

What issue does this PR close

Closes #599

Changes Proposed ( a list of new changes introduced by this PR)

  • Add a modal inviting the user to support the community
  • Add a modal to accept payments via PayPal's order API (using their Smart Buttons)
  • Add a thank you page to show after receiving a payment
  • Add a page to show all community supporters
  • Add supporter information and button to support the community on the community page
  • Add a new environment variable, PAYPAL_CLIENT_ID, which should contain the PayPal client id
  • Add a beforeUpdate "hook" on Input, which allows us to treat the input String before sending the update msg

How 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:

  1. Log into a community
  2. If not on the Dashboard already, go to the Dashboard
  3. You should see the modal inviting you to donate to the community
  4. Click on "Sponsor {{community}}"
  5. You should see a form to select an amount and pay with PayPal. Make sure you can't enter anything below 1 USD
  6. Once you donate, you should be redirected to the thank you page, and from there you should be able to see all supporters from that community

CHECKING SPONSORS FLOW:

  1. Log into a community
  2. Click on "How to earn {{symbol}}"
  3. Scroll to the bottom of the page
  4. You should see two new cards ("Our Supporters" and "Sponsor {{community}}"). You can either see all supporters or support the community. If you choose to support the community, you should follow the sponsor flow above starting from step 4.

@henriquecbuss henriquecbuss requested a review from lucca65 October 14, 2021 02:43
@lucca65
Copy link
Member

lucca65 commented Oct 14, 2021

Bro looks the same 🤔

Captura de Tela 2021-10-14 às 10 41 16

@lucca65
Copy link
Member

lucca65 commented Oct 14, 2021

oh, looking at the code I've noticed that you added a padding to the date, I was thinking about the space between the image and the number, not the date label

Captura de Tela 2021-10-14 às 10 42 19

@henriquecbuss
Copy link
Member Author

Oh, I totally didn't see that, my bad 🤦‍♂️. Should be fixed now!

lucca65
lucca65 previously approved these changes Oct 14, 2021
Copy link
Member

@lucca65 lucca65 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐉 💀

🗡🏇

@juramos-2020
Copy link

OI @NeoVier

eu estou acessando com esse ultimo link netlify / angry-wing-8fcdd0 / deploy-preview - Implante a visualização pronta!

Mas essa atualização do sponsor ainda não está disponível.

image

@lucca65
Copy link
Member

lucca65 commented Oct 15, 2021

@juramos-2020 habilitei as contribuições pra essa comunidade (por hora as comunidades vão precisar nos pedir pra habilitar)

@juramos-2020
Copy link

⚠ 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?

@lucca65
Copy link
Member

lucca65 commented Oct 15, 2021

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

@henriquecbuss
Copy link
Member Author

Ao acessar, no Dashboard, o modal para realizar a contribuição não abriu.

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?

@juramos-2020
Copy link

juramos-2020 commented Oct 15, 2021

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 👌😉

Ju só mostramos as que já foram aprovadas pelo PayPal, pode demorar bastante tempo a depender do contexto (tipo de pagamento, país etc).

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.
Aqui o design atualizado.

@henriquecbuss henriquecbuss requested a review from lucca65 October 16, 2021 00:36
Copy link
Member

@lucca65 lucca65 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚗 ✨

@henriquecbuss
Copy link
Member Author

@juramos-2020 Adicionamos as tags de status da contribuição, acho que estamos prontos 💪. Poderia dar uma última olhada nessa parte das tags?

@juramos-2020
Copy link

Ficou show 👌😉✨
tudo ok!

@henriquecbuss
Copy link
Member Author

@lucca65 lembro que falamos de deixar esse PR aberto por enquanto. Ainda vamos deixar aberto ou posso mergear?

@henriquecbuss
Copy link
Member Author

lembro que falamos de deixar esse PR aberto por enquanto. Ainda vamos deixar aberto ou posso mergear?

ping @lucca65

@lucca65
Copy link
Member

lucca65 commented Oct 21, 2021

ué jurava que tinha respondido... pode mandar bala no merge!

@henriquecbuss henriquecbuss merged commit 97cb118 into master Oct 21, 2021
@henriquecbuss henriquecbuss deleted the feature/sponsorship branch October 21, 2021 13:21
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.

Add Sponsorship (PayPal)
3 participants