{"action":"created","issue":{"url":"https://api.github.com/repos/Payum/Payum/issues/416","labels_url":"https://api.github.com/repos/Payum/Payum/issues/416/labels{/name}","comments_url":"https://api.github.com/repos/Payum/Payum/issues/416/comments","events_url":"https://api.github.com/repos/Payum/Payum/issues/416/events","html_url":"https://github.com/Payum/Payum/issues/416","id":110233932,"number":416,"title":"PayPal Express Checkout - missing step or how to add \"confirm order\" page","user":{"login":"uvoelkel","id":460553,"avatar_url":"https://avatars.githubusercontent.com/u/460553?v=3","gravatar_id":"","url":"https://api.github.com/users/uvoelkel","html_url":"https://github.com/uvoelkel","followers_url":"https://api.github.com/users/uvoelkel/followers","following_url":"https://api.github.com/users/uvoelkel/following{/other_user}","gists_url":"https://api.github.com/users/uvoelkel/gists{/gist_id}","starred_url":"https://api.github.com/users/uvoelkel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/uvoelkel/subscriptions","organizations_url":"https://api.github.com/users/uvoelkel/orgs","repos_url":"https://api.github.com/users/uvoelkel/repos","events_url":"https://api.github.com/users/uvoelkel/events{/privacy}","received_events_url":"https://api.github.com/users/uvoelkel/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Payum/Payum/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-10-07T14:10:43Z","updated_at":"2015-10-09T06:58:55Z","closed_at":null,"body":"Hi!\r\n\r\nMaybe I'm missing something fundamental but it seems to me that the express checkout process is missing one crucial step namely the \"confirm order\" step.\r\n\r\nIf you look at the flow here: https://developer.paypal.com/docs/classic/express-checkout/integration-guide/ECGettingStarted/#id084RN0F0OPN\r\nand here: https://developer.paypal.com/docs/classic/express-checkout/integration-guide/ECGettingStarted/#id076LF0X0F5Z\r\nthere should be a \"confirm order\" step before executing the payment.\r\n\r\nThe CaptureAction seems to execute SetExpressCheckout and DoExpressCheckoutPayment in one step. Is this the intended way it should work? Do I need to write my own actions or is there some other way to implement the \"confirm order\" page that I overlooked?"},"comment":{"url":"https://api.github.com/repos/Payum/Payum/issues/comments/146773250","html_url":"https://github.com/Payum/Payum/issues/416#issuecomment-146773250","issue_url":"https://api.github.com/repos/Payum/Payum/issues/416","id":146773250,"user":{"login":"makasim","id":143206,"avatar_url":"https://avatars.githubusercontent.com/u/143206?v=3","gravatar_id":"","url":"https://api.github.com/users/makasim","html_url":"https://github.com/makasim","followers_url":"https://api.github.com/users/makasim/followers","following_url":"https://api.github.com/users/makasim/following{/other_user}","gists_url":"https://api.github.com/users/makasim/gists{/gist_id}","starred_url":"https://api.github.com/users/makasim/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/makasim/subscriptions","organizations_url":"https://api.github.com/users/makasim/orgs","repos_url":"https://api.github.com/users/makasim/repos","events_url":"https://api.github.com/users/makasim/events{/privacy}","received_events_url":"https://api.github.com/users/makasim/received_events","type":"User","site_admin":false},"created_at":"2015-10-09T06:58:55Z","updated_at":"2015-10-09T06:58:55Z","body":"I am thinking of an extra CaptureWithConfirmationAction, which additionally renders the confirmation template, must be set via constructor, and once user confirms it continue. Not sure it has to replace current one, I think a developer has to explicitly replace current impl with a confirmation one."}}