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

Set default active configuration to 0 #56

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lytesaber
Copy link

On module install the Zip method is set as active "1" by default. All modules should be disabled by default and enabled by the user within the Magento admin system configuration. There shouldn't be an assumption that even though the module is installed and enabled that the payment method has been configured with API keys and ready to be used as an active payment method in the checkout. Currently if the Zip module is installed on a production site, the Zip method will appear in the checkout due to this default config even without API keys configured.

On module install the Zip method is set as active "1" by default. All modules should be disabled by default and enabled by the user within the Magento admin system configuration. There shouldn't be an assumption that even though the module is installed and enabled that the payment method has been configured with API keys and ready to be used as an active payment method in the checkout. Currently if the Zip module is installed on a production site, the Zip method will appear in the checkout due to this default config even without API keys configured.
@mahbub-zip
Copy link
Contributor

Thank you @lytesaber . we are taking is chance to repo code.

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.

2 participants