forked from argoproject/Argo
-
-
Notifications
You must be signed in to change notification settings - Fork 83
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
Undefined variable: post in homepages/templates/top-stories.php on line 73 #1723
Labels
Milestone
Comments
benlk
added
type: bug
priority: low
Nice-to-have in a release.
Estimate: < 2 Hours
labels
Jun 10, 2019
@benlk Do you think we should we just change
|
If possible, we should pass the WP_Post object to largo_excerpt rather than the post ID: https:/INN/largo/blob/37e7b9dbf3e40072fb15b9ebf25e81f73d20daef/inc/post-tags.php#L184-L198 |
@benlk So couldn't we just pass |
Yeah, that looks like it would work. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
With the "Top Stories" layout active,
$post
is undefined in several places on the homepage.The text was updated successfully, but these errors were encountered: