[09:56:39]
<eric_G> I would like to see an option to creat sub folders in the `data_dir` in `manifest.toml`.
Something like:
```
[resources.data_dir]
subfolder = [ "groups", "recordings" ]
```
(It may not be useful for many apps…)
[10:05:16]
<rodinux[m]> Hello, I try understand how debug my issue https://github.com/YunoHost-Apps/cryptpad_ynh/issues/173 talking with cryptpad community
[10:08:41]
<rodinux[m]> Perhaps it is because the storage folders should own `1:1` and not `cryptpad:www-data`. Is it a constraint need to keep the owner `www-data` ?
[11:52:44]
<Yunohost Git/Infra notifications> [apps] @tituspijean [commented](https://github.com/YunoHost/apps/pull/1638#issuecomment-1446193209) on [issue #1638](https://github.com/YunoHost/apps/pull/1638) Update app levels according to CI results: > Bad UX because shitload of warnings Some of these are due to the service restart not being detected by the helper, ...
[12:14:39]
<Aleks (he/him/il/lui)> > <@rodinux07:matrix.org> Perhaps it is because the storage folders should own `1:1` and not `cryptpad:www-data`. Is it a constraint need to keep the owner `www-data` ?
uuuh what do you mean `1:1` o_O
[12:16:34]
<rodinux[m]> I mean a developper from cryptpad told me perhaps is not a good way have owners like `cryptpad:www-data` for the storage folders...
[12:18:19]
<rodinux[m]> I have tried on a test cryptpad `chown -R cryptpad:` for the folders datastore, data/archive, data/pins, data/task, block, blob, data/blobstage, data/decrees, data/logs
[12:18:29]
<Aleks (he/him/il/lui)> wokay, but did he explain why ...
[12:19:33]
<rodinux[m]> No, he don't know about how YunoHost is cored...
[12:21:18]
<rodinux[m]> I think the problem is more about `Strict-Transport-Security : le site a spécifié un en-tête qui n’a pas pu être analysé correctement.
2 08a7e60d2dc1c2059e4b9ddf0108bae45d797b4f1b667efa
Blocage d’une requête multiorigine (Cross-Origin Request) : la politique « Same Origin » ne permet pas de consulter la ressource distante située sur https://cryptpad.domain.tld/blob/08/08a7e60d2dc1c2059e4b9ddf0108bae45d797b4f1b667efa. Raison : l’en-tête CORS « Access-Control-Allow-Origin » ne correspond pas à « 'https://sandbox-cryptpad.domain.tld' »`
[12:37:00]
<Yunohost Git/Infra notifications> [apps] @alexAubin [commented](https://github.com/YunoHost/apps/pull/1638#issuecomment-1446254988) on [issue #1638](https://github.com/YunoHost/apps/pull/1638) Update app levels according to CI results: >Some of these are due to the service restart not being detected by the helper, its a bit unfair to drop the level due ...
[14:40:48]
<eric_G> > <@ericg:matrix.org> I would like to see an option to creat sub folders in the `data_dir` in `manifest.toml`.
> Something like:
> ```
> [resources.data_dir]
> subfolder = [ "groups", "recordings" ]
> ```
>
> (It may not be useful for many apps…)
Aleks (he/him/il/lui): Any thought on this one?
[14:53:28]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @KaeruCT [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/485#issuecomment-1446473151) on [issue #485](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/485) Upgrade fails due to moving data elsewhere and deleting the original directory: I ran into this issue as well. My biggest gripe is that it also prevents backups from being restored, because the resto...
[14:56:16]
<Aleks (he/him/il/lui)> Eeeh are there many apps for which this is relevant ?
[14:56:37]
<Aleks (he/him/il/lui)> You mean you have to mkdir those subfolders during install/upgrade ?
[15:23:42]
<Yunohost Git/Infra notifications> [apps] @alexAubin [commented](https://github.com/YunoHost/apps/pull/1638#issuecomment-1446531101) on [issue #1638](https://github.com/YunoHost/apps/pull/1638) Update app levels according to CI results: (Will try to dig into both items this afternoon)
[16:56:05]
<eric_G> Aleks (he/him/il/lui): about 35 apps are using `mkdir -p $datadir/some_file`
[16:57:49]
<eric_G> ```
funkwhale
jirafeau
transmition
galene
mobilizon
code server
lychee
garage
forgejo
gitea
lxd-dashboard
fallback
distbin
answer
photoprism
tracim
airsonic
fallback
pleroma
snserver
akkoma
spacedeck
anfora
minetest
nomad
lemmy
paperless
gogs
limesurvey
flood
zeronet
lufi
peachpub
mstream
mopidy
```
[18:42:41]
<Aleks (he/him/il/lui)> huhu have you been `grep`ing the whole app catalog ? :D
[18:43:25]
<Aleks (he/him/il/lui)> but okay that sounds like a legit thing to handle from the app resource then 👍️
[18:44:09]
<eric_G> I searched for `mkdir -p $datadir/` in GitHub YunoHost app repo... 😅
[18:45:32]
<Aleks (he/him/il/lui)> 👍️
[18:57:53]
<Yunohost Git/Infra notifications> [apps] @alexAubin [commented](https://github.com/YunoHost/apps/pull/1638#issuecomment-1446884621) on [issue #1638](https://github.com/YunoHost/apps/pull/1638) Update app levels according to CI results: Followup PRs, to be tested https://github.com/YunoHost/yunohost/pull/1617 https://github.com/YunoHost/yunohost/pull/161...
[19:06:30]
<gredin67> Aleks (he/him/il/lui): on parle registar sur bbb
[21:02:58]
<Yunohost Git/Infra notifications> [dokuwiki_ynh] @Gofannon [commented](https://github.com/YunoHost-Apps/dokuwiki_ynh/pull/95#discussion_r1119309187) on pull request #95 Apply example_ynh: To me, the file is for "packagers" or "advancers users" who would like to know what is inside the new version released. ...
[22:00:40]
<Krakinou> Hello hello again : I'm still struggling on the v2 conversion for my apps. One of the main issue I have is that the toml file does not allows to make conditionnal ressources. I have a bunch of them for package dependencies (as mentionned a few days ago), but also for port (https://github.com/YunoHost-Apps/domoticz_ynh/blob/27311ad7b8fc72fb065f60c769f21546e9b647a9/scripts/install#L86) or even permission (https://github.com/YunoHost-Apps/domoticz_ynh/blob/27311ad7b8fc72fb065f60c769f21546e9b647a9/scripts/install#L264)
[22:00:40]
<Krakinou> Although I find it much more convenient than the previous way of doing (and also easier to implement one you get the idea), the point is that it's much less "flexible" so far
[22:00:40]
<Krakinou> I must also admit that my scripts are maybe not 100% "orthodox" 😕
[22:02:19]
<Krakinou> anyway, I can overcome this by letting the old logic in the script (still using the previous helpers), but package_linter goes crazy with warnings all over the place in this case