-
Notifications
You must be signed in to change notification settings - Fork 1
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
Warmup routes with query string #5
Comments
Hi @thisJerem , Interesting question, I see the use case for this. But how does the addon know how many paginated page there is?
This information would need to be present in the configuration somehow. I am not sure how to encode this, do you have any idea? Cheers |
Thinking again about this, the addon could simply offer a list of additional URLs to warmup, e.g.
This also allows to create the static cache for URLs resolved via template routes. @thisJerem What do you think about this? |
Hi wanze ! I think that would be great because that solved the case with template routes, mine for example are like this :
For query params your solution is cleary the most simple as I think we don't need to warmup all paginated pages but maybe only a few, at least the most consulting based on tracking data, so manual adding for this kind or url are a good choice ! I don't had time yet to test it but I was thinking about warmup the cache based on the sitemap, what do you think about it ? |
It is a super addon for Statamic but in the case we have a page with a mounted collection with pagniation, it will be great to have the possibility to "warmup" also all this pages.
We could add the same option than for whitelisting querystrings :
What do you think ? Is it possible easily ?
The text was updated successfully, but these errors were encountered: