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

Add dimensions and weight to product samples #3291

Merged

Conversation

BravoSimone
Copy link
Contributor

@BravoSimone BravoSimone commented Aug 2, 2019

Description

I'm adding weight, height, width, and depth to all products added by product samples file. I decided to add this data when I run in a simple issue adding Easypost to my project. In development didn't work properly because products didn't have a proper weight set. I think this should be set directly into the solidus gem since we already have samples here and would fix all these kind of issues.

Checklist:

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have updated Guides and README accordingly to this change (if needed)
  • I have added tests to cover this change (if needed)

I'm adding weight, height, width and depth to all products added by
product samples file.
Copy link
Member

@spaghetticode spaghetticode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@BravoSimone thank you 👍

Copy link
Contributor

@jacobherrington jacobherrington left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am fine with this, thanks for helping out @BravoSimone! 🥇

@jacobherrington jacobherrington merged commit e83680d into solidusio:master Aug 4, 2019
@BravoSimone BravoSimone deleted the add-weight-and-size-to-samples branch August 5, 2019 08:24
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.

3 participants