Skip to content

Conversation

@jasonbahl
Copy link
Collaborator

Login mutation needs to be registered using register_graphql_mutation

  • also fixes some test issues

…_graphql_mutation`

- also fixes some test issues
@jasonbahl jasonbahl requested a review from kidunot89 October 16, 2019 20:07
@jasonbahl jasonbahl self-assigned this Oct 16, 2019
Copy link
Collaborator

@kidunot89 kidunot89 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add composer.lock to .gitignore. Delete codeception.yml, composer.lock and re-commit

codeception.yml Outdated
title: 'Test'
plugins: ['wp-graphql/wp-graphql.php', 'wp-graphql-jwt-authentication/wp-graphql-jwt-authentication.php']
activatePlugins: ['wp-graphql/wp-graphql.php', 'wp-graphql-jwt-authentication/wp-graphql-jwt-authentication.php', ]
configFile: 'tests/_data/config.php'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Delete this file.

/tests/*.suite.yml
/tests/*.suite.yml
.env
.codeception.yml
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add composer.lock

composer.lock Outdated
@@ -1,10 +1,10 @@
{
"_readme": [
"This file locks the dependencies of your project to a known state",
"Read more about it at https://getcomposer.org/doc/01-basic-usage.md#composer-lock-the-lock-file",
"Read more about it at https://getcomposer.org/doc/01-basic-usage.md#installing-dependencies",
"This file is @generated automatically"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Delete this file

@jasonbahl jasonbahl merged commit 91b76ec into wp-graphql:master Oct 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants