Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add compatibility with new app API request in TrueNAS 24.10 #180

Merged
merged 5 commits into from
Nov 14, 2024

Conversation

itsniawan
Copy link
Contributor

@itsniawan itsniawan commented Nov 6, 2024

Proposed change

Type of change

  • Bugfix
  • New feature
  • Code quality improvements to existing code or addition of tests
  • Documentation

Additional information

Checklist

  • The code change is tested and works locally.
  • The code has been formatted using Black.
  • Tests have been added to verify that the new code works.
  • Documentation added/updated if required.

add new attribute names that changes in 24.10, system will show matching attribute names and ignores the other
In TS 24.10, app API request changed from "chart/release" to "app" and minor change in attribute names.
This commit check if TrueNAS version is before 24.10 or after, and use the right parameter to request app list and its attributes.
Change default value of "updated_version" from "unknown" to current version.
@TheOfficialMrBlah
Copy link

It seems to work fine for me, thank you very much!

@tomaae tomaae merged commit e23f611 into tomaae:master Nov 14, 2024
6 of 8 checks passed
@tomaae
Copy link
Owner

tomaae commented Nov 14, 2024

thanks, not sure about using default like that as it may cause infinite crash loop, but we will see. otherwise looks great

@github-actions github-actions bot locked and limited conversation to collaborators Nov 24, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants