Skip to content

Update MerlinAU.sh#26

Merged
ExtremeFiretop merged 1 commit intoExtremeFiretop:devfrom
Martinski4GitHub:dev
Dec 7, 2023
Merged

Update MerlinAU.sh#26
ExtremeFiretop merged 1 commit intoExtremeFiretop:devfrom
Martinski4GitHub:dev

Conversation

@Martinski4GitHub
Copy link
Collaborator

Fixed bug causing the ZIP file to be deleted even when located on a USB-attached drive where Entware is installed & some code improvements in "Get_Custom_Setting()" function. Also modified "DoCleanUp" function to try to keep the ZIP file from being deleted under some conditions. Minor cleanup.

Fixed bug causing the ZIP file to be deleted even when located on a USB-attached drive where Entware is installed & some code improvements in "Get_Custom_Setting()" function. Also modified "_DoCleanUp_" function to try to keep the ZIP file from being deleted under some conditions. Minor cleanup.
@ExtremeFiretop ExtremeFiretop self-assigned this Dec 7, 2023
@ExtremeFiretop ExtremeFiretop added bug Something isn't working enhancement New feature or request labels Dec 7, 2023
@ExtremeFiretop ExtremeFiretop merged commit 55d48d3 into ExtremeFiretop:dev Dec 7, 2023
@ExtremeFiretop
Copy link
Owner

This looks solid to me.
I also just tested it, and confirmed the expected behavior on my router.

I think this is pretty much ready for master honestly.
Any concerns or last changes you'd want to make before pushing this to the "main" branch?

@Martinski4GitHub
Copy link
Collaborator Author

This looks solid to me. I also just tested it, and confirmed the expected behavior on my router.

I think this is pretty much ready for master honestly. Any concerns or last changes you'd want to make before pushing this to the "main" branch?

Yep, it's looking very good, more polished & production-ready for a "Main" branch release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants