Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Create a migration guide in the docs.#11330

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
wouterj merged 1 commit intosymfony:4.2fromdbrumann:migration_guide
Jun 8, 2019
Merged

Create a migration guide in the docs.#11330

wouterj merged 1 commit intosymfony:4.2fromdbrumann:migration_guide
Jun 8, 2019

Conversation

@dbrumann
Copy link
Contributor

The general idea is to provide advice on how to migrate
an existing application over to Symfony using some
proven approaches.

The page should provide around 3 alternatives and some
discussion when they might be suitable as well as
additional resources like videos on this topics from
the official Symfony YouTube-channel.

derrabus reacted with hooray emojiderrabus reacted with rocket emoji
@dbrumann
Copy link
ContributorAuthor

@mpdude and@derrabus feel free to add commits to this PR

@wouterjwouterj added the ⭐️ EU-FOSSA Hackathonhttps://symfony.com/blog/the-symfony-and-api-platform-hackathon-is-coming labelApr 7, 2019
@mpdude
Copy link
Contributor

Just for the record, I've talked about this at the Symfony User Group Cologne back in 2013. Unfortunately, slides are in German and of limited use without the sound track:

https://www.slideshare.net/webfactory/marry-me-public

Additionally, we've published thehttps://github.com/webfactory/WebfactoryLegacyIntegrationBundle back then and since used it successfully on several legacy systems.

dbrumann reacted with thumbs up emoji

@derrabus
Copy link
Member

My talk is from 2016 which in itself is somewhat legacy now. ;-)

@wouterj
Copy link
Member

@weaverryan@javiereguiluz I'm also interested to know your opinion about this article. We shortly discussed this at EU-FOSSA and came up with this idea to explain some general ideas on how to migrate from legacy code to Symfony in the documentation.

As it's all very specific to the project, we can't document one approach that works for all. But showing some approaches (from the least to most radical) can imo be a nice addition for projects facing this problem.

Copy link
Contributor

@OskarStarkOskarStark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Great work so far Denis

@dbrumanndbrumann changed the title[WIP] Create a migration guide in the docs.Create a migration guide in the docs.Apr 28, 2019
@dbrumanndbrumann marked this pull request as ready for reviewApril 28, 2019 11:23
Copy link
Member

@javiereguiluzjaviereguiluz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

@dbrumann thanks for working on this! I think this is ready to merge. Do you agree?

@dbrumann
Copy link
ContributorAuthor

Thanks for your feedback@javiereguiluz and everyone else :)
I am happy with the current state and think this is ready to be merged.

@derrabus
Copy link
Member

@dbrumann I'd like to do another review on the weekend, if that's okay for you.

dbrumann reacted with thumbs up emoji

@dbrumann
Copy link
ContributorAuthor

What do you say, is this ready to be merged?@OskarStark,@javiereguiluz

derrabus reacted with thumbs up emoji

@OskarStark
Copy link
Contributor

Yes but this should go into 4.2 instead of master IMHO

dbrumann reacted with thumbs up emoji

The general idea is to provide advice on how to migratean existing application over to Symfony using someproven approaches.The page should provide around 3 alternatives and somediscussion when they might be suitable as well asadditional resources like videos on this topics fromthe official Symfony YouTube-channel.
@dbrumanndbrumann changed the base branch frommaster to4.2June 8, 2019 10:51
@dbrumann
Copy link
ContributorAuthor

I rebased and changed the target branch to 4.2

OskarStark reacted with heart emoji

@wouterjwouterj self-assigned thisJun 8, 2019
@wouterjwouterj merged commitb91249f intosymfony:4.2Jun 8, 2019
wouterj added a commit that referenced this pull requestJun 8, 2019
This PR was merged into the 4.2 branch.Discussion----------Create a migration guide in the docs.The general idea is to provide advice on how to migratean existing application over to Symfony using someproven approaches.The page should provide around 3 alternatives and somediscussion when they might be suitable as well asadditional resources like videos on this topics fromthe official Symfony YouTube-channel.Commits-------b91249f Creates a migration guide.
wouterj added a commit that referenced this pull requestJun 8, 2019
* 4.2:  [#11330] Restructured the Migration guide a bit while proof-reading  Creates a migration guide.
wouterj added a commit that referenced this pull requestJun 8, 2019
* 4.3:  [#11330] Restructured the Migration guide a bit while proof-reading  Creates a migration guide.
wouterj added a commit that referenced this pull requestJun 8, 2019
* 4.4:  [#11330] Restructured the Migration guide a bit while proof-reading  Creates a migration guide.
@wouterj
Copy link
Member

wouterj commentedJun 8, 2019
edited
Loading

Hi@dbrumann! Thanks a lot of the huge effort put into this PR!!

As I wasn't able to review your PR anywhere soon after you created it, I've merged it directly and did a proof-read and changes afterwards. You can see all these changes indb82d54. It's mostly structuring the "Prerequisites" section a bit and some minor clearifications/breaking long sentences/paragraphs.

Again thanks and also thanks to@derrabus for closely reviewing this one!

OskarStark reacted with thumbs up emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@OskarStarkOskarStarkOskarStark requested changes

@javiereguiluzjaviereguiluzjaviereguiluz approved these changes

+1 more reviewer

@derrabusderrabusderrabus requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

@wouterjwouterj

Labels

⭐️ EU-FOSSA Hackathonhttps://symfony.com/blog/the-symfony-and-api-platform-hackathon-is-comingStatus: Needs Review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@dbrumann@mpdude@derrabus@wouterj@OskarStark@javiereguiluz@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp