Skip to content

[Route][Attribute] Fix FileLocatorFileNotFoundException micro_kernel_… #18597

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 1 commit into from

Conversation

IssamRaouf
Copy link
Contributor

…trait.rst fix

We don't have the config directory yet and in the configuredRoutes method we have imported routes from dir config that caused exception FileLocatorFileNotFoundException

To ignore this exception, either we create the config directory or override the configureRoutes method and take the statements only for load the route annotation.

…trait.rst fix

We don't have the config directory yet and in the configuredRoutes method we have imported routes from dir config that caused exception FileLocatorFileNotFoundException

To ignore this exception, either we create the config directory or override the configureRoutes method and take the statements only for load the route annotation.
@carsonbot carsonbot added this to the 6.2 milestone Jul 22, 2023
@OskarStark OskarStark modified the milestones: 6.2, 6.3 Aug 14, 2023
@OskarStark
Copy link
Contributor

Please rebase on 6.3 branch, thanks

@IssamRaouf IssamRaouf changed the base branch from 6.2 to 6.3 August 15, 2023 14:00
@IssamRaouf
Copy link
Contributor Author

@OskarStark It's done, thank you

@xabbuh xabbuh modified the milestones: 6.3, 6.4 Dec 16, 2024
@javiereguiluz
Copy link
Member

@IssamRaouf thanks for this proposal! I tested it and it works ... but I'm closing it without merging. Let me explain why:

  • This is no longer needed in newer Symfony 7.x versions
  • For the still maintained 6.4 version, we show this message before running the server:start command:

As you said, creating the config/ dir is one of the possible solutions and that's what we currently recommend in the docs. Given that this is not needed in newer Symfony versions, we prefer to keep this solution instead of adding more code that makes the full example look a bit more complex.

Thanks for understanding.

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