-
Notifications
You must be signed in to change notification settings - Fork 2
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
125 save user info #134
125 save user info #134
Conversation
app/user.go
Outdated
} | ||
} | ||
|
||
// Public returns the public key (address) of the given user |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Public key and address are not the same. Address is hash of key, is that right @chadlagore ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good call, I'll remove that "(address)" comment, it's not really necessary.
app/user.go
Outdated
// current working directory in JSON format. On success this returns | ||
// a pointer to a new user constructed from the information in the file. | ||
// If an error occurrs it is returned. | ||
func Load(fileName string) (*User, error) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd call this LoadUser
since it's in the app
namespace. app.Load
seems like it should load the app
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will do.
Related Issue
Closes #125
Closes #131
Description
Save
andLoad
functions to user and blockchain to save and load their contents from disk on startup.user
command to the console to view and edit user info.WIKI Updates
Todos
General: