-
Notifications
You must be signed in to change notification settings - Fork 197
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
Pass checks argument to sensei_can_user_view_lesson filter #7657
Conversation
Test the previous changes of this PR with WordPress Playground. |
Test the previous changes of this PR with WordPress Playground. |
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.
LGTM!
If you decide to apply the change I suggested, feel free to merge after tweaked since it's very small and not super relevant.
Test the previous changes of this PR with WordPress Playground. |
Resolves #7630
In the original issue, the Preview setting didn't work if the user's access expired.
Here, we pass our initial checks to the filter to allow making a final decision inside hooks.
Based on the discussion: #7655 (comment)
Proposed Changes
Testing Instructions
New/Updated Hooks
sensei_can_user_view_lesson
-$checks
param added.Pre-Merge Checklist