github.com/wp-graphql/wp-graphql commits | Ecosyste.ms: OpenCollective

Merge pull request #3057 from jasonbahl/fix/3056-allow-null-for-admin_enqueu_scripts-callback

fix: admin_enqueue_scripts callback should expect a possible null value passed to it

a2cab54948501ae6db25ddd431aef9d954e1fb49 authored 11 months ago by Jason Bahl <[email protected]>

Update wordpress-coding-standards.yml

revert back to old workflow

437f7f7d6aa79bfee09c4fb4540e6e721f896bb3 authored 11 months ago by Jason Bahl <[email protected]>

Update wordpress-coding-standards.yml

f19362363955bd8266239ae85e808c78ca6333e5 authored 11 months ago by Jason Bahl <[email protected]>

Update wordpress-coding-standards.yml

f1a59ac1ac80c57581300399d84003914015cffd authored 11 months ago by Jason Bahl <[email protected]>

Update wordpress-coding-standards.yml

99b77b4574792f61d734ff3b36fc0eedc1598864 authored 11 months ago by Jason Bahl <[email protected]>

Merge branch 'ci/modify-wordpress-coding-standards-to-autofix-when-possible' into develop

86cc495fe232ab9609ebc01d368150b37a6080d1 authored 11 months ago by Jason Bahl <[email protected]>

- update workflow to auto-commit phpcbf fixed files then run phpcs to output any other errors

a53948e6502798a397c49e05b4a6e0e920764b1b authored 11 months ago by Jason Bahl <[email protected]>

- composer fix-cs

7093cc82a1194b67211f9f051db29b9246011a16 authored 11 months ago by Jason Bahl <[email protected]>

- if the admin_enqueue_scripts hook passes null for the $hook_suffix argument don't throw errors. It's supposed to pass a string, but it's been reported to not always pass a string.

da96afe157b3bfc6d1ba2e01bd24dfd3395214db authored 11 months ago by Jason Bahl <[email protected]>

Merge pull request #3048 from jasonbahl/fix/2514-is-posts-page-on-content-type

fix: isPostsPage on content type

b2712bd28d45c558d9a94c888859534eac553b1f authored 11 months ago by Jason Bahl <[email protected]>

- composer fix-cs

7ca1dcd0c87f6dc9c0d17aa76576f74d999bf750 authored 11 months ago by Jason Bahl <[email protected]>

Merge branch 'develop' into fix/2514-is-posts-page-on-content-type

969650ab0cd29cdc8a4e2dbcc0a5b1272a68be32 authored 11 months ago by Jason Bahl <[email protected]>

- composer fix-cs

d0794996c0455e37c0ba723a2a5a9ea4d76d8989 authored 11 months ago by Jason Bahl <[email protected]>

- revert change to the Post.isPostsPage model logic

f8f44372d9645de0227af8a620d91e870df4c763 authored 11 months ago by Jason Bahl <[email protected]>

- add `isPostsPage` and `isFrontPage` to the UniformResourceIdentifiable Interface as any node with a URI should be able to identify whether it is or is not the posts page or front page. Defaults to false, but individual implementing types (ContentType, Page) can override the resolver as applicable.

- update the PostType Model to return true if show_on_front is set to "page" and update Post mode...

3744e1d68cdde3cf7a31628edd44ff5d8beaf055 authored 11 months ago by Jason Bahl <[email protected]>

- adds test to assert that when a query to the uri for the blog page returns a `ContentType` that `isPostsPage` field returns true

710943672d0c686056f6cc6b1c70b37604a89961 authored 11 months ago by Jason Bahl <[email protected]>

update mutations guide to show extending existing mutations

54d03523d4fee4064900e093be1518cbacb2cb41 authored 11 months ago by Jason Bahl <[email protected]>

Merge pull request #3044 from justlevine/dev/graphql_pre_resolve_menu_item_connected_node

feat: add `graphql_pre_resolve_menu_item_connected_node` filter

e8262a93cc1927f1826cd5cb2d3650eb907c5952 authored 11 months ago by Jason Bahl <[email protected]>

Merge pull request #3039 from justlevine/feat/comment-uri

