Skip to content

Fix and Improvements#454

Merged
ExtremeFiretop merged 1 commit intoExtremeFiretop:devfrom
Martinski4GitHub:dev
Apr 12, 2025
Merged

Fix and Improvements#454
ExtremeFiretop merged 1 commit intoExtremeFiretop:devfrom
Martinski4GitHub:dev

Conversation

@Martinski4GitHub
Copy link
Collaborator

@Martinski4GitHub Martinski4GitHub commented Apr 11, 2025

- Fixed bug introduced in PR ExtremeFiretop#453 which was allowing installation in all unsupported models.
- Miscellaneous code improvements.
@ExtremeFiretop
Copy link
Owner

ExtremeFiretop commented Apr 11, 2025

Actually not a bug but by design. Has it get installed to be uninstalled.
From the very beginning of the script this was the design.

@ExtremeFiretop
Copy link
Owner

ExtremeFiretop commented Apr 11, 2025

@Martinski4GitHub

I believe you might have the impression the unsupported models should stop the install?
But since the original design was implemented it allowed install but blocked updates if unsupported and prompted to be uninstalled completely.

If we were to do your method were it blocks install completely we could remove this
image

Or this?
image

@ExtremeFiretop
Copy link
Owner

Reply from user:

Post in thread 'MerlinAU v1.4.1 - The Ultimate Firmware Auto-Updater (GNUTON SUPPORT!)' https://www.snbforums.com/threads/merlinau-v1-4-1-the-ultimate-firmware-auto-updater-gnuton-support.91326/post-951249

@ExtremeFiretop
Copy link
Owner

All good to go! Merging 🥇

@ExtremeFiretop ExtremeFiretop merged commit 8854210 into ExtremeFiretop:dev Apr 12, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants