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

[Task #4] Verification of ownership #3

Closed
puzzledsean opened this issue Apr 24, 2018 · 2 comments
Closed

[Task #4] Verification of ownership #3

puzzledsean opened this issue Apr 24, 2018 · 2 comments
Assignees

Comments

@puzzledsean
Copy link
Collaborator

puzzledsean commented Apr 24, 2018

  • Write a verify function that takes in:
    • Item ID
    • User ID
  • Outputs a True/False of whether the owner was the last user ID in a transaction with that Item ID

Can probably make a call to the history function in issue #4 and output True/False based on the output

@puzzledsean puzzledsean changed the title Verification of ownership [Task #4] Verification of ownership Apr 24, 2018
@KenG98
Copy link
Collaborator

KenG98 commented Apr 26, 2018

For whoever of us takes care of this: there's an "IsBlockValid" function in blockmanager. It takes the new block as well as the last verified block. If you can, please modify that function to take the new block as well as the ENTIRE blockchain (not just last block), and not just return a boolean for whether or not it's valid, but also return a string with an error code (invalid hash, invalid ownership, invalid item ID, etc etc) - this will make for a dope demo.

@KenG98 KenG98 self-assigned this Apr 26, 2018
@KenG98
Copy link
Collaborator

KenG98 commented Apr 27, 2018

This is like... semi done. It doesn't catch nearly half of cases, but it works on certain cases. There's enough to demo.

@KenG98 KenG98 closed this as completed Apr 27, 2018
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

No branches or pull requests

2 participants