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

Completed(ish) PR #22

Open
wants to merge 89 commits into
base: rtown/master
Choose a base branch
from

Conversation

rileslovesyall
Copy link

Trello board: https://trello.com/b/3LFDwbpS/rtown-shipping-service-fun

We completed most of the base requirements, with some bugs. Have decent error handling, timeout handling, and rspec coverage. However, packages with same item multiple times from bEtsy aren't adding up properly, and we realized some of our error messages aren't transmitting exactly as expected. With more time, we would investigate what is happening with the error logic and make sure that every item being shipping is accounted for.

@sojeri sojeri self-assigned this Feb 3, 2016
@@ -54,9 +63,15 @@ GEM
execjs
coffee-script-source (1.10.0)
concurrent-ruby (1.0.0)
crack (0.4.3)
Copy link

Choose a reason for hiding this comment

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

I want to share this unnecessary supplemental reading I did after seeing this gem name: What is crack?

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