-
Notifications
You must be signed in to change notification settings - Fork 130
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
Inline Operation - Document if it requires 'fetch' operation also #489
Comments
Yes, you need to use the fetch operation. It's documented here under the Hope this would get you through. Please remember to close the issue. For more queries, this would be a better place https://github.com/Laravel-Backpack/community-forum/issues |
Thanks, but add a side note into the documentation of I suggest to explain it also here: https://backpackforlaravel.com/docs/6.x/crud-operation-inline-create because it's more related to inline that to relationship Also, this is not a question about how to do things. We already found the answer, it's a (big) suggestion to improve doc. This is why we decided to open a doc issue and not a Q/A on forum. Also, as you can see, we are NOT using |
Hmm I don't think that's intentional. I think that's a bug, to be honest. If you ask me, you should be able to use @pxpm let me do a TLDR for you (everybody correct me if I'm wrong here):
Are you aware of this restriction @pxpm ? Did we choose to do this? Should we document or fix this? |
What I get after reading the conversations is, Info is already there but not in the highlights. @tabacitu do you feel to do something here? It feels like you as the right person to take over this. |
Page: https://backpackforlaravel.com/docs/6.x/crud-operation-inline-create
I am setting up field as
Note: i am omitting ajax
Anyway, this field is automatically a select 2 with ajax.
This need fetch operation, I think. Because opening the select2, an ajax call to a non-existent route ../fetch/... is fired
Please, document it, (or explain me what am I doing wrong).
The text was updated successfully, but these errors were encountered: