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

move storage factory init from PayumBundle::build to PayumExtension::load #443

Merged
merged 1 commit into from
Dec 22, 2017

Conversation

lsmith77
Copy link
Contributor

No description provided.

@makasim
Copy link
Member

makasim commented Nov 24, 2017

@lsmith77 Does it work for you?

@lsmith77
Copy link
Contributor Author

it fixes the issue I encountered .. but hoping that the travis tests reveal if this creates other issues ..
I tried to figure out from the commit history, why you previously didn't already have this code in the extension, since this seems like the logical place ..

@makasim
Copy link
Member

makasim commented Nov 24, 2017

for ref Sylius/Sylius#8979 (comment)

@lsmith77 lsmith77 force-pushed the fix_storage_factory_init branch from 08f6924 to afcd614 Compare November 24, 2017 09:04
@lsmith77 lsmith77 force-pushed the fix_storage_factory_init branch from afcd614 to b94e0d8 Compare November 24, 2017 09:10
@lsmith77
Copy link
Contributor Author

ok looks better now .. but didn't really do a manual test to confirm that everything is still working.

@makasim makasim added the bug label Nov 24, 2017
@makasim makasim merged commit 0c0863d into Payum:master Dec 22, 2017
@lsmith77 lsmith77 deleted the fix_storage_factory_init branch December 22, 2017 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants