Skip to content

Conversation

@adammkelly
Copy link
Contributor

Basic reboot for the JetKVM device within the settings page.

No handling of after the reboot happens from the UI due to it being a disruptive path.
This could be a future improvement were it polls for the device to be responsive again if known to be on the same IP address.

image
image

@CLAassistant
Copy link

CLAassistant commented May 22, 2025

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@IDisposable IDisposable left a comment

Choose a reason for hiding this comment

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

Looks great

<div className="mt-2 flex items-center justify-between gap-x-2">
<SettingsItem
title="Reboot Device"
description="Power cycle the JetKVM"
Copy link
Contributor

Choose a reason for hiding this comment

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

It's not actually power-cycling, though - it's only rebooting the OS - not the same.

@adokitkat
Copy link

+1

@ym ym merged commit 488276f into jetkvm:dev Jul 9, 2025
2 checks passed
@Rhys1210
Copy link

How do we get this function as from the screen shot above mine doesn't have the reboot option.

Screenshot 2025-07-14 223754

@dseven
Copy link
Contributor

dseven commented Jul 15, 2025

How do we get this function as from the screen shot above mine doesn't have the reboot option.

Build from source, or wait for a future release that includes this feature.

@adamshiervani adamshiervani moved this to Done in JetKVM Aug 5, 2025
@adamshiervani adamshiervani added this to the 0.5.0 milestone Aug 5, 2025
@adamshiervani adamshiervani removed this from JetKVM Aug 5, 2025
@adamshiervani adamshiervani removed this from the 0.5.0 milestone Aug 5, 2025
ym pushed a commit to ym/jetkvm-kvm that referenced this pull request Sep 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants