Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Yii2 fix #86

Open
wants to merge 22 commits into
base: develop
Choose a base branch
from
Open

Yii2 fix #86

wants to merge 22 commits into from

Conversation

ratacibernetica
Copy link

Long ago i made some changes so i could use it with Yii2.

Changed the bare minimum to get it working.

Classes names and namespaces mostly.

@oncesk
Copy link
Owner

oncesk commented Mar 1, 2017

Hello it is good news!)

As i think will be better if will be ability to choose between https and not, it can be done via few configuration changes, because now keys are hardcoded instead of that we can pass it throught configuration.

I can send you worked version with https support? after you made changes related to https i can merge it!

Great work!

@ratacibernetica
Copy link
Author

Sure, sounds good! Actually i did those changes but in another machine and didn't commit changes.

As far as i can remember, just changed 'http://' to '//' so it wouldn't matter what protocol developes use.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants