Skip to content

[WIP][payum] update payum to 0.9#1620

Merged
pjedrzejewski merged 1 commit intoSylius:masterfrom
formapro-forks:payum-09-upgrade
Jun 21, 2014
Merged

[WIP][payum] update payum to 0.9#1620
pjedrzejewski merged 1 commit intoSylius:masterfrom
formapro-forks:payum-09-upgrade

Conversation

@makasim
Copy link
Copy Markdown
Contributor

@makasim makasim commented Jun 17, 2014

Do not merge, I dont have a change to test it because of the bug. When I click "add to cart" I am redirected to cart page where I see the message "your cart is empty".

@pjedrzejewski
Copy link
Copy Markdown
Contributor

Nice, I love the simpler config! I will look into the bug.

@makasim
Copy link
Copy Markdown
Contributor Author

makasim commented Jun 19, 2014

@pjedrzejewski any news about the bug? did you repo it?

@makasim
Copy link
Copy Markdown
Contributor Author

makasim commented Jun 20, 2014

@pjedrzejewski never mind. It is my local problem. The session does not work properly.

It somehow related to php upgrade to 5.5.13 but I am not sure.

Just for the record: To fix it I had to set session save path explicitly:

framework:
    session:
        save_path: /var/lib/php5

pjedrzejewski pushed a commit that referenced this pull request Jun 21, 2014
@pjedrzejewski pjedrzejewski merged commit d4b294b into Sylius:master Jun 21, 2014
@pjedrzejewski
Copy link
Copy Markdown
Contributor

Perfect, thank you Maksim! 👍

@makasim
Copy link
Copy Markdown
Contributor Author

makasim commented Jun 21, 2014

@pjedrzejewski I think you have to revert the merge, I dont have a chance to test. It still WIP.

@makasim
Copy link
Copy Markdown
Contributor Author

makasim commented Jun 21, 2014

I dont expect serious problems but...

@pjedrzejewski
Copy link
Copy Markdown
Contributor

Sorry, a was a bit too fast. :) Let's leave it for 0.11.x then.

pjedrzejewski pushed a commit that referenced this pull request Jun 21, 2014
@pjedrzejewski
Copy link
Copy Markdown
Contributor

Reverted in 0dfff11.

pamil pushed a commit to pamil/Sylius that referenced this pull request May 7, 2019
…pgrade"

This reverts commit d4b294b, reversing
changes made to a525a7e.
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.

2 participants