-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add deprecation notice for 6M flash devices #75
Conversation
lib/config_default.js
Outdated
@@ -200,9 +200,21 @@ export const config = { | |||
"AP121U", | |||
"D-Link DIR-615", | |||
"D-Link DIR-615 D", | |||
"D-Link DIR-825", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why? This has been fixed and the limit only applies to factory installations (Which are removed from Gluon)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I went through the list of targets of the latest release and it was marked as only 6M flash
https://github.com/freifunk-gluon/gluon/blob/7feebe96a6b93d567bd342ffd01abcb3022383c5/targets/ath79-generic#L124
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is outdated then, see openwrt/openwrt@aca8bb5
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
1654ffe
to
7b6dae6
Compare
@blocktrron can we merge this PR? |
@grische please rebase |
I prefer #81 |
Let's focus on this PR then. |
Description
6M flash devices are not future-proof and have been dropped by some communities already.
Motivation and Context
freifunk-darmstadt/gluon-firmware-selector#159
How Has This Been Tested?
N/A
Checklist: