Skip to content

Conversation

@WendyH1108
Copy link

No description provided.

const wallet=new Wallet(req.body.senderPrivateKey);
const trans=new Transaction();
trans.newTransaction(wallet, req.body.recipientPublicKey, req.body.amount);
const bc=Blockchain(Block.genesis());
Copy link
Contributor

Choose a reason for hiding this comment

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

first it should be new Blockchain(...) and also it should not be initalized to genesis block. It should query the MySQL database and initalize the Blockchain object with the data fetched from the db

@xavierlyu
Copy link
Contributor

xavierlyu commented Oct 28, 2020

First of all, the PR and the commits should be named properly and more descriptive, such as "added ... feature" or "updated ... to ....".

Also, make sure you pull from master first before making any changes. It looks like the conn mysql object in api.js that fetches data is gone

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