-
Notifications
You must be signed in to change notification settings - Fork 15
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
Handle Notifications from PayPal API #2826
Closed
Closed
Conversation
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
Use new MembershipIdGenerator
Replace LegacyPayPalURLGenerator with PayPalAPIURLGenerator by passing a PayPalAPIURLGeneratorConfig into the PaymentURLFactory Add PayPal configuration files and reference to them from the main config file. Change config schema. We removed the "delay in days" option from the schema, because it will be replaced by something different for delaying the membership. Describe .env files and change override files Improve README Co-Authored-By: Abban Dunne <abban.dunne@wikimedia.de> Co-Authored-By: Corinna Hillebrand <corinna.hillebrand@wikimedia.de> Co-Authored-By: Sperling-0 <tanuja.doriya@wikimedia.de>
The PayPal API configuration is in a YAML file that gets read and validated by `PayPalURLGeneratorConfigReader`. To avoid parsing and validating on every request, we cache the output of the class in `PayPalAPIURLGeneratorConfigLoader`. We realized that we can reuse the cache for the campaign configuration, because the "source" files for the configuration are in the same path (`app/config`), so the cache keys won't collide. Renamed "CampaignCache" to "ConfigurationCache". Co-Authored-By: Abban Dunne <abban.dunne@wikimedia.de> Co-Authored-By: Corinna Hillebrand <corinna.hillebrand@wikimedia.de> Co-Authored-By: Sperling-0 <tanuja.doriya@wikimedia.de>
Add services to Symfony configuration so its dependency injection framework can find the commands for creating and listing subscription plans in the payment bounded context. Loading the credentials from the `.env` files is done automatically in `bin/console`, so the credentials will exist when the environ-specific factory needs them. https://phabricator.wikimedia.org/T341568 Co-Authored-By: Corinna Hillebrand <corinna.hillebrand@wikimedia.de> Co-Authored-By: Sperling-0 <tanuja.doriya@wikimedia.de>
Add a command that can validate a PayPal API configuration file. Will be used during deployment.
This commit implements the authentication interfaces for the bounded contexts, with persistence. I called the implementation `OldStyleTokens` to leave the option to have a different implementation that uses only tokens. This also removes the reliance on the Doctrine event system for generating tokens. All the code in the Context Factories in the bounded contexts can be removed. I accidentally force-pushed to the bounded context branches, which led to errors during the rebase of the `payment-with-ppl` branch. I removed the composer.lock during the rebase and I'm adding it back here. This commit fixes the broken `payment-with-ppl` branch. Tickets: https://phabricator.wikimedia.org/T341795 https://phabricator.wikimedia.org/T344271
Don't use application log file because by the time the factory is called, it's still a NullLogger.
Recurring payments conducted through the PayPal API will have a transaction type of `recurring_payment`. Add a failing test for a recurring donation This is for https://phabricator.wikimedia.org/T329162 Co-Authored-By: Corinna Hillebrand <corinna.hillebrand@wikimedia.de> Co-Authored-By: Sperling-0 <tanuja.doriya@wikimedia.de>
This commit implements the authentication interfaces for the bounded contexts, with persistence. I called the implementation `OldStyleTokens` to leave the option to have a different implementation that uses only tokens. This also removes the reliance on the Doctrine event system for generating tokens. All the code in the Context Factories in the bounded contexts can be removed. I accidentally force-pushed to the bounded context branches, which led to errors during the rebase of the `payment-with-ppl` branch. I removed the composer.lock during the rebase and I'm adding it back here. This commit fixes the broken `payment-with-ppl` branch. Tickets: https://phabricator.wikimedia.org/T341795 https://phabricator.wikimedia.org/T344271
gbirke
force-pushed
the
ppl-api-fixes
branch
2 times, most recently
from
October 10, 2023 09:56
6228e58
to
cf6022e
Compare
Closing this, because it contains just some TODOs for the next steps and then we realized we need to do all the authentication stuff. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
https://phabricator.wikimedia.org/T329162