-
Notifications
You must be signed in to change notification settings - Fork 29
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/paypal permissions api integration #70
Open
dmytro-pro
wants to merge
30
commits into
makasim:master
Choose a base branch
from
dmytro-pro:feature/paypal-permissions-api-integration
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
5e6d7c1
Changed .gitignore
dmytro-pro a418f73
Added app.php & app_dev.php according to modern official userguide
dmytro-pro 50e7267
Updated AppKernel.php to use local configs too
dmytro-pro 704bf4b
Added variables initialization to index.php (to make env selection wo…
dmytro-pro 151c073
Merge branch 'feature/d-pro_master' into feature/core-improvements
dmytro-pro 6f5ac4f
Added PayPal permissions SDK to dependencies
dmytro-pro de2ad86
Made Permissions API -> getRequestToken + getAccess as a single action
dmytro-pro 75dfaa7
Added stubs; added "get-request-token" action controller
dmytro-pro 6acc4c3
Added "get-access-token" action controller; small refactoring
dmytro-pro b3afaec
Cleaning & small refactoring
dmytro-pro 579d805
Added composer.lock to repository
dmytro-pro 4dee726
Restored Details:view.html.twig
dmytro-pro 7be8804
Config fix
dmytro-pro ff536d7
Added new API and Factory for "ExpressCheckout via token" feature; ad…
dmytro-pro 4f5802c
Added demo action for merchant token feature
dmytro-pro f6a3a57
Fixes
dmytro-pro 1d3c115
Customized new ApiViaToken class to use AuthToken for authorization, …
dmytro-pro d0ba6cf
Added subject field to parameters.yml.dist; made the code work (not c…
dmytro-pro ee0a968
Config fix
dmytro-pro fd3d9d3
Removed old factories, and used new ones from dev-version of Payum. M…
dmytro-pro 83b8ac2
Fixed demo text
dmytro-pro ddb5cfc
Small fix
dmytro-pro e586f3e
Merge branch 'feature/core-improvements' into feature/paypal-permissi…
dmytro-pro 730ff2b
Revert "Changed .gitignore"
dmytro-pro 89f9547
Revert "Added variables initialization to index.php (to make env sele…
dmytro-pro 56cd0c7
Revert "Updated AppKernel.php to use local configs too"
dmytro-pro b1e154f
Revert "Added app.php & app_dev.php according to modern official user…
dmytro-pro 848016b
Moved PayPal sandbox app id into parameters.yml.dist
dmytro-pro 1a20bff
Config fix
dmytro-pro 2e4a3f9
tokenSecret -> token_secret
dmytro-pro File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -71,6 +71,16 @@ payum: | |
factory: paypal_express_checkout | ||
payum.api: @acme.payment.payum.paypal_express_checkout_api | ||
|
||
paypal_express_checkout_permission: | ||
factory: paypal_express_checkout_permission | ||
username: %paypal.express_checkout.username% | ||
password: %paypal.express_checkout.password% | ||
signature: %paypal.express_checkout.signature% | ||
token: %paypal.express_checkout.token% | ||
token_secret: %paypal.express_checkout.token_secret% | ||
third_party_subject: %paypal.express_checkout.third_party_subject% | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. do not allow it to be configured this way. see above |
||
sandbox: true | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. paypal_express_checkout_permission:
factory: paypal_express_checkout_permission
username: foo
password: foo
access_token: foo
access_token_secret: foo |
||
paypal_pro_checkout: | ||
factory: paypal_pro_checkout | ||
username: %paypal.pro_checkout.username% | ||
|
@@ -187,6 +197,7 @@ payum: | |
config_key: %sofort.config_key% | ||
|
||
foo_bar_gateway: | ||
factory: core | ||
payum.action.capture: @acme_payment.foo_bar.capture | ||
payum.action.status: @acme_payment.foo_bar.status | ||
|
||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if user needs a subject he can set it to the payment model directly, remote it from here and from the api options as well