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

Improve docs for Yii integration #10

Merged
merged 1 commit into from
Jul 16, 2014
Merged

Improve docs for Yii integration #10

merged 1 commit into from
Jul 16, 2014

Conversation

alexandernst
Copy link
Contributor

No description provided.

@alexandernst alexandernst mentioned this pull request Jul 16, 2014
makasim added a commit that referenced this pull request Jul 16, 2014
Improve docs for Yii integration
@makasim makasim merged commit f71d046 into Payum:master Jul 16, 2014
@makasim
Copy link
Member

makasim commented Jul 16, 2014

@alexandernst thanks, do you have plans to migrate the extension to Yii2, As far as I can see it uses composer all the way.

@alexandernst
Copy link
Contributor Author

@makasim I'll probably have to, if I'm still assigned to the current project by the time Yii2 gets out. Anyways, I'm not expecting Yii2 to come out for another 6 months or so, so it's still far in the future 😄

@makasim
Copy link
Member

makasim commented Jul 16, 2014

so it was not released? I think it is already....

@makasim
Copy link
Member

makasim commented Jul 16, 2014

I'll try the extension in the clean yii1 env and it works release a stable version. I think I can find time to do some next week

@alexandernst
Copy link
Contributor Author

Great! 😄

Anyways, I'm not done patching it. I must do some tests, but maybe I'll need active record support, so I was looking at #4 ... What is the current state of that patch?

@makasim
Copy link
Member

makasim commented Jul 16, 2014

What is the current state of that patch?

I've never run it myself but The author say it works. I think the solution could be simplified a lot but in any case it is a good starting point.

@alexandernst
Copy link
Contributor Author

Ok, I'll clone from #4 and have a good look.

@makasim
Copy link
Member

makasim commented Jul 17, 2014

@alexandernst FYI updated the doc on site: http://payum.org/doc/0.9/PayumYiiExtension/installation

@alexandernst
Copy link
Contributor Author

@makasim \o/ great 👍

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