-
Notifications
You must be signed in to change notification settings - Fork 153
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
Allow snapshot to be loaded from an http request and add progress #762
Labels
Comments
austinabell
added
Status: On Ice
Priority: 4 - Low
Limited impact and can be implemented at any time
Type: Enhancement
labels
Oct 19, 2020
I can look into it this week :) |
Just for more guidance, I made this PR for importing snapshots #789 |
snaumov
added a commit
to snaumov/forest
that referenced
this issue
Nov 5, 2020
snaumov
added a commit
to snaumov/forest
that referenced
this issue
Nov 5, 2020
snaumov
added a commit
to snaumov/forest
that referenced
this issue
Nov 5, 2020
snaumov
added a commit
to snaumov/forest
that referenced
this issue
Nov 5, 2020
snaumov
added a commit
to snaumov/forest
that referenced
this issue
Nov 5, 2020
snaumov
added a commit
to snaumov/forest
that referenced
this issue
Nov 5, 2020
snaumov
added a commit
to snaumov/forest
that referenced
this issue
Nov 13, 2020
snaumov
added a commit
to snaumov/forest
that referenced
this issue
Nov 18, 2020
snaumov
added a commit
to snaumov/forest
that referenced
this issue
Nov 19, 2020
austinabell
pushed a commit
that referenced
this issue
Nov 19, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Issue summary
Once #761 is in, would be very nice to allow for loading a snapshot from a url, as Lotus does it.
They also added in a progress bar for the snapshot import, and I think this would be a very nice UX to add. For this progress to show up well, it has to be before any of the async tasks are started, as the logger conflicts with the progress bar output (see
fetch-params
progress bar when requests are done async), but should be pretty straightforwardThis issue is definitely low priority since it wouldn't add any new functionality.
Other information and links
The text was updated successfully, but these errors were encountered: