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

Delete oauth consumer entry on integration:delete (#1287) #1290

Merged

Conversation

cmuench
Copy link
Member

@cmuench cmuench commented Sep 12, 2023

Magerun pull-request check-list:

  • Pull request against develop branch (if not, just close and create a new one against it)

Fixes #1287
Changes proposed in this pull request:

  • Inject Magento\Integration\Api\OauthServiceInterface in command class
  • Call Magento\Integration\Api\OauthServiceInterface::deleteConsumer

@cmuench cmuench force-pushed the bugfix/1287-integration-delete-command branch from 5436458 to 69fd9b2 Compare September 12, 2023 07:24
@github-actions
Copy link

github-actions bot commented Sep 12, 2023

Download the artifacts for this pull request:

@cmuench cmuench force-pushed the bugfix/1287-integration-delete-command branch from 69fd9b2 to 63a5669 Compare September 12, 2023 07:26
@cmuench cmuench force-pushed the bugfix/1287-integration-delete-command branch from 63a5669 to 867fa63 Compare September 12, 2023 07:32
@cmuench cmuench merged commit 7637c2f into netz98:develop Sep 12, 2023
@cmuench cmuench added this to the 7.2.0 milestone Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

integration:delete command doesn't delete associated oauth consumer and oauth token
1 participant