Brian Wilson

🤔 We currently batch messages to EDSM and send about once per minute (with the intent of keeping the EDSM dashboard reasonably up to date without hammering the server). As...

Anthor has given the all-clear to increase our send frequency.

There actually is already an option to disable telemetry reporting via the config files, specifically via the `DisableTelemetry` property in the `eddi.json` config file.

I've got several other branches cooking at the moment but yes that is a possibility.

Fair points. - I have no problem with putting CheckUpgrade() and StartUpgrade() into a separate class callable from app.xaml.cs. - Hmm. I agree that we should probably be handling these...

No. Global state is as you see it in EDDI.cs. It's not tangled up in the front end UI thank goodness. I'm referring to the `OnEvent()` method in EDDI.cs: ```cs...

Agreed, the highlighted code ought to be moved. The section immediately below the section you highlighted ```cs // Update the commander object, if it exists if (Cmdr != null) {...

It occurs to me that there may be a simpler approach. We could set up two way bindings between the UI elements you identified and the `Cmdr` object, making provisions...

Yes, `BindingOperations.EnableCollectionSynchronization` is part of how we enable two way binding for the observable collection `inventory` in the Cargo Monitor. https://docs.microsoft.com/en-us/dotnet/framework/wpf/data/how-to-create-and-bind-to-an-observablecollection Please also note that some bound items in EddiCargoMonitor/ConfigurationWindow.xaml...

Correct. A monitor can be a producer but isn't necessarily so.