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

Shipping improvements #305

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

Shipping improvements #305

wants to merge 1 commit into from

Conversation

monomon
Copy link

@monomon monomon commented Jan 12, 2015

Hi, I have been working on a shipping module and came across some shortcomings that I tried to address.
Let me know if you are interested in merging this.

Here is a general list of the fixes:

  • Use selected shipping method instead of the first one available
  • Add an info object on which shipping module can set extra data about
    the order
  • Add a selected_module property to shipping class
  • Add an after_process method to shipping - shipping modules may need to
    do additional processing

* Add an info object on which shipping module can set extra data about
  the order
* Add a selected_module property to shipping class
* Add an after_process method to shipping - shipping modules may need to
  do additional processing
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.

None yet

1 participant