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

[PW-5743] Disable all payment methods by default #1371

Merged
merged 12 commits into from Feb 25, 2022
Merged

Conversation

michaelpaul
Copy link
Contributor

@michaelpaul michaelpaul commented Feb 17, 2022

Description

Right after installing the plugin, the payment methods "Credit Card" and "OneClick" come enabled. They won't work without setting up all the required settings (e.g. API key).

Normally, this is not an issue for regular merchants. But it breaks Magento core MFTF tests, which then blocks us from publishing on their Marketplace.

This PR disables all methods by default and introduces a Github Actions workflow to check if the core compatibility is maintained.

Tested scenarios

  • Disabling these settings did not effect existing values
  • Run Core MFTF tests

Payment methods must be enabled selectively by merchants after setting up other required settings, like API key.
Morerice
Morerice previously approved these changes Feb 17, 2022
peterojo
peterojo previously approved these changes Feb 18, 2022
@michaelpaul michaelpaul dismissed stale reviews from peterojo and Morerice via 4fa7b48 February 21, 2022 18:21
acampos1916
acampos1916 previously approved these changes Feb 23, 2022
peterojo
peterojo previously approved these changes Feb 23, 2022
acampos1916
acampos1916 previously approved these changes Feb 23, 2022
acampos1916
acampos1916 previously approved these changes Feb 24, 2022
Morerice
Morerice previously approved these changes Feb 24, 2022
@michaelpaul michaelpaul changed the title [PW-5743] Disable all methods by default [PW-5743] Disable all payment methods by default Feb 24, 2022
@sonarcloud
Copy link

sonarcloud bot commented Feb 24, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants