[GitHub] jfmitchell opened a new pull request #2: Adding a new option to make adding the suffix to the include optional

[GitBox <gi...@apache.org>]

jfmitchell opened a new pull request #2: Adding a new option to make adding the suffix to the include optional
URL: https://github.com/apache/sling-org-apache-sling-dynamic-include/pull/2
 
 
   We had a use case where by we needed to make the include semi dynamic. We wanted the parent page that included suffixes to not be cacheable - but wanted the included resources to be cached. This was for a search results page, where it was necessary to cache the content of an individual search result - but not the actual results to display.
   
   However, since the parent request had a suffix - this was passed onto the included resources, which made them non-cacheable. This "appendSuffix" option was added to maintain the current default behaviour, but allow it to be toggled off.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services