[x] Make sure you are making a pull request against the develop branch (left side). Also you should start your branch off our develop.
[x] Make sure you are requesting to pull request from a topic/feature/bugfix/devops branch (right side). Don't pull request from your master!
[x] Have you ensured/updated that CLI tests to extend coverage to any new logic. Learn how to modify the tests here.
What does this implement/fix? Explain your changes.
Changes hook used for WPGraphQL\WooCommerce\Utils\Session_Transaction_Manager::pop_transaction_id to graphql_mutation_response
woographql_session_transaction_complete hook added to the end of WPGraphQL\WooCommerce\Utils\Session_Transaction_Manager::pop_transaction_id
Changes hook used for WPGraphQL\WooCommerce\Utils\QL_Session_Handler::save_if_dirty() to woographql_session_transaction_complete
With these changes the proper session token should be generated post login mutations from WPGraphQL JWT Authentication and WPGraphQL Headless Login, resolving the following issues.
Resolves #684
Resolves #682
Resolves #817
Does this close any currently open issues?
…
Any relevant logs, error output, GraphiQL screenshots, etc?
Your checklist for this pull request
Thanks for sending a pull request! Please make sure you click the link above to view the contribution guidelines, then fill out the blanks below.
🚨Please review the guidelines for contributing to this repository.
What does this implement/fix? Explain your changes.
With these changes the proper session token should be generated post
loginmutations from WPGraphQL JWT Authentication and WPGraphQL Headless Login, resolving the following issues.Resolves #684 Resolves #682 Resolves #817
Does this close any currently open issues?
…
Any relevant logs, error output, GraphiQL screenshots, etc?
(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)
Any other comments?
…
Where has this been tested?