Skip to content

Commit 70dc88b

Browse files
authored
Merge pull request #2286 from jasonbahl/feature/remove-old-graphiql
Remove old GraphiQL JavaScript
2 parents a8b7361 + 6490368 commit 70dc88b

27 files changed

Lines changed: 3 additions & 12539 deletions

codeception.dist.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,7 @@ coverage:
2020
exclude:
2121
- wp-graphql.php
2222
- vendor/*
23-
- src/Admin/GraphiQL/app/*
24-
- src/Admin/GraphiQL/js/*
23+
- packages/*
2524
show_only_summary: false
2625
extensions:
2726
enabled:

composer.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,6 @@
9696
"*.yml",
9797
"!vendor/",
9898
"plugin-build/",
99-
"!src/Admin/GraphiQL/app/",
10099
"node_modules/",
101100
"!.wordpress-org/",
102101
"docs/",

docs/authentication.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ For remote HTTP requests to the `/graphql` endpoint, existing authentication plu
2727
- https://github.com/WP-API/OAuth1 (labeled for use with the WP REST API, but works well with WPGraphQL)
2828
- https://make.wordpress.org/core/2020/11/05/application-passwords-integration-guide/ (For WordPress 5.6 and above)
2929

30-
If the remote request is within the WordPress admin, such as the WPGraphiQL plugin, you can use the existing Auth nonce as seen in action [here](https://github.com/wp-graphql/wp-graphiql/blob/master/assets/app/src/App.js#L16-L29).
30+
If the remote request is within the WordPress admin, such as the WPGraphiQL plugin, you can use the existing Auth nonce as seen in action [here](https://github.com/wp-graphql/wp-graphiql/blob/master/packages/graphiql-auth-switch).
3131

3232
For non-remote requests (PHP function calls), if the context of the request is already authenticated, such as an Admin page in the WordPress dashboard, existing WordPress authentication can be used, taking advantage of the existing session. For example, if you wanted to use a GraphQL query to populate a dashboard page, you could send your query to the `do_graphql_request( $query )` function, and since the request is already authenticated, GraphQL will execute with the current user set, and will resolve fields that the users has permission to resolve.
3333

docs/security.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ For remote HTTP requests to the `/graphql` endpoint, existing authentication plu
5959
- https://github.com/WP-API/Basic-Auth (even though it’s labeled for the REST API, it works well with WPGraphQL – but not recommended for non-SSL connections)
6060
- https://github.com/WP-API/OAuth1 (labeled for use with the WP REST API, but works well with WPGraphQL)
6161

62-
If the remote request is within the WordPress admin, such as the WPGraphiQL plugin, you can use the existing Auth nonce as seen in action [here](https://github.com/wp-graphql/wp-graphiql/blob/master/assets/app/src/App.js#L16-L29).
62+
If the remote request is within the WordPress admin, such as the WPGraphiQL plugin, you can use the existing Auth nonce as seen in action [here](https://github.com/wp-graphql/wp-graphiql/blob/master/packages/graphiql-auth-switch).
6363

6464
For non-remote requests (PHP function calls), if the context of the request is already authenticated, such as an Admin page in the WordPress dashboard, existing WordPress authentication can be used, taking advantage of the existing session. For example, if you wanted to use a GraphQL query to populate a dashboard page, you could send your query to the `do_graphql_request( $query )` function, and since the request is already authenticated, GraphQL will execute with the current user set, and will resolve fields that the users has permission to resolve.
6565

src/Admin/GraphiQL/app/.gitignore

Lines changed: 0 additions & 19 deletions
This file was deleted.

src/Admin/GraphiQL/app/README.md

Lines changed: 0 additions & 22 deletions
This file was deleted.

src/Admin/GraphiQL/app/build/asset-manifest.json

Lines changed: 0 additions & 23 deletions
This file was deleted.

src/Admin/GraphiQL/app/build/index.html

Lines changed: 0 additions & 1 deletion
This file was deleted.

src/Admin/GraphiQL/app/build/static/css/2.8d9709c7.chunk.css

Lines changed: 0 additions & 2 deletions
This file was deleted.

src/Admin/GraphiQL/app/build/static/css/2.8d9709c7.chunk.css.map

Lines changed: 0 additions & 1 deletion
This file was deleted.

0 commit comments

Comments
 (0)