Skip to content
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

Show the template that is being used on the page. #35

Merged
merged 2 commits into from
Jun 3, 2021

Conversation

StevenDufresne
Copy link
Contributor

@StevenDufresne StevenDufresne commented Jun 1, 2021

Try including the template file when explaining the source of the error.

@carolinan Thoughts?

Related to: #34

"/?p=1241" (via: single.php) contains PHP errors: Notice: Undefined variable: bool in wp-content/themes/test-theme/functions.php on line 145
See: https://github.com/WordPress/theme-review-action/blob/trunk/docs/ui-errors.md#page-should-not-have-php-errors


"/?p=1241" (via: single.php) contains javascript errors. Found Error: TypeError: Cannot read property 'querySelectorAll' of null  at new navMenu (assets/js/primary-navigation.js?ver=1.0.1:170:47)  at assets/js/primary-navigation.js?ver=1.0.1:182:3
See: https://github.com/WordPress/theme-review-action/blob/trunk/docs/ui-errors.md#browser-console-should-not-contain-errors


"bigtime.com" found when viewing "/?p=1241" (via: single.php) is not an approved link.
See: https://github.com/WordPress/theme-review-action/blob/trunk/docs/ui-errors.md#page-should-not-have-unexpected-links


"/" (via: index.php) contains PHP errors: Notice: Undefined variable: bool in wp-content/themes/test-theme/functions.php on line 145
See: https://github.com/WordPress/theme-review-action/blob/trunk/docs/ui-errors.md#page-should-not-have-php-errors


"/" (via: index.php) contains javascript errors. Found Error: TypeError: Cannot read property 'querySelectorAll' of null  at new navMenu (assets/js/primary-navigation.js?ver=1.0.1:170:47)  at assets/js/primary-navigation.js?ver=1.0.1:182:3
See: https://github.com/WordPress/theme-review-action/blob/trunk/docs/ui-errors.md#browser-console-should-not-contain-errors


"/?cat=1" (via: archive.php) contains PHP errors: Notice: Undefined variable: bool in wp-content/themes/test-theme/functions.php on line 145
See: https://github.com/WordPress/theme-review-action/blob/trunk/docs/ui-errors.md#page-should-not-have-php-errors


"/?cat=1" (via: archive.php) contains javascript errors. Found Error: TypeError: Cannot read property 'querySelectorAll' of null  at new navMenu (assets/js/primary-navigation.js?ver=1.0.1:170:47)  at assets/js/primary-navigation.js?ver=1.0.1:182:3
See: https://github.com/WordPress/theme-review-action/blob/trunk/docs/ui-errors.md#browser-console-should-not-contain-errors


"/?tag=alignment-2" (via: archive.php) contains PHP errors: Notice: Undefined variable: bool in wp-content/themes/test-theme/functions.php on line 145
See: https://github.com/WordPress/theme-review-action/blob/trunk/docs/ui-errors.md#page-should-not-have-php-errors


"/?tag=alignment-2" (via: archive.php) contains javascript errors. Found Error: TypeError: Cannot read property 'querySelectorAll' of null  at new navMenu (assets/js/primary-navigation.js?ver=1.0.1:170:47)  at assets/js/primary-navigation.js?ver=1.0.1:182:3
See: https://github.com/WordPress/theme-review-action/blob/trunk/docs/ui-errors.md#browser-console-should-not-contain-errors


"/?post_format=gallery" (via: archive.php) contains PHP errors: Notice: Undefined variable: bool in wp-content/themes/test-theme/functions.php on line 145
See: https://github.com/WordPress/theme-review-action/blob/trunk/docs/ui-errors.md#page-should-not-have-php-errors


"/?post_format=gallery" (via: archive.php) contains javascript errors. Found Error: TypeError: Cannot read property 'querySelectorAll' of null  at new navMenu (assets/js/primary-navigation.js?ver=1.0.1:170:47)  at assets/js/primary-navigation.js?ver=1.0.1:182:3
See: https://github.com/WordPress/theme-review-action/blob/trunk/docs/ui-errors.md#browser-console-should-not-contain-errors

@carolinan
Copy link
Contributor

Yes this is an improvement that will help with troubleshooting.

@StevenDufresne StevenDufresne changed the title [WIP] Show the template that is being used on the page. Show the template that is being used on the page. Jun 3, 2021
@StevenDufresne StevenDufresne merged commit 045b8fb into trunk Jun 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants