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

Default Shipping Method #28

Closed
mageuser opened this issue Feb 7, 2018 · 4 comments
Closed

Default Shipping Method #28

mageuser opened this issue Feb 7, 2018 · 4 comments

Comments

@mageuser
Copy link

mageuser commented Feb 7, 2018

Hello, Daniel.
I want put my own module as a default shipping method, but it does not appear in the list to select.

My module is working on checkout page (its a bit bugged because my fault).

@mageuser
Copy link
Author

mageuser commented Feb 7, 2018

@danslo I know it's not part of your module, but could you take a look at what I'm doing wrong when calling api and displaying the results? Already tried help in other places but without success :(

https://magento.stackexchange.com/questions/212448/forced-multiple-select-shipping-method?noredirect=1#comment295745_212448

@danslo
Copy link
Owner

danslo commented Feb 7, 2018

That list is generated not by Clean Checkout, but by a core Magento source model. So that would be your first place to debug.

This magento2 issue indicates that it may be necessary to set a 'value' XML node for your shipping method to show up in this list.
edit: Nevermind, that was talking about payment methods.

In any case, if you don't want to debug Magento 2 internals, you can set clean_checkout/general/default_shipping_method in core_config_data manually. I'm not convinced this is an issue with Clean Checkout itself.

@danslo
Copy link
Owner

danslo commented Feb 7, 2018

As for why they all seems to be selected (rather than individually), it's probably because they all have the same method code.

Quite sure you're problems will go away once you start giving your methods a unique value in $method->setMethod(). Out of scope for this module though, so closing this issue.

@mageuser
Copy link
Author

mageuser commented Feb 7, 2018

Thanks, I could solve both problems!

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

No branches or pull requests

2 participants