feat: add `UniformResourceIdentifiable` interface to `Comment` type

c7f03fe68e9fa97b41136237d93ea9916669cd90 authored 11 months ago by Jason Bahl <[email protected]>

Merge pull request #3043 from justlevine/fix/menu-items-location-filter

fix: return empty when filtering `menuItems` by a `location` with no assigned items

387b289ce3efe043bae48e360ce662754706db61 authored 11 months ago by Jason Bahl <[email protected]>

Merge pull request #3045 from justlevine/fix/user-searchColumns-filter

fix: `UsersConnectionSearchColumnEnum` values should be prefixed with `user_`

0fd78202afec3d5b35cb1184fa56072845b1a73b authored 11 months ago by Jason Bahl <[email protected]>

Merge pull request #3020 from MuhammedAO/check-if-nodes-are-resolved

feat: check if nodes are resolved

f4c4cc74bda6ed18e69817ebd8b566ef4c962b1e authored 11 months ago by Jason Bahl <[email protected]>

- remove changes in favor of the new filter

903d84d154ce70f8e46343aab99225c338f3c565 authored 11 months ago by Jason Bahl <[email protected]>

Merge branch 'check-if-nodes-are-resolved' of github.com:MuhammedAO/wp-graphql into check-if-nodes-are-resolved

1b843b07c538463b5f1908c5c24dca414cd50745 authored 11 months ago by Jason Bahl <[email protected]>

- revert changes, add filter

958e78e5191459f829503d2cf6bd699fc69dde23 authored 11 months ago by Jason Bahl <[email protected]>

chore: check if nodes are resolved

ebef58cbac2b8e045964e612a403fd172e3b6db5 authored 11 months ago by Muhammed ogunsanya <[email protected]>

fix: `UsersConnectionSearchColumnEnum` values should be prefixed by `user_`.

ada326e054449a8bd30e5e512b3350259987cae6 authored 11 months ago by David Levine <[email protected]>

dev: add `graphql_pre_resolve_menu_item_connected_node` filter

e65fbc84ddd6929c851002f48cd47e971ea50b9b authored 11 months ago by David Levine <[email protected]>

tests: create ignored menuItems at start of testLocationWhereArgs

670b1bf2d6fb56cdc31f89dc6f0f811876e80363 authored 11 months ago by David Levine <[email protected]>

fix: return empty when filtering `menuItems` by a `location` with no items

40b0d76f531fddfefb79b737c3624dbe485ab462 authored 11 months ago by David Levine <[email protected]>

Auto-merge master back to develop

afe6a522eb8e322b80ad8c1b1374f13b0c0d5c79 authored 11 months ago by Github Actions <[email protected]>

Merge pull request #3041 from wp-graphql/release/v1.21.0

release: v1.21.0

8b106a542e75b844457c42452b7e4bad98b5a741 authored 11 months ago by Jason Bahl <[email protected]>

- only run GraphiQL unit and e2e tests manually (they're currently broken and being ported to the new wpgraphql-ide repo where we'll get them all up and running properly again)

052fe78c2e56429bd64d1eac5a4fd2eab7994115 authored 11 months ago by Jason Bahl <[email protected]>

- replace @since versions

- remove some unneeded todo comments

fb2f6b09e271216453e842ebb7f2715b4582f381 authored 11 months ago by Jason Bahl <[email protected]>

Merge commit '28f09523c4637d2dc6ce723042819a863253bd00' into release/v1.21.0

4ee748f45eb9d5072a488308d7e53daf413104ae authored 11 months ago by Jason Bahl <[email protected]>

- update versions, changelog for release

4d0a9c2c559c9a3d3162bf634f252bcd8724820c authored 11 months ago by Jason Bahl <[email protected]>

Merge pull request #3027 from wp-graphql/acf-deprecation-notice

feat: Add Admin Notice for WPGraphQL for ACF

28f09523c4637d2dc6ce723042819a863253bd00 authored 11 months ago by Jason Bahl <[email protected]>

- upgrade composer

e6cd74a4725b92043b5b1c116cd5e1d6aaaa3cc6 authored 11 months ago by Jason Bahl <[email protected]>

