[00:34:18]
<Yunohost Git/Infra notifications> [apps] OniriCorpe [commented](https://github.com/YunoHost/apps/pull/1875#issuecomment-2489824368) on [issue #1875](https://github.com/YunoHost/apps/pull/1875) Add Affine to wishlist: reject the license aint clear and that sounds like a big huge framework thing that does everything and may be unintegr...
[00:35:15]
<Émy – OniriCorpe> the wishlist rejection command isn't working anymore?
[05:47:43]
<tituspijean> It is only working if the PR is still opened, since it pushes a commit and renames the PR. 😉
[05:49:08]
<tituspijean> But Affine is already in the rejection list: https://github.com/YunoHost/apps/blob/master/rejectedlist.toml#L88
[05:51:14]
<tituspijean> I have just browsed its repo again, and even though there is still this weird goose chase to find all LICENSE files, it seems that now all the server packages do not have such a LICENSE file. So everything might be MIT-licensed now
[05:57:01]
<tituspijean> I've installed the app with Podman earlier this week, and it looks and feels very nice.
However, it is not fully thought for selfhosting yet, as there are several mentions/naggings to their Pro or Cloud subscription that do not apply for selfhosting. I was a bit distraught by them.
Additionally, there is generative algorithm integration that feels very forced, that cannot be disabled, and I was unsure if it was local or dependent on OpenAI.
[06:33:53]
<thomas> > 3.100? 🫠
Thought about i, but as manifest.toml has autoupdate entries, I fear it might prevent future autoupdates...
[07:24:23]
<Yunohost Git/Infra notifications> [wordpress_ynh] ericgaspar merged [pull request #241](https://github.com/YunoHost-Apps/wordpress_ynh/pull/241): [autopatch] Automatic patch attempt for helpers 2.1
[07:24:24]
<Yunohost Git/Infra notifications> [wordpress_ynh/testing] Update _common.sh - Éric Gaspar
[07:24:24]
<Yunohost Git/Infra notifications> [wordpress_ynh] ericgaspar pushed 7 commits to testing ([dd02182fcba6...42868e9c2fc9](https://github.com/YunoHost-Apps/wordpress_ynh/compare/dd02182fcba6...42868e9c2fc9))
[07:24:24]
<Yunohost Git/Infra notifications> [wordpress_ynh/testing] cleaning - Éric Gaspar
[07:24:25]
<Yunohost Git/Infra notifications> [wordpress_ynh/testing] Merge pull request #241 from yunohost-bot/auto-helpers-2.1 [autopatch] Automatic patch attempt for helpers 2.1 - eric_G
[07:26:09]
<Yunohost Git/Infra notifications> [wordpress_ynh] ericgaspar pushed 2 commits to testing ([42868e9c2fc9...ab67ab96697e](https://github.com/YunoHost-Apps/wordpress_ynh/compare/42868e9c2fc9...ab67ab96697e))
[07:26:13]
<Yunohost Git/Infra notifications> [wordpress_ynh/testing] Merge branch testing of https://github.com/YunoHost-Apps/wordpress_ynh into testing - Éric Gaspar
[07:26:13]
<Yunohost Git/Infra notifications> [wordpress_ynh/testing] Update tests.toml - Éric Gaspar
[07:26:23]
<Yunohost Git/Infra notifications> [wordpress_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/wordpress_ynh/pull/246#issuecomment-2490259719) on [issue #246](https://github.com/YunoHost-Apps/wordpress_ynh/pull/246) Update manifest.toml: testme
[07:26:27]
<Yunohost Git/Infra notifications> [wordpress_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/wordpress_ynh/pull/246#issuecomment-2490259837) on [issue #246](https://github.com/YunoHost-Apps/wordpress_ynh/pull/246) Update manifest.toml: Living in the past, are we? :cow::eye:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bullseye-dev.y...
[07:26:27]
<Yunohost Git/Infra notifications> [wordpress_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/wordpress_ynh/pull/246#issuecomment-2490259807) on [issue #246](https://github.com/YunoHost-Apps/wordpress_ynh/pull/246) Update manifest.toml: Test it, book it, worm it, technologic. :robot: :notes:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-ap...
[07:26:38]
<miro5001> > <thomas> > 3.100? 🫠
> Thought about i, but as manifest.toml has autoupdate entries, I fear it might prevent future autoupdates...
Post in the forum announcements that there was a mistake in the version of the package and users need to run the update manually
[07:38:45]
<oliviermgx> ynh doesn't accept docker container ?
Nor npm packages ?
Only git repo, right ?
[07:42:52]
<Aleks (he/him/il/lui)> Explain your actual issue..
[07:50:27]
<oliviermgx> moi ?
[08:00:16]
<eric_G> https://aria.im/_matrix/media/v1/download/matrix.org/GLtGRzWxwajOvTHTXWAfOrXf
[08:05:09]
<oliviermgx> My app could be installed using docker or npm. But I don't see the way to do that in the manifest.toml/install script. I only read :
`ynh_setup_source` is used to install an app from a zip or tar.gz file,
downloaded from an upstream source, like a git repository.
`ynh_setup_source` use the file manifest.toml
[08:05:39]
<oliviermgx> My app could be installed using docker or npm. But I don't see the way to do that in the manifest.toml/install script. I only read :
`ynh_setup_source` is used to install an app from a zip or tar.gz file,
downloaded from an upstream source, like a git repository.
`ynh_setup_source` use the file manifest.toml
but I would like to be sure
[08:23:48]
<thomas> > > <thomas> > 3.100? 🫠
> > Thought about i, but as manifest.toml has autoupdate entries, I fear it might prevent future autoupdates...
>
> Post in the forum announcements that there was a mistake in the version of the package and users need to run the update manually
Yup, a post in the forum might be a good idea. Also, I 'll add that in the package doc somewhere.
[08:59:33]
<eric_G> you can have a look to n8n_ynh package https://github.com/YunoHost-Apps/n8n_ynh
[09:16:09]
<oliviermgx> My app doesnt need any DB. But it seems it is necessary to configure one in manifest [resources.database], right ?
[09:24:06]
<tituspijean> > <thomas> > > <thomas> > 3.100? 🫠
> > > Thought about i, but as manifest.toml has autoupdate entries, I fear it might prevent future autoupdates...
> >
> > Post in the forum announcements that there was a mistake in the version of the package and users need to run the update manually
> Yup, a post in the forum might be a good idea. Also, I 'll add that in the package doc somewhere.
Additionally, since not all users are browsing the forum, I would suggest to:
Make one more commit on the wrong version naming (v<wrong version>~ynh<one more version>) that adds a `doc/POST_UPGRADE.d/v<wrong version>~ynh<one more version>.md` with the instruction to run a forced upgrade with `sudo yunohost app upgrade <the app> -u https://github.com/YunoHost-Apps/<the app>_ynh/tree/master -F`
Then wait for the catalog to be updated, then push the corrected version on master.
[09:24:18]
<tituspijean> That way, all admins will get a notification
[11:39:32]
<Aleks (he/him/il/lui)> > My app doesnt need any DB. But it seems it is necessary to configure one in manifest [resources.database], right ?
Ifyour app doesnt need a db, just remove the 'database resource' stuff from the manifest
[15:05:40]
<Yunohost Git/Infra notifications> [nextcloud_ynh] Thatoo [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/753#issuecomment-2491479666) on [issue #753](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/753) Forward Email?: testme
[15:05:52]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/753#issuecomment-2491480226) on [issue #753](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/753) Forward Email?: Living in the future, are we?
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api...
[15:10:19]
<oliviermgx> > <@Alekswag:matrix.org> Ifyour app doesnt need a db, just remove the 'database resource' stuff from the manifest
if I do this : https://paste.yunohost.org/raw/oyoletuqin
[15:11:37]
<orhtej2> > if I do this : https://paste.yunohost.org/raw/oyoletuqin
get rid of this section: https://github.com/oliviermgx/silex_ynh/blob/649be859fb7d2e44765ccbbaec77a2779838e551/manifest.toml#L136
[15:11:38]
<Yunohost Git/Infra notifications> [apps] yunohost-bot created new branch add-to-wishlist-twenty
[15:11:39]
<Yunohost Git/Infra notifications> [apps] yunohost-bot pushed 1 commit to add-to-wishlist-twenty: Add Twenty to wishlist ([3ef024e6](https://github.com/YunoHost/apps/commit/3ef024e6000faad0dc0b7a65570345c17435dacb))
[15:11:41]
<orhtej2> et voila
[15:11:43]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2718](https://github.com/YunoHost/apps/pull/2718): Add Twenty to wishlist
[15:11:43]
<Yunohost Git/Infra notifications> [apps] yunohost-bot labeled Wishlist on [pull request #2718](https://github.com/YunoHost/apps/pull/2718): Add Twenty to wishlist
[15:12:30]
<orhtej2> i.e. https://github.com/YunoHost-Apps/trilium_ynh/blob/master/manifest.toml does not use db resource
[15:18:28]
<oliviermgx> > i.e. https://github.com/YunoHost-Apps/trilium_ynh/blob/master/manifest.toml does not use db resource
thanks, what yhou see in my repo is a new test, before that, without db, I have had the error error:
> Specifying the type of db ('mysql' or 'postgresql') is mandatory for db resources
But I will try again
[15:21:12]
<orhtej2> > thanks, what yhou see in my repo is a new test, before that, without db, I have had the error error:
> > Specifying the type of db ('mysql' or 'postgresql') is mandatory for db resources
> But I will try again
you need to remove the whole section, not just db type
[15:21:55]
<oliviermgx> > you need to remove the whole section, not just db type
I see
[15:43:15]
<thomas> > > <thomas> > > <thomas> > 3.100? 🫠
> ...view rest of paste (11 lines): https://conference.yunohost.org/pastebin/47b218ce-d729-4170-aa33-3ccda7471863
Well... 3.10.0~ynh1 was already pushed to master... It's a very recent yunohost app, most likely not installed in any servers other than mine. Maybe I'll leave it this way for the moment.
[16:18:35]
<oliviermgx> thanks
[16:18:36]
<oliviermgx> delete line `type = "postgresql"` is not enough
delete `[resources.database]` is neceesary too
that's why ...
thx