-
Notifications
You must be signed in to change notification settings - Fork 800
Add info about IPublishedContentQuery
in background task
#6997
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
base: main
Are you sure you want to change the base?
Conversation
IPublishedContentQuery
in background task
@kjac @AndyButland I added some documentation about this and not just on comment of method in source code :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @bjarnef for the PR 🙌 I have added a suggestion to keep the warning concise and moved the extra info as a paragraph above and below it.
Let me know if it doesn't make sense, or feel free to adjust.
Co-authored-by: Esha Noronha <[email protected]>
@eshanrnh great! 🙌 it should probably also be added to v13 docs .. and a link from Delivery API filter from here https://docs.umbraco.com/umbraco-cms/reference/content-delivery-api/extension-api-for-querying#custom-filter would be great. |
Hi all, @bjarnef thanks for this 💪 I do feel this is very specific for |
@eshanrnh yes, I think it would be great to move it to https://docs.umbraco.com/umbraco-cms/reference/querying/ipublishedcontentquery and perhaps we can link to more info about Feel free to make any adjustments to this PR 🙌 |
Description
Based on issue: umbraco/Umbraco-CMS#18883
It would be useful to link to this from here regarding documentation about custom filter:
https://docs.umbraco.com/umbraco-cms/reference/content-delivery-api/extension-api-for-querying#custom-filter
Type of suggestion
Product & version (if relevant)
Deadline (if relevant)
When should the content be published?