-
-
Notifications
You must be signed in to change notification settings - Fork 540
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
Related items limit exceeded due to GET request query length #4264
Comments
There's no problem that you're wanting to select so many items, but I am curious what the use case is there. |
I can't tell too much, but it's basically a portal where the client can add items to a city. |
Hey @jasonvarga, I didn't realize you are working at Statamic. We are currently using Statamic for a bigger application where we often have to deal with a large number of collections, entities, and We just want to make sure it is the right tool for our use case. If this is something Statamic is not built for, please let us know, because otherwise, we might run into similar issues more often on the way. |
One could resort to using POST for storing the selection, or to split the GET request into multiple – but I don’t think that second approach would scale too well, since the more selections, the more requests, the longer the user would have to wait. |
It was only a GET route because we were using the CRUD methodology. Changing a route to POST isn't the end of the world. |
Hey @jasonvarga,
|
This is next on @jesseleite's list. I don't think there's a problem with doing what you're doing. Once it's addressed you can try it out. |
Thanks @jasonvarga. |
Bug Description
When you have a
relationship:fieldtype
there is currently alimit
of related entries to select due to the fact that aGET
request is being used to store them.In our case, the limit was
216
, and it failed with trying to add one more. (see screenshot below)How to Reproduce
relationship:fieldtype
select
button to saveExtra Detail
Given error message:
Selecting the entries:
The issue is similar to #3293, except that we haven't customized any limit. (from what I see)
Environment
Statamic 3.2.4 Pro
Laravel 8.60.0
PHP 8.0.8
Install method (choose one):
statamic/statamic
The text was updated successfully, but these errors were encountered: