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

Feature Enhancement: Allow users to manually restart, or manually start, bundled go daemon within the app (not only via tray icon) #2293

Open
SgtPooki opened this issue Oct 10, 2022 · 3 comments
Labels
effort/days Estimated to take multiple days, but less than a week exp/beginner Can be confidently tackled by newcomers good first issue Good issue for new contributors help wanted Seeking public contribution on this issue P3 Low: Not priority right now status/deferred Conscious decision to pause or backlog

Comments

@SgtPooki
Copy link
Member

Is your feature request related to a problem? Please describe.
go daemon startup is tightly coupled to app startup. If you start IPFS desktop and the node fails to start due to repo lock or other issue, users have no ability to get things working without a full app restart.

We're all familiar with this screen which is indicative that a full app restart, or separate daemon start (via CLI) is required:
image

Describe the solution you'd like
ipfs go daemon status should be toggle-able, restartable, stoppable, etc, from the ipfs desktop application

Describe alternatives you've considered
N/A

Additional context
Related: #1977

@SgtPooki SgtPooki added the need/triage Needs initial labeling and prioritization label Oct 10, 2022
@SgtPooki
Copy link
Member Author

@juliaxbow restart is possible from tray menu, we should probably make this more accessible, if restarts are viable solution for daemon startup failure.

@SgtPooki SgtPooki added exp/beginner Can be confidently tackled by newcomers effort/days Estimated to take multiple days, but less than a week status/deferred Conscious decision to pause or backlog and removed need/triage Needs initial labeling and prioritization labels Oct 10, 2022
@SgtPooki SgtPooki changed the title Feature Enhancement: Allow users to manually restart, or manually start, bundled go daemon Feature Enhancement: Allow users to manually restart, or manually start, bundled go daemon within the app (not only via tray icon) Oct 10, 2022
@juliaxbow
Copy link

Did you want this to be available from just the "could not connect" screen? OR should users be able to access wherever they are in the app?

Assuming the latter based on your suggestion of "toggle-able" here are some options to include in the menu for initial feedback:

Ellipses.Dropdown.mov

The toggles would require addt'l help text to explain what's happening.

IPFS.Icon.Toggle.mov
Standard.On.Off.Toggle.mov

@SgtPooki SgtPooki added help wanted Seeking public contribution on this issue P3 Low: Not priority right now good first issue Good issue for new contributors labels Oct 31, 2023
@SgtPooki
Copy link
Member Author

I feel like we should use the icon that's on the far right,
image

to toggle it instead of simply redirecting to the status page and opening the advanced view... open to other thoughts here though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
effort/days Estimated to take multiple days, but less than a week exp/beginner Can be confidently tackled by newcomers good first issue Good issue for new contributors help wanted Seeking public contribution on this issue P3 Low: Not priority right now status/deferred Conscious decision to pause or backlog
Projects
No open projects
Status: UX Design Needed
Development

No branches or pull requests

2 participants