Steven Dufresne
Steven Dufresne
I originally went with an HTML approach but had to abandon it. I'm cursing myself for not documenting why. I think it was related to overflow... results ended up being...
Fortunately I said "I think" so I can't be held liable :). I'll look through the commit history and add my findings to this thread.
The change was made in .org before the initial commit so I can't find the reason. It's worth trying again with fresh eyes to confirm the approach. 👍
Reopening this issue to add more information and test ideas.
Perfect. Thanks for the ticket. >I find the /?p=1241 confusing. I understand that this is a page or post id but when troubleshooting, it would be more helpful to know...
> I may be doing something wrong. > > If the path to theme review action is `D:/theme-review-action/` > and my theme is in: `D:\wordpress-develop\src\wp-content\themes\twentytwentyone` > > Then `npm run...
I don't have the problem on my windows VM in both bash or powershell: Can you make sure of the following: 1. You have the newest theme-review-action changes. 2. You...
I'm not sure if we need this to work on your machine seeing that the `npm run {command} -- --prop` API is well documented and appears specific to your environment.
> Except I am not the only person experiencing these errors. I had to recommend another tester to place the theme in the folder manually. Yep, definitely agree with that...
> This is a child theme, this seems like a false positive: > https://themes.trac.wordpress.org/ticket/98667#comment:7 No, this is a problem with the [parent theme](https://themes.svn.wordpress.org/blogjr/1.0.8/inc/construct.php): ``` printf( esc_html__( ' BlogJr by %1$s...