Skip to content

feat: add app version compatibility checks - #3568

Merged
JammingBen merged 2 commits into
mainfrom
feat/admin-settings-app-compatibility
Oct 7, 2026
Merged

JammingBen merged 2 commits into
mainfrom
feat/admin-settings-app-compatibility

Conversation

@JammingBen

Copy link
Copy Markdown
Member

Add version compatibility checks to the app list in the admin-settings by reading the min and max OC versions from the awesome-apps repository.

Also add the supported max version to the app store.

closes #3412

image

Add version compatibility checks to the app list in the admin-settings
by reading the min and max OC versions from the awesome-apps repository.

Also add the supported max version to the app store.
id?: string
name?: string
icon?: string
/** @deprecated use the iconFillType on ApplicationFileExtension instead */

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed the deprecation because I found this to be actually useful, and a dozen of apps use it.

@JammingBen
JammingBen marked this pull request as ready for review October 6, 2026 13:55
@dschmidt

dschmidt commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

Have you thought about single incompatible versions or incompatible ranges?
Like 8.1.0 accidentally changed an api and the change was reverted in 8.1.1. So it's incompatible with 8.1.0 although otherwise the app should work from 7.0 to 9.x+

@JammingBen

Copy link
Copy Markdown
Member Author

Have you thought about single incompatible versions or incompatible ranges? Like 8.1.0 accidentally changed an api and the change was reverted in 8.1.1. So it's incompatible with 8.1.0 although otherwise the app should work from 7.0 to 9.x+

Tackling single versions is not planned for now. In this case I'd say the app is compatible with >7.0. I'd expect a user running 8.1.0 to then update to 8.1.1 to ensure that everything works.

@AlexAndBear AlexAndBear left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very helpful 👍

@JammingBen
JammingBen merged commit 9bf25c9 into main Oct 7, 2026
31 checks passed
@JammingBen
JammingBen deleted the feat/admin-settings-app-compatibility branch October 7, 2026 06:58
@openclouders openclouders mentioned this pull request Oct 7, 2026
1 task
openclouders pushed a commit that referenced this pull request Oct 7, 2026
…mpatibility

feat: add app version compatibility checks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Extend apps in admin-settings by version compatibility info

3 participants