ac2cz / FoxTelem

AMSAT Fox-1 Telemetry Decoder
GNU General Public License v3.0
51 stars 17 forks source link

Properties message inappropriate for upgrade #761

Open burnsfisher opened 1 year ago

burnsfisher commented 1 year ago

The message that one gets when one runs an upgraded FoxTelem version for the first time sounds much more serious than it is. I think the messge is something like "Can not read properties file". It later maybe implies that it is ok and can create a new one, but it still sounds like a worse problem than it is.

I wonder if there is some way to tell when the message should be just "Upgrading your properties file to work with a new version" vs when there is really a problem. This is sort of a problem upgrading the FIAB/CubeSatSim cards.

ac2cz commented 1 year ago

Ah yes, this is something I would like to fix. I agree the message is old and dated. I don't think we need to display it anymore. I think we can silently process the changes.

I just made the release for 1.12, so this sort of missed the cut. But could make a new version with this change, perhaps 1.12z3, for you to use on the Cubesatsim cards. What do you think?

burnsfisher commented 1 year ago

Yes, I think that would be great. Thanks!

On Wed, Oct 26, 2022 at 4:45 PM Chris Thompson @.***> wrote:

Ah yes, this is something I would like to fix. I agree the message is old and dated. I don't think we need to display it anymore. I think we can silently process the changes.

I just made the release for 1.12, so this sort of missed the cut. But could make a new version with this change, perhaps 1.12z3, for you to use on the Cubesatsim cards. What do you think?

— Reply to this email directly, view it on GitHub https://github.com/ac2cz/FoxTelem/issues/761#issuecomment-1292633070, or unsubscribe https://github.com/notifications/unsubscribe-auth/AASNGPHNVEF5BGAM6ZPBPDLWFGJ65ANCNFSM6AAAAAARPH45TE . You are receiving this because you authored the thread.Message ID: @.***>

ac2cz commented 1 year ago

This will be in 1.12z3 shortly