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

Return and Result url are essential #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

0xbeven
Copy link

@0xbeven 0xbeven commented Oct 17, 2020

  • The Response object from Paynow is too vague. Since this is an API why not make this a json response with all the information on success?
  • Since most applications using this API would be web based i think the return and result urls are essential not optional.
    -- Paynow should redirect the client to the return url automatically on success on the url set by the merchant
    -- Paynow should sent results of the transaction to the mercant's results url or portal whichever way h/her backend is configured
  • Create two methods separate ie for mobile payments why should i select the method for now? Why not make it an attribute of the Paynow object.

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.

1 participant