connmon icon indicating copy to clipboard operation
connmon copied to clipboard

Code Improvements

Open Martinski4GitHub opened this issue 1 year ago • 1 comments

  1. Added code to check if sufficient JFFS storage space is available before moving database files/folders from USB location to JFFS partition. An error message is reported if not enough space is available, and the move request is aborted.

  2. Added code to check if available JFFS storage space falls below 20% of total space or 10MB (whichever is lower) and report a warning when it does. A warning message is also shown on the SSH CLI menu and WebGUI page.

  3. Added and modified code so that every time the SSH CLI menu is run, it checks if the WebGUI page has already been mounted. If not found mounted, the script will run the code to remount the WebGUI.

  4. Improved code that creates (during installation) and removes (during uninstallation) the "AddOns" menu tab entry for the WebGUI to make sure it checks for and takes into account other add-ons that may have been installed before or were later installed after the initial installation.

  5. Added "export PATH" statement to give the built-in binaries higher priority than the equivalent Entware binaries.

  6. Various code improvements & fine-tuning.

Martinski4GitHub avatar Feb 10 '25 03:02 Martinski4GitHub

@jackyaz.

This latest PR includes improvements & checks to help users who at some point attempt to change the current "Storage Location" from USB to JFFS to make sure that there's enough available free space to hold all the database-related files & folders.

Example screenshot when the currently available JFFS storage space looks good:

Connmon_v3 0 3_CLI_Menu_JFFS_OK

Example screenshots when the currently available JFFS storage space falls below the minimum reserved space (using a fake limit for testing & debugging purposes only) :

Connmon_v3 0 3_CLI_Menu_JFFS_Warning

Connmon_v3 0 3_WebGUI_JFFS_Warning

Martinski4GitHub avatar Feb 10 '25 09:02 Martinski4GitHub

@jackyaz,

Let me know if you have any questions or concerns about the latest changes in this PR. A handful of people have been running this latest code and it's been working well, so IMO it's ready to be merged.

Thanks.

Martinski4GitHub avatar Mar 02 '25 09:03 Martinski4GitHub

Thank you, Jack Yaz.

Martinski4GitHub avatar May 24 '25 03:05 Martinski4GitHub