-
-
Notifications
You must be signed in to change notification settings - Fork 566
[5.x] Only apply the published filter when not in preview mode #11652
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: 5.x
Are you sure you want to change the base?
Conversation
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.
Would you be able to add a test for this?
Sure, i'll setup a test for checking if draft entries are received with a token and omitted without a token. 👍 |
I've added the testcase. While running the test I also needed to fix a line where the filters would default to |
@duncanmcclean have you had a chance to review the changes? 😃 |
Sorry, I don't normally review PRs, beyond giving them a quick look over, so I don't have an ETA for when we'll review this pull request. In the meantime, you can pull this PR into your project with a composer patch. |
Possible fix for issue #6134
Checks if we're in
isLivePreview
before appending$filters['status'] = 'published';
Works similar as the fix made for the REST API #10229