Skip to content

fix/tools_update: return upgradable apps, not all apps #2144

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

Closed
wants to merge 1 commit into from

Conversation

kay0u
Copy link
Member

@kay0u kay0u commented Aug 4, 2025

The problem

apps contains all the apps installed, not the upgradable ones

Solution

use upgradable_apps instead

PR Status

...

How to test

...

@alexAubin
Copy link
Member

Eeeh but that's actually a feature and not a bug (i think ?) because the point was to also list apps that are in special cases such as url_required, possibly because they were in the catalog in the past but are not anymore (otherwise you may never notice it)

Can you elaborate what made you notice this and want to patch it ?

@kay0u
Copy link
Member Author

kay0u commented Aug 4, 2025

Edit: It seems that my "fix" doesn't fix my main problem explained bellow
Edit² it does fix it, just need to restart yunohost-api, but as you said, it's intentional, so the problem is elsewhere.
Edit²²Hmm after the reboot, without this patch, I don't have this problem anymore, could it be a browser cache problem?

Can you elaborate what made you notice this and want to patch it ?

Sure, the command sudo yunohost tools update return the list of my installed apps, but maybe that's intentional.

For example:

  33:
    description: Manage passwords and other sensitive informations
    domain_path: my.domain.tld/
    id: vaultwarden
    name: Vaultwarden
    upgrade:
      current_version: 1.34.3~ynh1
      message: Force-upgrading the app (to the same version) can sometimes be useful to rebuild the app and configurations.
      new_revision: d2a91e14c0c1537f286c33d8d6a05c0eb4ebdbc3
      new_version: 1.34.3~ynh1
      requirements: None
      specific_channel: None
      specific_channel_message: None
      status: up_to_date
      url: None
    version: 1.34.3~ynh1

My problem is that, when I navigate to the update page in the web admin, I can see apps that are already up to date.

Before the YNH upgrade (I'm was on testing already), I had the Actual apps to upgrade, I did the YNH upgrade, upgrade Actual, but the app is still tagged as upgradable in the web page.

Actual (actual) de 25.7.1ynh1 à 25.8.0ynh1

But when I check the current version:

# cat /etc/yunohost/apps/actual/manifest.toml | grep version
version = "25.8.0~ynh1"

When i navigate to domain.tld/yunohost/admin/#/apps/actual/_core, I can see the message:

Upgrade

The app can be upgraded from version 25.7.1~ynh1 to 25.8.0~ynh1

@kay0u kay0u closed this Aug 4, 2025
@kay0u kay0u deleted the fix-tools-update branch August 4, 2025 21:56
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