Skip to content

Conversation

@justlevine
Copy link
Collaborator

What does this implement/fix? Explain your changes.

This PR updates appsero/client to v2.0.4, which contains the compatibility fix for WP 6.7

To prevent 3rd party conflicts (and a repeat of #2599), use of Appsero\Client has been wrapped in a try/catch block.

Important

This PR is based on #3242 which should be merged first.

Does this close any currently open issues?

Fixes #3239

Any relevant logs, error output, GraphiQL screenshots, etc?

Any other comments?

Alternatively tried namespacing the dependency (which is the code-ideal solution), however Strauss is not compatible with PHP 7.3 (what Composer uses to compile on).

Where has this been tested?

Operating System: Ubuntu 24.04 (WSL + ddev + PHP 8.2.24 )

WordPress Version: 6.7.1

@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit 29cfbf4 and detected 0 issues on this pull request.

View more on Code Climate.

@justlevine justlevine added type: bug Issue that causes incorrect or unexpected behavior status: in review Awaiting review before merging or closing needs: reviewer response This needs the attention of a codeowner or maintainer labels Nov 30, 2024
@coveralls
Copy link

Coverage Status

coverage: 83.968%. remained the same
when pulling 29cfbf4 on fix/appsero-client
into ff78dd4 on develop.

jasonbahl
jasonbahl previously approved these changes Dec 3, 2024
@justlevine justlevine dismissed jasonbahl’s stale review December 3, 2024 22:19

The merge-base changed after approval.

@jasonbahl jasonbahl merged commit ebae1d7 into develop Dec 3, 2024
33 checks passed
@jasonbahl jasonbahl mentioned this pull request Dec 4, 2024
@Messa1
Copy link

Messa1 commented Dec 9, 2024

@justlevine I got some error on updating over composer

wp-graphql-smart-cach
wp-graphql/wp-graphql-smart-cache

Mabye, also need some composer update?

I can't install the new wp-graphql/wp-graphql version.

@justlevine
Copy link
Collaborator Author

@Messa1 can you be more specific about what "over composer" means? Are you using WPackagist or something else?

@Messa1
Copy link

Messa1 commented Dec 9, 2024

@justlevine

https://github.com/wp-graphql/wp-graphql-smart-cache/blob/5ffd6f59538800342ec2257754b7d9e5d8638c9e/composer.json#L41

https://github.com/wp-graphql/wpgraphql-acf/blob/ef88bd167a7b29489aa56027635f07385b6be443/composer.json#L14

composer.json

  "require": {
    "php": ">=8.0",
    "composer/installers": "^2.2",
    "vlucas/phpdotenv": "^5.5",
    "oscarotero/env": "^2.1",
    "roots/bedrock-autoloader": "^1.0",
    "roots/bedrock-disallow-indexing": "^2.0",
    "roots/wordpress": "6.6.2",
    "roots/wp-config": "1.0.0",
    "roots/wp-password-bcrypt": "1.1.0",
    "wpackagist-theme/twentytwentyfour": "^1.0",
    "wpengine/advanced-custom-fields-pro": "^6.3",
    "wpengine/wp-graphql-content-blocks": "dev-main",
    "wp-graphql/wp-graphql": "^1.29",
    "wp-graphql/wpgraphql-acf": "^2.4",
    "wp-graphql/wp-graphql-smart-cache": "^1.3.3",
    }

@justlevine
Copy link
Collaborator Author

Yup as expected, you're installing the source (from wp-graphql/wp-graphql instead of using WPackagist, so Composer is trying to rebuild the dependencies instead of using those shipped in the plugins.

(Iirc smart-cache is on .org too so you might want to use wpackagist-plugin/wp-graphql-smart-cache too)

@justlevine justlevine deleted the fix/appsero-client branch December 9, 2024 15:42
@Messa1
Copy link

Messa1 commented Dec 9, 2024

@justlevine

  Problem 1
    - Root composer.json requires wpackagist-plugin/wp-graphql-smart-cache, it could not be found in any version, there may be a typo in the package name.

@jasonbahl
Copy link
Collaborator

@Messa1 it's wpgraphql-smart-cache

see: https://wpackagist.org/search?q=wpgraphql-smart-cache&type=any&search=

At some point WordPress.org stopped allowing plugins that started with wp- so we had to go with wpgraphql without the first - when we released WPGraphQL Smart Cache.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs: reviewer response This needs the attention of a codeowner or maintainer status: in review Awaiting review before merging or closing type: bug Issue that causes incorrect or unexpected behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

WordPress 6.7 changes in translation throw a warning when WP_DEBUG is on

5 participants