Skip to content

Update the_big_picture.rst #19820

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

Closed
wants to merge 7 commits into from
Closed

Update the_big_picture.rst #19820

wants to merge 7 commits into from

Conversation

WDI-Json
Copy link

Walkthrough first page setting up with PHP 8.3 and fresh installation of Symfony 7. I received errors with a clean setup while going through this and fixed them using my changes.

Walkthrough first page setting up with PHP 8.3 and fresh installation of Symfony 7.
@carsonbot carsonbot added this to the 7.1 milestone Apr 22, 2024
@WDI-Json
Copy link
Author

The changes aren't that much, but I needed these in order to make it work somehow :/

@OskarStark
Copy link
Contributor

Symfony 7 requires PHP 8.2

feedback received in PR
@WDI-Json
Copy link
Author

@OskarStark I've changed to 8.2.0 as stated in https://symfony.com/releases/7.0

@xabbuh
Copy link
Member

xabbuh commented Apr 30, 2024

I think we even need to do a bit more as the skeleton application looks a bit different than documented since Symfony 6 (see #19834 for my proposal).

WDI-Json and others added 2 commits May 5, 2024 14:08
Co-authored-by: Christian Flothmann <christian.flothmann@gmail.com>
@javiereguiluz
Copy link
Member

Let's close this proposal in favor of #19834, which was already merged a while ago.

Even if this wasn't merged, thanks for creating this PR because it helped us improve the related docs. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants