Skip to content

Commit

Permalink
Merge pull request #318 from 10up/fix/release-1.8.1-issues-313
Browse files Browse the repository at this point in the history
Fix issues reported for v1.8.1 release
  • Loading branch information
dkotter authored Sep 20, 2024
2 parents 1872aaf + 33d39ae commit 48bf448
Show file tree
Hide file tree
Showing 13 changed files with 703 additions and 286 deletions.
6 changes: 5 additions & 1 deletion .github/workflows/phpcs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,13 @@ on:
branches:
- develop
- trunk
paths:
- "**.php"
pull_request:
branches:
- develop
paths:
- "**.php"

jobs:
phpcs:
Expand All @@ -28,4 +32,4 @@ jobs:
run: composer install

- name: Test
run: ./vendor/bin/phpcs -v
run: ./vendor/bin/phpcs --runtime-set testVersion 7.4 .
4 changes: 2 additions & 2 deletions assets/js/edit.js
Original file line number Diff line number Diff line change
Expand Up @@ -436,11 +436,11 @@ function Edit( props ) {
{src ? (
<>
<div className="wp-block-podcasting-podcast__container">
{showImage && displayArt && (
{displayArt && (featuredImageUrl || showImage) && (
<div className="wp-block-podcasting-podcast__show-art">
<div className="wp-block-podcasting-podcast__image">
<img
src={showImage}
src={featuredImageUrl ? featuredImageUrl : showImage}
alt={showName}
/>
</div>
Expand Down
4 changes: 2 additions & 2 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,10 @@
"php": ">=7.3"
},
"require-dev": {
"10up/phpcs-composer": "dev-master",
"10up/phpcs-composer": "^3.0",
"10up/wp_mock": "^0.4.2",
"phpunit/phpunit": "^9.5",
"phpcompatibility/phpcompatibility-wp": "*"
"phpcompatibility/php-compatibility": "dev-develop as 9.99.99"
},
"config": {
"allow-plugins": {
Expand Down
Loading

0 comments on commit 48bf448

Please sign in to comment.