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

Fix #1837: integrating Yii with different Software and libs #1981

Merged
merged 4 commits into from
Jan 21, 2014

Conversation

klimov-paul
Copy link
Member

Fixes #1837: integrating Yii with different Software and libs

Integration with Yii1 included

@samdark
Copy link
Member

samdark commented Jan 15, 2014

Looks OK.

@qiangxue
Copy link
Member

Do we really need to cover integrating Yii1 with Yii2?

@klimov-paul
Copy link
Member Author

I suppose a lot of developers (including myself), which uses Yii1 may want to migrate old projects to Yii2. However such migration can not be made at once, so it would be common then at some stage project will use both Yii1 and Yii2.
It would be a shame if we do not provide any hint of how to integrate Yii2 with Yii1.

@creocoder
Copy link
Contributor

@klimov-paul

    "require": {
        "yiisoft/yii": "*",
        "yiisoft/yii2": "*",
    },

into composer.json, Integration done.

@samdark
Copy link
Member

samdark commented Jan 15, 2014

@creocoder except both provide Yii class.

@Ragazzo
Copy link
Contributor

Ragazzo commented Jan 15, 2014

why its not .md file?

@klimov-paul
Copy link
Member Author

why its not .md file?

My mistake, fixed.

@Ragazzo
Copy link
Contributor

Ragazzo commented Jan 15, 2014

Well, my opinion is that this article could be needed by some developers, because they will be searching for the way of how to reuse Yii1 code or make fast portable of extensions to not to wait while they will be adopted for Yii2. So it is better to have some official guide for this case, rather then just fixing bugs of different custom implementations. I think this should be linked to the article of the updating from Yii1 to Yii2.

@samdark
Copy link
Member

samdark commented Jan 15, 2014

@Ragazzo good idea.

@cebe
Copy link
Member

cebe commented Jan 16, 2014

+1 for covering Yii1 + Yii2

What needs to be done:

  • should add note about composer.json to adjust like this:
"require": {
        "yiisoft/yii": "*",
        "yiisoft/yii2": "*",
    },
  • also add link to the "Updating from yii1" guide page.

After that it's ready for merge imo.

cebe added a commit that referenced this pull request Jan 21, 2014
…ov-paul/yii2

* '1837-3rd-party-doc' of https://github.com/klimov-paul/yii2:
  Missing "md" extension added for "Using 3rd-Party Libraries" guide part
  "Using 3rd-Party Libraries" guide updated
  "Using 3rd-Party Libraries" guide added
@cebe cebe merged commit eecfba6 into yiisoft:master Jan 21, 2014
@cebe
Copy link
Member

cebe commented Jan 21, 2014

Merged. Thanks! Adjustments in ca816ad

qiansen1386 pushed a commit to qiansen1386/yii2 that referenced this pull request Mar 9, 2014
…om/klimov-paul/yii2

* '1837-3rd-party-doc' of https://github.com/klimov-paul/yii2:
  Missing "md" extension added for "Using 3rd-Party Libraries" guide part
  "Using 3rd-Party Libraries" guide updated
  "Using 3rd-Party Libraries" guide added
@klimov-paul klimov-paul deleted the 1837-3rd-party-doc branch April 6, 2016 09:24
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.

Could need some docs about integrating Yii with different Software and libs
6 participants