Fix The Update Check Status Post Upgrade

[ExtremeFiretop]

Fix The Update Check Status Post Upgrade in the WebUI

Issue seems to be it does not check or set the FW_UPDATE_CHECK value in the settings file post upgrade as seen below:

We should be seeing this run:

Which sets the value.

Solution seems to be to simply add the missing: ConfirmCronJobForFWAutoUpdates within the DoInstallation routine.
Without this, on first upgrade after running: sh /jffs/scripts/MerlinAU.sh develop

We get this result in the configuration file:

Simply running the script a second time also fixes the "bug" but we can add this to the Install routine I think without any impact.

[Martinski4GitHub]

Fix The Update Check Status Post Upgrade in the WebUI

Issue seems to be it does not check or set the FW_UPDATE_CHECK value in the settings file post upgrade as seen below:
...

We should be seeing this run:
....

Which sets the value.

Solution seems to be to simply add the missing: ConfirmCronJobForFWAutoUpdates within the DoInstallation routine.

Yeah, that should work just fine.

Without this, on first upgrade after running: sh /jffs/scripts/MerlinAU.sh develop

We get this result in the configuration file:

Simply running the script a second time also fixes the "bug" but we can add this to the Install routine I think without any impact.

Sounds good.

[Martinski4GitHub]

Good to go!!

[ExtremeFiretop]

Fix The Update Check Status Post Upgrade in the WebUI
Issue seems to be it does not check or set the FW_UPDATE_CHECK value in the settings file post upgrade as seen below:
...
We should be seeing this run:
....
Which sets the value.
Solution seems to be to simply add the missing: ConfirmCronJobForFWAutoUpdates within the DoInstallation routine.

Yeah, that should work just fine.

Without this, on first upgrade after running: sh /jffs/scripts/MerlinAU.sh develop
We get this result in the configuration file:
Simply running the script a second time also fixes the "bug" but we can add this to the Install routine I think without any impact.

Sounds good.

Now that's in dev; I'll just test it again! :)

[ExtremeFiretop]

@Martinski4GitHub

Bingo! Was his name-o! I now see the missing step when doing the update process:

And loading the WebUI right away shows "Enabled"

Thanks buddy for the review!