8000 [New routing] Convert sylius_admin_customer_order_index route by loic425 · Pull Request #18744 · Sylius/Sylius · GitHub
[go: up one dir, main page]
Skip to content

[New routing] Convert sylius_admin_customer_order_index route #18744

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.

Sign up for GitHub

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

Jump to bottom
Open
loic425 wants to merge 1 commit into
base: new-routing
Choose a base branch
from
Open

[New routing] Convert sylius_admin_customer_order_index route #18744

loic425 wants to merge 1 commit into from
+12 −0

Conversation

@loic425
Copy link
Member
@loic425 loic425 commented Jan 22, 2026
Q A
Branch? new-routing
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Related tickets partially #18212
License MIT

@loic425 loic425 requested review from a team as code owners January 22, 2026 13:00
@probot-autolabeler probot-autolabeler bot added the Admin AdminBundle related issues and PRs. label Jan 22, 2026
@coderabbitai
Copy link
coderabbitai bot commented Jan 22, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @codera 8000 bbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
github-actions bot commented Jan 22, 2026 •
edited
Loading

❗ Preview Environment deployment failed on Bunnyshell

See: Environment Details | Pipeline Logs

Check https://github.com/Sylius/Sylius/actions/runs/21249443987 for details.

Available commands:

  • 🚀 /bns:deploy to redeploy the environment
  • ❌ /bns:delete to remove the environment

@loic425 loic425 mentioned this pull request Jan 22, 2026
Open
50 tasks
loic425
loic425 commented Jan 22, 2026
View reviewed changes
src/Sylius/Bundle/AdminBundle/Resources/config/app/sylius/resources/order.php Outdated
),
new Index(
path: '/customers/{id}/orders',
routeName: 'sylius_admin_customer_order_index',
Copy link
Member Author
@loic425 loic425 Jan 22, 2026

Choose a reason for hiding this comment

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

Suggested change
routeName: 'sylius_admin_customer_order_index',
routeName: '_sylius_admin_customer_order_index',

@loic425 loic425 force-pushed the admin-customer-orders branch from a69f3a4 to bdb4b09 Compare January 22, 2026 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Admin AdminBundle related issues and PRs.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@loic425
0