[Freshrss] YNH_APP_INSTANCE_NAME in hook script

Contribute Room Apps packaging
1

Hi,
I have a problem with freshrss package.
In post_user_create and post_user_delete hooks, I need YNH_APP_INSTANCE_NAME in order to create/delete users into the right instance.
But it looks like the variable is not provided. Am I doing something wrong or is it a bug ?

And something not related :
freshrss app doen’t appear in the jenkins ci anymore, should I do something do get it back ?

Thanks.

2

Hi

I don’t think hooks have the variable $YNH_APP_INSTANCE_NAME, because hooks are not executed by apps, but by yunohost directly.
But, if you need this variable, the better way si to set it in the hook script during the install of the app.

PS: https://ci-apps.yunohost.org/jenkins/job/freshrss%20(Community)/

3

Ok, I did it that way. Thanks a lot.

Thanks, I need to bookmark this one.
I was going on https://ci-apps-dev.yunohost.org/

4

This other CI is for packagers who don’t (or don’t want to) use Package check on their own.
On this CI, any packager can push his apps on a personal directory, and then the CI will add and check them.
More informations here.

5

Hello @plopoyop,

I created an issue on the app repository and found this topic after… To keep a writing of it, here is a link to the issue : https://github.com/YunoHost-Apps/freshrss_ynh/issues/53

6

Hi,

Is this solution still the best way to find App instance name in hook user create ?

By the way @plopoyop , it seems that you replace APPNAMETOCHANGE only in your upgrade script and not in the install script ?

7

Hi,
I fixed that this week.
You can upgrade on the testing branch.

1 Like