Merge branch 'develop' into acf-deprecation-notice

198c321130d11367b84ab5148c75ea1017bb9d81 authored 11 months ago by Jason Bahl <[email protected]>

- regenerate composer.lock

9c7a37e5ae760edcdfba266c1bc2afd92640b85a authored 11 months ago by Jason Bahl <[email protected]>

- merge

c90ae7eae68c130a78602ad3ac6ad324bf0ccc9b authored 11 months ago by Jason Bahl <[email protected]>

rebuild composer.lock

8ea75efa0565dd6e8095822fefeeb12749a9eac4 authored 11 months ago by Jason Bahl <[email protected]>

test: cleanup after testCommentByUri()

d637750e8f920049c81a4f224c7bb7bf8e75d6c3 authored 11 months ago by David Levine <[email protected]>

chore: cleanup

2c690e3f28117ff3d3e3bbd4e3e9222a81adb7f4 authored 11 months ago by David Levine <[email protected]>

Merge pull request #3035 from jasonbahl/feat/3034-provide-better-error-when-type-does-not-exist

feat: provide better error when field references a type that does not exist

121a691189fbf332026c479c8b0940fb91eebcef authored 11 months ago by Jason Bahl <[email protected]>

feat: make `Comment` implement `UniformResourceIdentifiable`.

8295ff015c62ffb62ff0c97c2369e164cc426884 authored 11 months ago by David Levine <[email protected]>

- remove wp-mock

- extend WPGraphQLTestCase instead of TestCase

b3e6a21b9dd48a3faa2620dbbf69a1886a80b6e6 authored 11 months ago by Jason Bahl <[email protected]>

Merge pull request #3038 from wp-graphql/dependabot/composer/composer/composer-2.2.23

chore(deps-dev): bump the composer group across 1 directories with 1 update

1a7f4b0bf0d1ece6cd596dfda5c338125ddb0912 authored 11 months ago by Jason Bahl <[email protected]>

- esc_html and update translators comment

35e49fc1ca13f1bfeee2949b261ce3eedc0c797f authored 11 months ago by Jason Bahl <[email protected]>

chore(deps-dev): bump the composer group across 1 directories with 1 update

