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

Jump to bottom

Route expose false #404

Merged
merged 2 commits into from
Dec 15, 2021
Merged

Route expose false #404

merged 2 commits into from
Dec 15, 2021

Conversation

tugrul
Copy link
Contributor

@tugrul tugrul commented Apr 4, 2021

fixes expose false option #401

@tobias-93
Copy link
Collaborator

Maybe it is nice to use the isRouteExposed method here, so no double implementation can exist?

@tobias-93
Copy link
Collaborator

I see why you did not use that method, so merging now.

@tobias-93 tobias-93 merged commit 6c9ce16 into FriendsOfSymfony:master Dec 15, 2021
@tobias-93 tobias-93 mentioned this pull request Dec 15, 2021
Closed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tugrul @tobias-93