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

ups-package-tracker changes #39

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

Conversation

Frippo40
Copy link

@Frippo40 Frippo40 commented Nov 6, 2019

Here are some changes requested by @Amxx in pull request #37

Sorry, I'm not familiar enough with git to fix all the file commit issues after my bad commits within that same pull request so I had to create a new branch.

Does this look better? The oracle is now returning a uint that corresponds to the DeliveryStatus enum

1 = In Transit, 2 = Delivered, 3 = Exception, 4 = Pickup, 5 = Manifest Pickup

@Frippo40 Frippo40 changed the title DeliveryStatus enum, less files committed ups-package-tracker changes Nov 6, 2019
@Frippo40 Frippo40 mentioned this pull request Nov 6, 2019
@sulliwane
Copy link
Collaborator

Hey @Frippo40 , thanks for your contribution. Could you go on the Gitcoin bounty page, and click the button "submit work"? Otherwise, if you get selected, we won't be able to pay you. thanks

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.

2 participants