Bumps the composer group with 1 update in the /. directory: [composer/composer](https://github.c...

aba818d19df79d8ab509ee3c0ba8d40c4a479656 authored 11 months ago by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Update src/Admin/AdminNotices.php

022983f0e1fb8c9e0d40dfe17a48e013ec4f3238 authored 11 months ago by Jason Bahl <[email protected]>

Update src/Admin/AdminNotices.php

625843a22d8618f9bcdcd2f85bdec56bcc7d3ab1 authored 11 months ago by Jason Bahl <[email protected]>

Resolve linting issues

Signed-off-by: Joe Fusco <[email protected]>

433179e1c8b4ee734377f8eb6c11c3d628c9f62b authored 11 months ago by Joe Fusco <[email protected]>

Clean up white space

Signed-off-by: Joe Fusco <[email protected]>

8e92bb9b40628e8f786cd058355f9a0cf8404089 authored 11 months ago by Joe Fusco <[email protected]>

Config validation

Signed-off-by: Joe Fusco <[email protected]>

a84d1b06b14231fd3e0cdccd08a505f5996322c5 authored 11 months ago by Joe Fusco <[email protected]>

Display notices in Network admin

Signed-off-by: Joe Fusco <[email protected]>

18700e481756e3b4160d8545c419578b24a0d58c authored 11 months ago by Joe Fusco <[email protected]>

- add test (fails without new error message in place)

14a51b8ad9d92302f2d5582f637aa2c7f6cab591 authored 11 months ago by Jason Bahl <[email protected]>

- add more helpful error message when a field is registered to the Schema referencing a non-existent Type

524f3327e10a2e9a5d3d37a75d809a0454e30540 authored 11 months ago by Jason Bahl <[email protected]>

- add basic tests for AdminNotices

6a1294140c2f73f7fe802ec059d15180e1fc1edc authored 11 months ago by Jason Bahl <[email protected]>

Merge pull request #3031 from linucks/graphiql-broken-schema

fix(graphiql): Allow GraphiQL to run even if a valid schema cannot be returned.

f480fd23db9d0f09814afdcd635e453b79b6f42b authored 11 months ago by Jason Bahl <[email protected]>

- update build assets

624564325ec3c4b82d0480c470d4363fe43c3fda authored 11 months ago by Jason Bahl <[email protected]>

Merge pull request #3033 from jasonbahl/fix/2981-php-deprecation-error

fix: php deprecation error for dynamic properties on AppContext class

669898279412c51aee6b488a05501553b91428ad authored 11 months ago by Jason Bahl <[email protected]>

- update tests to drop php8.0 from the matrix and drop wp 6.1

- Add php8.0 + wp6.3 to the include list

a3696fffe3131db82452d3c68a0c6315d30a2131 authored 11 months ago by Jason Bahl <[email protected]>

- add connection_query_class as a defined property

- add // @phpcs:ignore to the #[\AllowDynamicProperties] line

af91b551a17739ac1e1f7e0b25771088c94e7da0 authored 11 months ago by Jason Bahl <[email protected]>

- add support for the red bubble on the admin menu

f0dd0996b0484df56b9f15e72e3c93822ea12edd authored 11 months ago by Jason Bahl <[email protected]>

Refactor to use usermeta instead of option table

Signed-off-by: Joe Fusco <[email protected]>

69d383c208c90c845e2b39a4354a7f2e11a4dd15 authored 11 months ago by Joe Fusco <[email protected]>

Include space for dismissable icon

Signed-off-by: Joe Fusco <[email protected]>

f6451aa6c74167506a5bcb17f8a17b5ae752e810 authored 11 months ago by Joe Fusco <[email protected]>

Resolve linting issues

Signed-off-by: Joe Fusco <[email protected]>

0045c0a8c2efd4c9eeb4255e8918b7f9a9d6668e authored 11 months ago by Joe Fusco <[email protected]>

Resolve linting issues

Signed-off-by: Joe Fusco <[email protected]>

dcd2cdc8554ccf93ef50652b1bc6c92d966ffa14 authored 11 months ago by Joe Fusco <[email protected]>

Include unit tests for AdminNotices

Signed-off-by: Joe Fusco <[email protected]>

5d495fc37a8adba90950e1f3f0329446763ea872 authored 11 months ago by Joe Fusco <[email protected]>

Add WP_Mock dep

Signed-off-by: Joe Fusco <[email protected]>

5880d4e77e1e8fa6f946ad676b47f31371440800 authored 11 months ago by Joe Fusco <[email protected]>

Remove unused variable

1d11f787e360c605d114c593d36ae37b9d81941e authored 11 months ago by linucks <[email protected]>

no message

0cbebc393fdc5fc17d4872327245c28a6794863f authored 11 months ago by Jason Bahl <[email protected]>

- add #[\AllowDynamicProperties] to AppContext class

5c32028d5a446e1416e337713322df8500cdffad authored 11 months ago by Jason Bahl <[email protected]>

- removed unused get_admin_notice function

- replaced unset calls with $this->remove_admin_notice

a42f47537b7585d1b5cf7a8aa681ca8bab9b86df authored 11 months ago by Jason Bahl <[email protected]>

no message

77d19177a6cd0ff855c931e235b06b3f000c7ca2 authored 11 months ago by Jason Bahl <[email protected]>

Allow GraphiQL to run even if a valid schema cannot be returned.

547432d242fc2ec6ce03a73157e41588f34c1d3e authored 11 months ago by linucks <[email protected]>

- move admin notices to a different class

8f61df93d9708bab0b706d1c2fb2a5dfc1e4b028 authored 11 months ago by Jason Bahl <[email protected]>

Merge commit '39429412cf78ccaa7caa93fa1889c344585f89bf' into acf-deprecation-notice

5d15b6ea8be7519e8cdce702f6936cee85b33e36 authored 11 months ago by Jason Bahl <[email protected]>

Add admin notice

Signed-off-by: Joe Fusco <[email protected]>

abfcb6c9361940991b01f71c6a323f24868f2684 authored 11 months ago by Joe Fusco <[email protected]>

Auto-merge master back to develop

49e2d5a24142d855bcbefa2738333647c2f3d0b9 authored 12 months ago by Github Actions <[email protected]>

Merge pull request #3025 from wp-graphql/release/v1.20.0

release: v1.20.0

39429412cf78ccaa7caa93fa1889c344585f89bf authored 12 months ago by Jason Bahl <[email protected]>

Merge commit 'cb3c1b68176ad223fe6a53452f6bedb601b3e943' into release/v1.20.0

25d9ef38e4555b9f987488b4ec7c668405e1968e authored 12 months ago by Jason Bahl <[email protected]>

- update versions, changelog for release

96cabc9d2675ec8a666d68e7e4cd170b053793ad authored 12 months ago by Jason Bahl <[email protected]>

Merge pull request #3022 from justlevine/vanity/500th-commit

chore: add @justlevine to list of contributors

cb3c1b68176ad223fe6a53452f6bedb601b3e943 authored 12 months ago by Jason Bahl <[email protected]>

chore: add @justlevine to list of contributors

1b66feb82aa71576f2d92857dfe32553f4b4ba5e authored 12 months ago by David Levine <[email protected]>

Merge pull request #3009 from justlevine/chore/strict-phpstan-fixes

chore: implement stricter PHPStan config and clean up unnecessary type-guards

5db11fb319297d8d742785b83d15439f3c26af4c authored 12 months ago by Jason Bahl <[email protected]>

Merge commit '8ee0e4c6e3b51e8c10bdd48e92645c591e8a3ef1' into chore/strict-phpstan-fixes

- update phpstan.neon.dist

19440ead7351bc3d79a9a285573606a8980f1918 authored 12 months ago by Jason Bahl <[email protected]>

Merge pull request #3008 from justlevine/dev/disable-qa-by-default

perf: Expose `graphql_should_analyze_queries` as setting

bed443d6a9f8a656da6dd1055df3141c633b0de2 authored 12 months ago by Jason Bahl <[email protected]>

Merge pull request #3006 from justlevine/fix/replace-deprecated-setQueryArg

fix: replace deprecated `AbstractConnectionResolver::setQueryArg()` call with `::set_query_arg()`

fa56cf17643d34b1ac35e95fa73674053d667223 authored 12 months ago by Jason Bahl <[email protected]>

revert: don't disable by default / update db

8ee0e4c6e3b51e8c10bdd48e92645c591e8a3ef1 authored 12 months ago by David Levine <[email protected]>

chore: check if nodes are resolved

f43255d73d126f5b7fd9e7b180b65df2494c4f2e authored 12 months ago by Muhammed ogunsanya <[email protected]>

dev: fix edge cases and add unit tests

8ab4aa4d7ed11bf9dc169df2fe15785a0e108a4d authored 12 months ago by David Levine <[email protected]>

Update src/Router.php

2620e07effdfbd877c087c4528872a8fcf90cebf authored 12 months ago by Dovid Levine <[email protected]>

Update activation.php

Set default value for get_option

fc6b1d90b52cadbc3d0c074a21f7b2c2c09843a8 authored 12 months ago by Jason Bahl <[email protected]>

dev: rework as exposed setting

0b82e9917dd1a916cb679f0d62a5149a9c47c2d2 authored 12 months ago by David Levine <[email protected]>

dev: default `graphql_should_analyze_queries` value to GraphQL Debugging status

95704eed28fccb452f583459a9949e8a0956004f authored 12 months ago by David Levine <[email protected]>

fix: check if query anaylzer is enabled before setting headers

38c079afbd1a73511830f4be7f29a4c4dc916237 authored 12 months ago by David Levine <[email protected]>

test: test graphql keys with debugging enabled

8e9ce6af4951c7f8688142243364d00c498b33e5 authored 12 months ago by David Levine <[email protected]>

Merge pull request #3010 from justlevine/chore/stricter-phpdoc-types

chore: implement stricter PHPDoc types

3d3d723c73b71ab1fa88aed745bb9f41d497dd9b authored 12 months ago by Jason Bahl <[email protected]>