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

Paypal integration #77

Closed
cbellone opened this issue Oct 2, 2015 · 9 comments
Closed

Paypal integration #77

cbellone opened this issue Oct 2, 2015 · 9 comments
Assignees
Milestone

Comments

@cbellone
Copy link
Member

cbellone commented Oct 2, 2015

Even if stripe is a great API, it is not (yet) available on all countries. Maybe we should integrate also paypal.
It seems that there is an official REST (OAUTH?) API:
https://developer.paypal.com/docs/api/

@cbellone cbellone added this to the BackLog milestone Oct 2, 2015
@alexproca
Copy link

+1 for paypal integration

@cbellone
Copy link
Member Author

Hi @alexproca,

Thank you for your interest in alf.io!

As you can see, Paypal integration is not on the roadmap right now, so it would be great if you could provide a pull request for that. WDYT?

Thank you very much!
Celestino

@cbellone cbellone modified the milestones: 1.8, BackLog Apr 7, 2016
@cbellone
Copy link
Member Author

cbellone commented Apr 7, 2016

Due to the high amount of requests from a lot of users, we decided to include this feature in the upcoming version (1.8)

@syjer
Copy link
Member

syjer commented Jun 28, 2016

syjer added a commit that referenced this issue Jul 7, 2016
@syjer
Copy link
Member

syjer commented Jul 7, 2016

To be done:

  • remove all hardcoded values
  • ui/ux cleanup
  • fix "payment experience"
  • add hmac on parameters + check
  • fetch correct transaction id
  • error handling
  • display only the final cost instead of each items
  • translations: de, nl

syjer added a commit that referenced this issue Jul 7, 2016
syjer added a commit that referenced this issue Jul 7, 2016
syjer added a commit that referenced this issue Jul 8, 2016
syjer added a commit that referenced this issue Jul 11, 2016
syjer added a commit that referenced this issue Jul 11, 2016
syjer added a commit that referenced this issue Jul 11, 2016
syjer added a commit that referenced this issue Jul 12, 2016
syjer added a commit that referenced this issue Jul 12, 2016
syjer added a commit that referenced this issue Jul 12, 2016
syjer added a commit that referenced this issue Jul 13, 2016
syjer added a commit that referenced this issue Jul 13, 2016
@syjer
Copy link
Member

syjer commented Aug 16, 2016

Hi @mg-1999 , could you provide the translations for the following texts ?

  • Confirm paypal payment
  • Confirm to continue.
  • Pay with your paypal account. You will be redirected to paypal.com.
  • Error in returned data: they may have been modified. You have not been charged. Please try again.

Thank you :)

syjer added a commit that referenced this issue Aug 16, 2016
cbellone added a commit that referenced this issue Aug 16, 2016
@cbellone
Copy link
Member Author

has been implemented :-)

@mg-1999
Copy link
Contributor

mg-1999 commented Sep 4, 2016

Hello guys,

A little late to the party, but time was not on my side. The translation to dutch for the requested sentences are:

  • Bevestig paypal betaling
  • Bevestig om verder te gaan
  • Betaal met je paypal account. U wordt doorverwezen naar paypal.com.
  • Fout in ontvangen gegevens: ze zijn waarschijnlijk aangepast. Er zijn geen kosten in rekening gebracht. Probeer later opnieuw.

Good luck :)

@syjer
Copy link
Member

syjer commented Sep 4, 2016

Hi @mg-1999 , thanks for the translations ! I'll add them ASAP

syjer added a commit that referenced this issue Sep 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants