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

feat: run garbage collector #1407

Merged
merged 2 commits into from
Apr 17, 2020
Merged

feat: run garbage collector #1407

merged 2 commits into from
Apr 17, 2020

Conversation

hacdias
Copy link
Member

@hacdias hacdias commented Apr 17, 2020

Supports running the GC from the menubar. This was a feature that was lost during the revamp. Closes #1355.

Screenshot 2020-04-17 at 10 25 17

Screenshot 2020-04-17 at 10 25 22

Screenshot 2020-04-17 at 10 25 34

Note: the Electron icon is replaced by IPFS Desktop's icon on the final version.

License: MIT
Signed-off-by: Henrique Dias hacdias@gmail.com

@hacdias hacdias added the kind/enhancement A net-new feature or improvement to an existing feature label Apr 17, 2020
@hacdias hacdias marked this pull request as ready for review April 17, 2020 10:10
Copy link
Contributor

@jessicaschilling jessicaschilling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few nitpicks in text 😊 and two questions:

  • Will it ever NOT run successfully? If so, how does the user know? If not, maybe the languages should just be “garbage collection run finished” or similar.
  • I assume it runs in the background so user can still work during “it may take a while”?
    Thanks!

assets/locales/en.json Outdated Show resolved Hide resolved
assets/locales/en.json Outdated Show resolved Hide resolved
assets/locales/en.json Outdated Show resolved Hide resolved
@hacdias
Copy link
Member Author

hacdias commented Apr 17, 2020

Thanks for the copy feedback @jessicaschilling!

Will it ever NOT run successfully? If so, how does the user know? If not, maybe the languages should just be “garbage collection run finished” or similar.

I believe it's unlikely, but it may happen. And yes, the user will end up knowing. We'll tell them that an error occurred and ask them if they'd like to open an issue here with the error report (which is generated automatically!)

I assume it runs in the background so user can still work during “it may take a while”?

Yes!

@jessicaschilling
Copy link
Contributor

Awesome, thanks for clarifying!

@hacdias hacdias changed the title feat: run gc feat: run garbage collector Apr 17, 2020
@hacdias hacdias mentioned this pull request Apr 17, 2020
22 tasks
License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>
License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>
@hacdias hacdias merged commit 6ca8c8c into master Apr 17, 2020
@hacdias hacdias deleted the feat/run-gc branch April 17, 2020 19:44
src/run-gc.js Show resolved Hide resolved
@RubenKelevra
Copy link
Contributor

Is there a progress bar or a wait icon of some kind? Since the operation can take extremely long if the datastore is large or the disk is slow.

@hacdias
Copy link
Member Author

hacdias commented Apr 18, 2020

No, but I was thinking of changing the tooltip while it's running so the user knew:

image

What do you think?

@hacdias
Copy link
Member Author

hacdias commented Apr 18, 2020

Uh, I have an idea with better UX. Will get back with a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A net-new feature or improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add manual garbage collector button
4 participants