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

Always spinning progress in GraphiQL IDE #2881

Closed
3 tasks done
odrowonz opened this issue Aug 4, 2023 · 16 comments
Closed
3 tasks done

Always spinning progress in GraphiQL IDE #2881

odrowonz opened this issue Aug 4, 2023 · 16 comments
Labels
Close Candidate Needs confirmation before closing Duplicate This issue or pull request already exists

Comments

@odrowonz
Copy link

odrowonz commented Aug 4, 2023

Description

I am using WordPress 6.2.2 and PHP FastCGI (Apache) 8.2.0 (alt). After installing release v1.14.9 and activating the plugin, I go to the GraphiQL IDE and see an endless process of progress rotation. The situation is similar when the plugin version is downgraded to 1.6.12 (I went through all the versions). Version 1.6.12 successfully opened the GraphiQL IDE for me. What could be wrong?
ErrorOfGraphQL

Steps to reproduce

  1. Go to Plugins and install and activate release v1.14.9.
  2. Go to GraphiQL IDE.
  3. See the always spinning progress.
    ErrorOfGraphQL

Additional context

No response

WPGraphQL Version

1.14.9

WordPress Version

6.2.2

PHP Version

FastCGI (Apache) 8.2.0 (alt)

Additional enviornment details

No response

Please confirm that you have searched existing issues in the repo.

  • Yes

Please confirm that you have disabled ALL plugins except for WPGraphQL.

  • Yes
  • My issue is with compatibility with a specific WordPress plugin, and I have listed all my installed plugins (and version info) above.
@justlevine
Copy link
Collaborator

Hey @odrowonz

Please follow the Debugging steps. After you follow those steps, if you take a look at the Network Response, you should see a more explicit error. Once you share that we'll have a better chance of being able to help.

(Note: this plugin isn't actively tested against PHP 8.2 yet, so that could be a contributing factor).

@jasonbahl jasonbahl added the Close Candidate Needs confirmation before closing label Aug 9, 2023
@roenfeldt
Copy link

roenfeldt commented Aug 9, 2023

I second @odrowonz 's problem, the same is happening to my WordPress 6.3 installation + WPGraphQL 1.14.10.

The issue doesn't seem to be related to PHP 8.2. I tried changing the PHP version in my hosting cPanel to 8.1, 8.0, and even 7.4, but that doesn't make any difference.

The four-dots spinner won't go away, and the browser's Inspection Tools are showing a JavaScript-related error. Hopefully the attached image below will help you @jasonbahl and/or @justlevine figure out the issue.

Thank you!
wpgraphql

@justlevine
Copy link
Collaborator

Hey @roenfeldt 👋

It's likely not the same bug (the IDE not loading is just a sign that there is a bug/conflict), but since @odrowonz doesn't seem to be using this ticket, I'm happy to help you debug this here in the off chance it does help the OP 😎

There's one thing missing from your screenshot that I'd need to begin: the Network Response (in your screenshot, it's on the top right of the console, the tab next to the Headers. Please make sure to turn on GraphQL debugging first 🙏).
That will show us what the server is trying to send to the GraphiQL IDE instead of the expected introspection.

@jasonbahl
Copy link
Collaborator

@odrowonz I attempted to reproduce on PHP 8.2 (we don't currently have automated tests for 8.2, so I thought maybe there's a chance that was the culprit) but I'm able to load GraphiQL IDE just fine on PHP 8.2

Here's the server details of the local server I set up:

CleanShot 2023-08-09 at 11 43 38

And GraphiQL IDE loads fine:

CleanShot 2023-08-09 at 11 44 27

There's a good chance there could be an issue with a plugin or theme you have active, so be sure to test in a clean install vs an install with many other existing plugins.

I do see that you have at least Jetpack and SpeedyCache active in your screenshot. It's important for us to know if we're dealing with an issue with WPGraphQL itself, a plugin conflict with a specific plugin / combination of plugins, or something else.

I also notice you might have WordPress set to use a different language, so there's a chance that's impacting this 🤔

@jasonbahl
Copy link
Collaborator

@odrowonz it looks like you're running WordPress in non-English, and if you have a custom theme active that has page templates that use non-ascii characters in the template name, there's a good chance that you're running into this issue: #2807

@jasonbahl
Copy link
Collaborator

@roenfeldt are you using non-ascii characters in page templates? Curious if you might also be running into #2807

@odrowonz
Copy link
Author

Yes, I use default theme "Twenty Twenty-Three" and pages with Russian encoding.

@jasonbahl
Copy link
Collaborator

@odrowonz what about page templates? Like page-*.php files with Russian encoding?

If it's just page titles, that won't impact GraphiQL but page templates will.

@odrowonz
Copy link
Author

I did an experiment. I installed standard Wordpress, and chose the English scheme, which is "Twenty Twenty-Three" by default. Then I switch the language to Russian in settings -> general. Then GraphiQL turns the hourglass - is not working. I return the language back to English. Then GraphiQL starts working . And of course all *.php files have English encoding.

@jasonbahl
Copy link
Collaborator

@odrowonz ok nice. This is good information. Will try to reproduce!

@justlevine
Copy link
Collaborator

justlevine commented Aug 12, 2023

@jasonbahl @odrowonz it sounds like #2492 - changing the locale translates the WP template name (#2807 is just a manual subset of the same issue).

@odrowonz
Copy link
Author

Colleagues, is there any deadline for fixing this bug?

@justlevine
Copy link
Collaborator

@odrowonz assuming your issue is the same as the ones I linked previously (and if so, please let us know so we can consolidate the tickets 🙏), then you can try #2808 and see if it resolves your issue.

That particular PR needs feedback and unit testing before it can be merged, and if it doesn't solve things for you then we likely need a separate fix. Either way, contributions are always welcome!

@odrowonz
Copy link
Author

@odrowonz assuming your issue is the same as the ones I linked previously (and if so, please let us know so we can consolidate the tickets 🙏), then you can try #2808 and see if it resolves your issue.

That particular PR needs feedback and unit testing before it can be merged, and if it doesn't solve things for you then we likely need a separate fix. Either way, contributions are always welcome!

Is there any release that includes this edit? (I tried 1.14.10 and it doesn't work there yet)

@odrowonz
Copy link
Author

Can you compile pre-release for test?

@justlevine
Copy link
Collaborator

@odrowonz

assuming your issue is the same as the #2881 (comment) (and if so, please let us know so we can consolidate the tickets 🙏),

You seemed to miss ☝️ part of my last reply, but from your responses I'm going to assume that you read that issue and it is a duplicate of this one. As such, I'm closing this in favor of #2492 (if I'm wrong, let us know and we'll reopen).

Is there any release that includes this edit? (I tried 1.14.10 and it doesn't work there yet)

Like I said, that particular PR needs feedback and unit testing before it can be merged.

Can you compile pre-release for test?

To "compile" that or any branch, you can pull / download it locally, and then run composer install --no-dev --optimize-autoloader from inside the plugin folder. (or just composer install if you want to include the dev-deps needed to contribute).

@justlevine justlevine added the Duplicate This issue or pull request already exists label Aug 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Close Candidate Needs confirmation before closing Duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

4 participants