-
Notifications
You must be signed in to change notification settings - Fork 108
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Minor improvements #831
Minor improvements #831
Conversation
Go version update
|
||
// WithSchemeID | ||
// WithSchemeID allows user to set a scheme using its ID. The scheme | ||
// is used to customize some behaviors inside the client | ||
func WithSchemeID(schID string) Option { |
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.
Wait now I realize that we miss something here: there should be a way for the client to automatically fetch the ID from the endpoint so we dont need to give it as option. That's how it works currently for the public key / genesis hash. Can you add that in a subsequent PR as a TODO item ?
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 will add a TODO for this. Could you explain me which are the options a client has to work? Using chainInfo file is the first one. Which are the other ones?
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'm happy to see this merged as is, and figure out how to infer scheme from chain info in a subsequent PR.
@willscott Let's merge it then! |
No description provided.