[00:25:30]
<m606> So the helper is fine and indeed compares old vs new value as expected.
For the record, here the issue was the getter `get__archives_paths` which was calling $archives_paths variable thinking it would come from the app's settings.yml. It is actually the case in "displaying context" (displaying config panel or running `yunohost app config get $app --full --debug`), but not in "saving a modified setting context" because in that context $archives_paths does not refer to the existing setting value but to the new one provided by admin and about to be validated and saved. Hence the fact that helper was uselessly comparing new value vs new value and the related bug. It was only working when having more than 2 line due I believie to the bad support for multiline text by the "validate" helper. But in my case, I simply changed the helper to grep the value from the app config file (altenatively I could probably have `ynh_app_setting_get`).
[00:25:56]
<m606> So the helper is fine and indeed compares old vs new value as expected.
For the record, here the issue was the getter `get__archives_paths` which was calling $archives_paths variable thinking it would come from the app's settings.yml. It is actually the case in "displaying context" (displaying config panel or running `yunohost app config get $app --full --debug`), but not in "saving a modified setting context" because in that context $archives_paths does not refer to the existing setting value but to the new one provided by admin and about to be validated and saved. Hence the fact that helper was uselessly comparing new value vs new value and the related bug. It was only working when having more than 2 lines due I believe to the bad support for multiline text by the "validate" helper. But in my case, I simply changed the helper to grep the value from the app config file (altenatively I could probably have `ynh_app_setting_get`).
[06:10:47]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch glances
[06:10:56]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch add-to-wishlist-299ko
[06:10:56]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch add-to-wishlist-venus
[06:11:05]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch add-to-wishlist-agorakit
[06:11:09]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch add-to-wishlist-organehrm
[06:11:17]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch add-to-wishlist-espocrm
[06:11:19]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch add-to-wishlist-prose
[06:11:21]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch add-to-wishlist-drupal-cms
[06:11:41]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to master: Add Davis to catalog (#2921) * Update apps.toml * Create davis.png ([c07750dc](https://github.com/YunoHost/apps/commit/c07750dc7a00bfc0ed95fc7555ccd2e13ef9a2eb))
[06:11:41]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #2921](https://github.com/YunoHost/apps/pull/2921): Add Davis to catalog
[06:11:41]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch davis
[07:28:12]
<Yunohost Git/Infra notifications> [synapse_ynh] centralscrutinizer21 [commented](https://github.com/YunoHost-Apps/synapse_ynh/pull/528#issuecomment-2812038161) on [issue #528](https://github.com/YunoHost-Apps/synapse_ynh/pull/528) Element call backend: With the last commits the error could not validate external IP has disappeared. Well done. Audio and video are still n...
[08:11:34]
<Yunohost Git/Infra notifications> [jellyfin_ynh] botagas opened [issue #189](https://github.com/YunoHost-Apps/jellyfin_ynh/issues/189): Avoid backing up data by Subtitle Edit
[08:32:45]
<Yunohost Git/Infra notifications> [jellyfin_ynh] botagas opened [issue #190](https://github.com/YunoHost-Apps/jellyfin_ynh/issues/190): All settings reset on upgrade
[08:34:54]
<Yunohost Git/Infra notifications> [jellyfin_ynh] botagas opened [issue #191](https://github.com/YunoHost-Apps/jellyfin_ynh/issues/191): Triggering restart within Jellyfin does not restart the service - only stops it
[08:57:32]
<Yunohost Git/Infra notifications> [jellyfin_ynh] botagas opened [issue #192](https://github.com/YunoHost-Apps/jellyfin_ynh/issues/192): PORT is not opened on upgrade
[09:47:46]
<eric_G> how do we fixe this README warning? https://ci-apps.yunohost.org/ci/job/17369
[09:50:57]
<tituspijean[m]> This is the check : https://github.com/YunoHost/package_linter/blob/86ae6b989add4672e1242e39ddf0340a9aec1d89/tests/test_app.py#L579
IIRC the old README version was adding a `<!-- This README was automatically generated -->` thingy at the beginning of the file
[09:54:10]
<tituspijean[m]> uh, it's there actually https://github.com/YunoHost/apps_tools/blob/7603de396b02b8f154870c7fef484c31d9cb69cb/readme_generator/README.md.j2#L19
(and line 14 of your app)
[09:54:13]
<tituspijean[m]> I dunno...
[09:56:48]
<tituspijean[m]> ah it's these (https://github.com/YunoHost/package_linter/blob/86ae6b989add4672e1242e39ddf0340a9aec1d89/tests/test_app.py#L581
https://github.com/YunoHost/package_linter/blob/86ae6b989add4672e1242e39ddf0340a9aec1d89/tests/test_app.py#L585)
which have changed with the new README format
[10:01:39]
<Thomas> https://github.com/YunoHost/apps_tools/pull/30
[10:15:09]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to testing: Update manifest.toml ([2350f9a8](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/2350f9a85a1791eb9a43d280265701fbed5b995e))
[10:19:39]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to testing: Update manifest.toml ([a3d27dee](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/a3d27dee42ccbd7147c1ac822a6aad775107a90d))
[10:24:51]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 3 commits to v2_PostgreSQL ([b713f30e3989...380730e444fc](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/b713f30e3989...380730e444fc))
[10:24:52]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Update manifest.toml - eric_G
[10:24:52]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Merge branch testing into v2_PostgreSQL - eric_G
[10:24:52]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Update manifest.toml - eric_G
[10:29:56]
<rodinux> eric_G: just a question, soon I will upgrade a Yunohost from 11 to 12 with a nextcloud on the branch v2_PostgeSQL, should I upgrade nextcloud to last version available for YNH11 with this branch first ? commit `42513866c8d597a4a75835b7ccc765e1b0828de3` ? 30.0.2 ?
[10:35:03]
<rodinux> Actually there are still on 29.0.7~ynh1
[10:36:23]
<eric_G> It should make no difference. The database migration process is the hot point: it hasn't been tested very much.
[10:37:14]
<rodinux> with postgresql... from 13 to 15 ??
[10:38:27]
<eric_G> from Mysql -> PostgreSQL
[10:40:44]
<rodinux> for this nextcloud, it is using postgresql, so I upgrade it only from the branch v2\_Postgresql...
[10:43:46]
<rodinux> I am just a little affraid about the process of migrations upgrading to Yunohost12...
[10:45:25]
<rodinux> I have try the process on another instance and indeed, I couldn't migrate mysql to postgesql...
[14:18:11]
<Yunohost Git/Infra notifications> [jellyfin_ynh] botagas edited [issue #189](https://github.com/YunoHost-Apps/jellyfin_ynh/issues/189): Avoid backing up data by Subtitle Extract
[14:42:46]
<Paprika> Can we get some traction on `flarum_ynh`? PRs have been standing for almost a month now, while the issues regarding them have been there ever since 2024
[14:43:01]
<Paprika> Can we get some traction on `flarum_ynh`? PRs have been standing for almost a month now, while the issues regarding them have been there since 2024
[15:54:46]
<thatoo> Is it possible to exclude a value for an item of config_panel.toml or manifest.toml? Something like
```
[section.my_option_id]
type = "group"
default = "visitors"
exclude = "admins"
```
[17:42:53]
<Yunohost Git/Infra notifications> [example_ynh] Salamandar created new branch main
[17:42:53]
<Yunohost Git/Infra notifications> Salamandar edited repository example_ynh: Example package for YunoHost applications. https://github.com/YunoHost/example_ynh
[17:42:55]
<Yunohost Git/Infra notifications> [package_linter] Salamandar created new branch main
[17:42:55]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 1 commit to main: example_ynh is now on main default branch ([45168613](https://github.com/YunoHost/package_linter/commit/45168613c130371f66296256940066938b09c63e))
[17:42:55]
<Yunohost Git/Infra notifications> [apps_tools] Salamandar pushed 1 commit to main: example_ynh is now on main default branch ([571232b3](https://github.com/YunoHost/apps_tools/commit/571232b3059bb19be61861acbabb2068f9841c1a))
[17:42:55]
<Yunohost Git/Infra notifications> [example_ynh] Salamandar deleted branch master
[17:42:55]
<Yunohost Git/Infra notifications> [package_linter] github-actions[bot] opened [pull request #176](https://github.com/YunoHost/package_linter/pull/176): Format Python code with Black
[17:42:56]
<Yunohost Git/Infra notifications> [joplin_ynh] DerDrachenKater opened [issue #90](https://github.com/YunoHost-Apps/joplin_ynh/issues/90): Error during installation
[17:45:39]
<Yunohost Git/Infra notifications> [package_check] Salamandar pushed 1 commit to master: Package linter default branch is now main ([8e97ca5a](https://github.com/YunoHost/package_check/commit/8e97ca5ae89db2107c354913f24945496d8b05d0))
[17:45:43]
<Yunohost Git/Infra notifications> Salamandar edited repository package_linter: Linter for YunoHost applications packages https://github.com/YunoHost/package_linter
[17:46:39]
<Yunohost Git/Infra notifications> [package_linter] Salamandar closed [pull request #164](https://github.com/YunoHost/package_linter/pull/164): Add test to validate SSO config for bookworm
[17:46:39]
<Yunohost Git/Infra notifications> [package_linter] Salamandar deleted branch master
[17:46:43]
<Yunohost Git/Infra notifications> [package_linter] Salamandar created new branch master
[17:46:45]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 12 commits to master ([161c647f8aa9^...86ae6b989add](https://github.com/YunoHost/package_linter/compare/161c647f8aa9^...86ae6b989add))
[17:46:48]
<Yunohost Git/Infra notifications> [package_linter/master] Complain about legacy YNH_APP_ARG_FOOBAR syntax - Alexandre Aubin
[17:46:53]
<Yunohost Git/Infra notifications> [package_linter/master] :art: Format Python code with Black - yunohost-bot
[17:47:59]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 1 commit to main: example_ynh is now on main default branch ([07e5a30a](https://github.com/YunoHost/package_linter/commit/07e5a30a5cf9448a40733ab721a55b6f9b9c152a))
[17:48:14]
<Yunohost Git/Infra notifications> [package_linter] Salamandar deleted branch master
[17:48:22]
<Yunohost Git/Infra notifications> [package_linter] Salamandar closed [pull request #176](https://github.com/YunoHost/package_linter/pull/176): Format Python code with Black
[17:48:25]
<Yunohost Git/Infra notifications> [package_linter] Salamandar deleted branch actions/black
[17:49:59]
<Yunohost Git/Infra notifications> [package_check] Salamandar pushed 1 commit to master: Package check default branch is now main ([57bc7703](https://github.com/YunoHost/package_check/commit/57bc7703d50d853bd0e3c4805250e30efe837141))
[17:49:59]
<orhtej2> > <@yunohostinfra:matrix.org> Salamandar edited repository example_ynh: Example package for YunoHost applications. https://github.com/YunoHost/example_ynh
That's risky, now all the repos created from template will have wrong default branch I think?
[17:50:00]
<Yunohost Git/Infra notifications> [package_check] Salamandar created new branch main
[17:50:21]
<Yunohost Git/Infra notifications> Salamandar edited repository package_check: Shell script which check package actions: install, remove, upgrade, backup, restore… https://github.com/YunoHost/package_check
[18:00:02]
<Salamandar> main is already supported as branch for the apps
[18:00:05]
<Salamandar> there are some that do that actually
[18:26:44]
<Yunohost Git/Infra notifications> [synapse_ynh] Josue-T [commented](https://github.com/YunoHost-Apps/synapse_ynh/pull/528#issuecomment-2813727715) on [issue #528](https://github.com/YunoHost-Apps/synapse_ynh/pull/528) Element call backend: Thank you for the test and the help. Im not sure that the issue is related to the .well-known config. But in some test ...
[19:11:14]
<Yunohost Git/Infra notifications> [nextcloud_ynh] JimboJoe approved [pull request #796](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/796#pullrequestreview-2776683456) Testing: LGTM 👍
[19:16:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to master: Testing (#796) * move function to common.sh (#764) * move function to common.sh * Update config * add default_phone_... ([a8321509](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/a8321509c0fa3c5c42dd492855530b6e30c37815))
[19:16:05]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar merged [pull request #796](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/796): Testing
[19:16:41]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 2 commits to testing ([a3d27dee42cc...1c9f4768589b](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/a3d27dee42cc...1c9f4768589b))
[19:16:42]
<Yunohost Git/Infra notifications> [nextcloud_ynh/testing] Merge branch master into testing - eric_G
[19:16:42]
<Yunohost Git/Infra notifications> [nextcloud_ynh/testing] Testing (#796) * move function to common.sh (#764) * move function to common.sh * Update config * add default_phone_... - eric_G
[19:16:53]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 3 commits to v2_PostgreSQL ([380730e444fc...3c169c88e3cf](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/380730e444fc...3c169c88e3cf))
[19:16:54]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Testing (#796) * move function to common.sh (#764) * move function to common.sh * Update config * add default_phone_... - eric_G
[19:16:55]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Merge branch testing into v2_PostgreSQL - eric_G
[19:16:55]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Merge branch master into testing - eric_G
[19:44:37]
<Yunohost Git/Infra notifications> ericgaspar created repository many-notes_ynh https://github.com/YunoHost-Apps/many-notes_ynh
[19:49:01]
<Yunohost Git/Infra notifications> ericgaspar edited repository many-notes_ynh https://github.com/YunoHost-Apps/many-notes_ynh
[19:49:38]
<Yunohost Git/Infra notifications> ericgaspar edited repository many-notes_ynh: Many Notes package for YunoHost https://github.com/YunoHost-Apps/many-notes_ynh
[19:49:38]
<Yunohost Git/Infra notifications> ericgaspar edited repository many-notes_ynh: Many Notes package for YunoHost https://github.com/YunoHost-Apps/many-notes_ynh
[19:54:46]
<Yunohost Git/Infra notifications> [apps] ericgaspar created new branch many-notes
[19:54:47]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to many-notes: Update apps.toml ([5995ab6c](https://github.com/YunoHost/apps/commit/5995ab6c8591e06b18244059ba48b75276765ac9))
[19:55:14]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to many-notes: Create many-notes.png ([1597a3c8](https://github.com/YunoHost/apps/commit/1597a3c87425b0b4d63e5277624178bb12260bdb))
[19:55:32]
<Yunohost Git/Infra notifications> [apps] ericgaspar opened [pull request #2922](https://github.com/YunoHost/apps/pull/2922): Add Many notes
[20:10:44]
<florent[m]> Using a yunohost package as a dependency of another one is a piece of nightmare, isn't it?
I try to use the dex package for my app, but there are scenarios that seem hard to solve, like how to restore an app with its dependency (here Dex) if an upgrade fails?
Are there good practices for these cases?
[20:11:27]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #2922](https://github.com/YunoHost/apps/pull/2922): Add Many notes
[20:11:28]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to master: Add Many notes (#2922) * Update apps.toml * Create many-notes.png ([8a2092f7](https://github.com/YunoHost/apps/commit/8a2092f707e92a9b8a0c45d51d202a81af2f3ac9))
[20:11:29]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch many-notes
[20:13:57]
<florent[m]> Answering to my own question, Outline has paved the road:
https://github.com/YunoHost-Apps/outline_ynh/blob/7ea877214edf1db867e8f0d3eefe2f7462b5f8b8/scripts/restore#L18-L26
[20:17:38]
<florent[m]> But that still seem awkward (but a lesser evil) to have the restore script not being symmetric with backup (as it does not trigger the backup of Dex, but that's legit):
https://github.com/YunoHost-Apps/outline\_ynh/blob/master/scripts/backup
[20:19:50]
<florent[m]> My understanding is that having a yunohost app depending on another app is something hard to handle and probably worth to avoid when possible.
[22:09:09]
<Yunohost Git/Infra notifications> [joplin_ynh] yunohost-bot opened [pull request #91](https://github.com/YunoHost-Apps/joplin_ynh/pull/91): Upgrade to v3.3.5
[22:14:53]
<Yunohost Git/Infra notifications> [piped_ynh] yunohost-bot opened [pull request #200](https://github.com/YunoHost-Apps/piped_ynh/pull/200): Upgrade sources
[22:20:52]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 20 pending update PRs
- 8 new apps PRs
- 11 failed apps updates: dokos, funkwhale, khatru-pyramid, localai, ofbiz, opencloud, phpldapadmin, phpmyadmin, pixelfedglitch, snweb, stremio
See the full log here: https://paste.yunohost.org/raw/irexosuwot
[22:47:31]
<Yunohost Git/Infra notifications> [penpot_ynh] orhtej2 merged [pull request #106](https://github.com/YunoHost-Apps/penpot_ynh/pull/106): Upgrade sources
[22:47:34]
<Yunohost Git/Infra notifications> [penpot_ynh] orhtej2 deleted branch ci-auto-update-sources-250418
[22:48:09]
<Yunohost Git/Infra notifications> [piped_ynh] orhtej2 merged [pull request #200](https://github.com/YunoHost-Apps/piped_ynh/pull/200): Upgrade sources
[22:48:13]
<Yunohost Git/Infra notifications> [piped_ynh] orhtej2 deleted branch ci-auto-update-sources-250418
[22:57:43]
<Yunohost Git/Infra notifications> [nextcloud_ynh] tio-trom opened [issue #798](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/798): Many errors after the latest update
[23:03:35]
<Yunohost Git/Infra notifications> [freshrss_ynh] michaelachrisco [commented](https://github.com/YunoHost-Apps/freshrss_ynh/issues/183#issuecomment-2814175179) on [issue #183](https://github.com/YunoHost-Apps/freshrss_ynh/issues/183) 1.24 upgrade breaks app (FreshRSS thinks it was just installed, without any data): Just had the same thing happen to me. Happened after an update and had to reset. The backup also didnt work (error pops ...
[23:04:50]
<Yunohost Git/Infra notifications> [nextcloud_ynh] tio-trom edited [issue #798](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/798): Many errors after the latest update
[23:08:38]
<Yunohost Git/Infra notifications> [nextcloud_ynh] tio-trom edited [issue #798](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/798): Many errors after the latest update
[23:25:57]
<Yunohost Git/Infra notifications> [nextcloud_ynh] rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/798#issuecomment-2814195158) on [issue #798](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/798) Many errors after the latest update: There is different issues here...
* But all are know issues except a new one I just have discover, the ROW_FORMAT=Dynam...