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

Add linkpay #2219

Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open

Add linkpay #2219

wants to merge 24 commits into from

Conversation

DinSmol
Copy link
Contributor

@DinSmol DinSmol commented Nov 19, 2021

close #2210
Replaced banklinks with linkpay (everypay).
Added linkpay to email and pdf.
Added qr to pdf.

@viezly
Copy link

viezly bot commented Nov 19, 2021

This pull request is split into 5 parts for easier review.
👀 Review pull request on Viezly

Changed files are located in these folders:

  • /
  • app/controllers/registrar
  • app/jobs
  • app/mailers
  • app/models
  • app/views
  • config
  • test

@vohmar
Copy link
Contributor

vohmar commented Dec 16, 2021

Few issues:

  • replace the payment button in registrar with a button called pay - it shows currently card payment icon that can be confusing
  • the payment windows should be openened in new tab or browser window so user can close it after the payment process is finished

now the important one

  • even though the payment was successful the invoice was not marked as paid

@vohmar vohmar assigned DinSmol and unassigned vohmar Dec 16, 2021
@DinSmol DinSmol assigned vohmar and unassigned DinSmol and vohmar Dec 22, 2021
@OlegPhenomenon OlegPhenomenon self-assigned this Mar 1, 2022
@OlegPhenomenon OlegPhenomenon force-pushed the 2210-upgrade-to-everypay-api-v4 branch from bbc8380 to 588b962 Compare April 13, 2022 10:36
@OlegPhenomenon OlegPhenomenon force-pushed the 2210-upgrade-to-everypay-api-v4 branch from 588b962 to 507fc6c Compare April 13, 2022 11:29
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.

Upgrade to EveryPay APIv4
3 participants