Skip to content
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

Nested CRUDs example #114

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Nested CRUDs example #114

wants to merge 3 commits into from

Conversation

tabacitu
Copy link
Member

@tabacitu tabacitu commented Dec 3, 2019

This PR includes a new demo entity, "Snippets", which can be manipulated through the sidebar entries:

  • All Snippets - regular CRUD for the Snippets model;
  • My Snippets - extends the regular SnippetCrudController to only show the logged in person's snippets;
  • Creator Snippets - nested CRUD - shows the snippets of a certain user (aka Creator);

We could include this in the main demo. We could. But in my opinion it adds quite a few more components, just to test one thing - that Nested CRUDs work. Food for thought...

@srigurubyo
Copy link

I am looking to run this sample but some of the functions are deprecated in new version (ex: str_limit()).

Can you please help me with the relationship_count.blade.php particularly as I am not very well versed in php coding.

Thanks!

@tabacitu
Copy link
Member Author

str_limit() should be replaced with Str::limit() as per Laravel 7 requirement - the others should be similar.

Sorry @srigurubyo this is not a priority, so it won't get any attention soon. Happy to merge PRs into this if anybody fixes it for Backpack 4.1.

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

Successfully merging this pull request may close these issues.

2 participants