-
Notifications
You must be signed in to change notification settings - Fork 67
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
gui: display error for rescan past pruned height #941
gui: display error for rescan past pruned height #941
Conversation
8c4a066
to
6288441
Compare
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've added a couple of comments based on initial testing. Will continue testing.
f66f846
to
a41d392
Compare
Thanks @jp1ac4 I fixed the comments and rebased |
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've added a couple more comments regarding the negative timestamp value. Otherwise, my tests have been going well.
952c06a
to
bed1220
Compare
bed1220
to
69fd189
Compare
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.
Self-ACK 69fd189
Thanks jp1ac4 for the good reviews
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.
ACK 69fd189.
close #900
After discussion with @kloaec it is better to not return error if date is prior to genesis block date, just take genesis block timestamp