Skip to content

Conversation

@bummytime
Copy link
Contributor

This PR is a dupe of #577 — it re-adds changes from that.

Testing

  • Build and run all the unit tests — verify everything is ✅

Here are some suggested testing steps, but feel free to test this however you want.

  1. Build and run the app

  2. Open the orders tab

  3. Find an order with Processing status where the line items have a quantity of 1

  4. Open the fullfillment screen

    • Verify the item price and quantity details looks good
  5. Find an order with Completed status where the line items have a quantity of 1

  6. Open the item details screen

    • Verify the item price and quantity details looks good
  7. Find an order with Processing status where the line items have a quantity of 2 or more

  8. Open the fullfillment screen

    • Verify the item price and quantity details looks good
  9. Find an order with Completed status where the line items have a quantity of 2 or more

  10. Open the item details screen

    • Verify the item price and quantity details looks good

@bummytime bummytime self-assigned this Jan 11, 2019
@astralbodies astralbodies added this to the App Store release ❄️ milestone Jan 11, 2019
Copy link
Contributor

@mindgraffiti mindgraffiti left a comment

Choose a reason for hiding this comment

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

:shipit: !

@bummytime bummytime changed the base branch from develop to release/1.0.0 January 11, 2019 17:13
@wpmobilebot
Copy link
Collaborator

2 Warnings
⚠️ PR is missing at least one label.
⚠️ Core Data: Do not edit an existing model in a release branch unless it hasn’t been released to testers yet. Instead create a new model version and merge back to develop soon.

Generated by 🚫 Danger

@bummytime bummytime merged commit cc86183 into release/1.0.0 Jan 11, 2019
@bummytime bummytime deleted the fix/358-item-price branch January 11, 2019 17:39
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.

6 participants