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

Update wallet.js #41

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update wallet.js #41

wants to merge 3 commits into from

Conversation

wbsmolen
Copy link

This change removes the requirement for encrypted wallet files. (Optional) If a passphrase is provided in private_constants.js, it will be used.

This change removes the requirement for encrypted wallet files. (Optional) If a passphrase is provided in private_constants.js, it will be used.
@knkrth
Copy link
Contributor

knkrth commented May 28, 2018

It through error

/home/ubuntu/workspace/opentrade/server/modules/users/wallet.js:732
});
 ^

SyntaxError: Unexpected token )

By adding } on line wallet.js:732 fix the problem & I don't know weather it works without the walletpassphrase

@wbsmolen
Copy link
Author

@knkrth try now

@knkrth
Copy link
Contributor

knkrth commented May 28, 2018

@wbsmolen Server starts without any error. @3s3s Can you check it & merge it.

@3s3s
Copy link
Owner

3s3s commented May 29, 2018

I have made this with new commit. Please check.

@CryptorClub
Copy link
Contributor

This branch may be closed...

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.

4 participants