-
Notifications
You must be signed in to change notification settings - Fork 28
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
Tell admins upfront when market doesn't have compatible update versions #12
Comments
@DeepDiver1975 for some reason I can't assign @VicDeo, can you adjust the repo settings ? Thanks |
done |
@VicDeo please have a look. Not sure if the check should be in the market app or core, but regardless we need to make sure we have it. |
@PVince81 @DeepDiver1975 I guess for web update only? |
Both web update and cli update from the updater. Need to clear out where to put the code (market app or updater app or both) |
@PVince81 into updatenotification app I guess. |
Hmm, but we should also show a warning on the updater page / CLI output when running the updater because that's where it matters most. |
We do that already in |
Not sure if we have this already in core, please look into it.
Goal is to avoid surprises and be able to know before the upgrade whether the market apps will work with the new version. For this, need to fetch the metadata of the apps for the next OC version.
If apps not compatible, display a warning about these that say they will be disabled and not available.
The admin can then decide to either upgrade later once these apps are available or skip the warning.
@VicDeo @DeepDiver1975
The text was updated successfully, but these errors were encountered: