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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Server Related to the server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants