Skip to content

Conversation

@sctanf
Copy link
Member

@sctanf sctanf commented Sep 23, 2025

@github-actions github-actions bot added the Area: Server Related to the server label Sep 23, 2025
@Eirenliel
Copy link
Member

Using tracker's HID directly should be an optional feature. Normally, the job of the server is to pair tracker to the dongle when it's connected, and nothing else. If it will also process data, it will confuse the user since the tracker will be automatically connected, but might not be paired to the dongle...

It needs to be behind a developer option.

@sctanf
Copy link
Member Author

sctanf commented Sep 23, 2025

I think the toggle will likely need to be on the tracker itself (and disabled by default) since having this feature enabled will prevent it from entering pairing automatically. I think it is okay to always detect on the server, as there is currently no way for the server to tell the tracker it is receiving data through the USB connection.

@Eirenliel
Copy link
Member

I think a global setting is better, you can make server accept or not accept connections from this device id if it's enable or disabled. Either way, it should be implemented before it's merged, in whichever way you prefer. Otherwise it breaks intended workflow.

@sctanf
Copy link
Member Author

sctanf commented Sep 23, 2025

Okay, I will still probably need to give trackers a toggle in the meantime

@sctanf sctanf marked this pull request as draft September 23, 2025 18:55
@loucass003
Copy link
Member

Whats the status on this @sctanf? why is it still a draft

@sctanf
Copy link
Member Author

sctanf commented Dec 2, 2025

while it works, it was going to be changed to be on a toggle instead, ie. to not potentially cause confusion on why a tracker may only work while plugged into the computer

1 similar comment
@sctanf
Copy link
Member Author

sctanf commented Dec 2, 2025

while it works, it was going to be changed to be on a toggle instead, ie. to not potentially cause confusion on why a tracker may only work while plugged into the computer

@sctanf
Copy link
Member Author

sctanf commented Dec 14, 2025

@github-actions github-actions bot added Area: Application Protocol Related to communication with apps like the GUI, overlay, games Area: GUI Related to the GUI Area: Translation Improvements or additions to translations labels Dec 14, 2025
@sctanf sctanf marked this pull request as ready for review December 14, 2025 22:09
@sctanf sctanf requested a review from loucass003 as a code owner December 14, 2025 22:10
@sctanf sctanf requested a review from ImSapphire as a code owner December 26, 2025 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Application Protocol Related to communication with apps like the GUI, overlay, games Area: GUI Related to the GUI Area: Server Related to the server Area: Translation Improvements or additions to translations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants