-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Fix path to crud in resource config in documentation #17575
Conversation
WalkthroughThe pull request introduces a comprehensive guide for adding a custom Changes
Sequence DiagramsequenceDiagram
participant Dev as Developer
participant Maker as SymfonyMakerBundle
participant Entity as Supplier Entity
participant Repo as SupplierRepository
participant Config as Sylius Configuration
participant Admin as Admin Panel
Dev->>Maker: Generate Supplier Entity
Maker-->>Entity: Create entity class
Dev->>Entity: Implement ResourceInterface
Dev->>Repo: Create repository
Dev->>Config: Register resource in sylius_resource.yaml
Dev->>Config: Update routes
Dev->>Admin: Add to admin menu
Admin->>Entity: Manage suppliers
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🔇 Additional comments (1)docs/the-customization-guide/customizing-models/how-to-add-a-custom-model.md (1)
The path change from ✅ Verification successfulTemplate path update is correct The verification confirms that Key findings:
🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Verify the existence and usage of the new template path in Sylius
# Search for other occurrences of the new template path pattern
echo "Searching for other usages of the new template path..."
rg "@SyliusAdmin\\\\shared\\\\crud" --type yaml
# Search for the old template path to ensure we're not missing any other instances
echo "Checking for any remaining instances of the old template path..."
rg "@SyliusAdmin\\\\Crud" --type yaml
Length of output: 341 Script: #!/bin/bash
# Let's try different search patterns and explore the template structure
# Search for template paths in yaml files with simpler patterns
echo "Searching for SyliusAdmin template paths..."
rg -i "syliusadmin.*template" --type yaml
# Search for crud-related template configurations
echo "Searching for crud-related template configurations..."
rg -i "crud.*template" --type yaml
# Look for any template configurations in config files
echo "Searching in config files..."
fd -e yaml -e yml -e xml config --exec grep -i "template" {} \;
# Check if there are any twig templates in the expected location
echo "Checking for twig templates..."
fd -e twig . --exec dirname {} \; | sort -u | grep -i "admin"
Length of output: 25947 Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Bunnyshell Preview Environment deletedAvailable commands:
|
It is just an documentation fix.
Summary by CodeRabbit
Documentation
Supplier
entity to Sylius, including step-by-step instructions and configuration details.New Features
Supplier
entity and its corresponding repository.Supplier
entity as a resource in the system.