la5nta / pat

A cross-platform Winlink client written in Go
https://getpat.io
MIT License
488 stars 86 forks source link

Post version info to Winlink CMS Web Services #8

Closed martinhpedersen closed 8 years ago

martinhpedersen commented 8 years ago

RMS Express and all other mainstream clients reports the user callsign, appname and version frequently to the winlink web API.

The feature/version_report branch implements this feature for Pat, but I guess there may be some privacy concerns about enabling this by default.

PROS:

CONS:

The feature will be optional, but enabled by default.

Feedback please :)

martinhpedersen commented 8 years ago

To clarify: I'm seeking feedback on whether or not to include this feature.

dranch commented 8 years ago

New user as of today but it seems the information being uploaded is pretty minor. I would also argue that getting this application more visibility through the Winlink system is key for it's adoption. I'm here as I'm interested in the ARDOP/Winmor connector support which was only previously available in BPQ32

martinhpedersen commented 8 years ago

dranch, thanks for joining the discussion!

I believe you make a valid point regarding getting some visibility through the Winlink system. The potential gain of uploading the data is very appealing from a developers point of view. Given the importance of user growth in this stage of the project, i believe users will benefit from this too.

I'm leaning towards merging the branch, and making sure that the future configuration UI (issue #5) prompts the user and provides info on what's being uploaded.


I see you have joined the pat-users list too, welcome! I haven't gotten around to writing a wiki page for ardop and winmor setup yet. So please don't hesitate to post questions if you're having questions.