[09:26:18]
<Yunohost Git/Infra notifications> [issues] @Thatoo opened [issue #2295](https://github.com/YunoHost/issues/issues/2295): UX/UI issue : Main domain helper cant be read with dark mode on
[09:26:18]
<Yunohost Git/Infra notifications> [issues] @Thatoo labeled :space_invader: bug on [issue #2295](https://github.com/YunoHost/issues/issues/2295): UX/UI issue : Main domain helper cant be read with dark mode on
[14:27:38]
<Yunohost Git/Infra notifications> [issues] @zamentur opened [issue #2296](https://github.com/YunoHost/issues/issues/2296): ynh_setup_source should not continue if previous patch failed but the last patch succeed
[14:27:38]
<Yunohost Git/Infra notifications> [issues] @zamentur labeled :space_invader: bug on [issue #2296](https://github.com/YunoHost/issues/issues/2296): ynh_setup_source should not continue if previous patch failed but the last patch succeed
[14:30:19]
<Yunohost Git/Infra notifications> [issues] @zamentur labeled :package: App management on [issue #2296](https://github.com/YunoHost/issues/issues/2296): ynh_setup_source should not continue if previous patch failed but the last patch succeed
[14:30:22]
<Yunohost Git/Infra notifications> [issues] @zamentur labeled :package: App packaging on [issue #2296](https://github.com/YunoHost/issues/issues/2296): ynh_setup_source should not continue if previous patch failed but the last patch succeed
[21:02:03]
<Yunohost Git/Infra notifications> [issues] @alexAubin [commented](https://github.com/YunoHost/issues/issues/1973#issuecomment-1839470924) on [issue #1973](https://github.com/YunoHost/issues/issues/1973) Magically keeping all custom-config panel settings during the conf upgrade via ynh_add_config without having to define all of them as yunohost settings: Yet another example today in https://github.com/YunoHost-Apps/fittrackee_ynh/pull/47 ...
[22:17:53]
<Yunohost Git/Infra notifications> [issues] @Tagadda labeled :maple_leaf: Web administration on [issue #2295](https://github.com/YunoHost/issues/issues/2295): UX/UI issue : Main domain helper cant be read with dark mode on
[22:17:53]
<Yunohost Git/Infra notifications> [issues] @Tagadda labeled :star2: ux on [issue #2295](https://github.com/YunoHost/issues/issues/2295): UX/UI issue : Main domain helper cant be read with dark mode on
[22:17:53]
<Yunohost Git/Infra notifications> [issues] @Tagadda labeled ♿ a11y on [issue #2295](https://github.com/YunoHost/issues/issues/2295): UX/UI issue : Main domain helper cant be read with dark mode on
[22:18:30]
<Yunohost Git/Infra notifications> [issues] @Tagadda labeled :globe_with_meridians: DNS on [issue #2293](https://github.com/YunoHost/issues/issues/2293): Unable to remove auto dns token because of mandatory input field
[22:18:30]
<Yunohost Git/Infra notifications> [issues] @Tagadda labeled :maple_leaf: Web administration on [issue #2293](https://github.com/YunoHost/issues/issues/2293): Unable to remove auto dns token because of mandatory input field
[22:18:31]
<Yunohost Git/Infra notifications> [issues] @Tagadda labeled Domain managment on [issue #2293](https://github.com/YunoHost/issues/issues/2293): Unable to remove auto dns token because of mandatory input field
[22:18:40]
<Yunohost Git/Infra notifications> [issues] @Tagadda labeled :package: App management on [issue #2294](https://github.com/YunoHost/issues/issues/2294): change_url script is not automagically regenerating Permission URL set in manifest.toml
[22:18:57]
<Yunohost Git/Infra notifications> [issues] @Tagadda labeled :lock: Certificate on [issue #2290](https://github.com/YunoHost/issues/issues/2290): Lets Encrypt button is disabled for subdomains
[22:18:57]
<Yunohost Git/Infra notifications> [issues] @Tagadda labeled Domain managment on [issue #2290](https://github.com/YunoHost/issues/issues/2290): Lets Encrypt button is disabled for subdomains
[23:16:11]
<Yunohost Git/Infra notifications> [issues] @zamentur [commented](https://github.com/YunoHost/issues/issues/1973#issuecomment-1839696812) on [issue #1973](https://github.com/YunoHost/issues/issues/1973) Magically keeping all custom-config panel settings during the conf upgrade via ynh_add_config without having to define all of them as yunohost settings: ## Solution 1: read values, upgrade the config and rewrite values yunohost app config --export --related-to="instal...
[23:17:09]
<Yunohost Git/Infra notifications> [yunohost] @zamentur [commented](https://github.com/YunoHost/yunohost/pull/1675#issuecomment-1839697743) on [issue #1675](https://github.com/YunoHost/yunohost/pull/1675) Magically keeping all custom-config panel settings during the conf upgrade via ynh_add_config without having to define all of them as yunohost settings: https://github.com/YunoHost/issues/issues/1973
[23:17:52]
<Yunohost Git/Infra notifications> [yunohost] @zamentur [commented](https://github.com/YunoHost/yunohost/pull/1675#issuecomment-1839697743) on [issue #1675](https://github.com/YunoHost/yunohost/pull/1675) Magically keeping all custom-config panel settings during the conf upgrade via ynh_add_config without having to define all of them as yunohost settings: Related https://github.com/YunoHost/issues/issues/1973
[23:18:00]
<Yunohost Git/Infra notifications> [yunohost] @zamentur [commented](https://github.com/YunoHost/yunohost/pull/1675#issuecomment-1839697743) on [issue #1675](https://github.com/YunoHost/yunohost/pull/1675) Magically keeping all custom-config panel settings during the conf upgrade via ynh_add_config without having to define all of them as yunohost settings: Related to https://github.com/YunoHost/issues/issues/1973
[23:18:51]
<Yunohost Git/Infra notifications> [issues] @zamentur [commented](https://github.com/YunoHost/issues/issues/1973#issuecomment-1839696812) on [issue #1973](https://github.com/YunoHost/issues/issues/1973) Magically keeping all custom-config panel settings during the conf upgrade via ynh_add_config without having to define all of them as yunohost settings: ## Solution 1: read values, upgrade the config and rewrite values yunohost app config --export --related-to="instal...
[23:19:20]
<Yunohost Git/Infra notifications> [yunohost] @zamentur [commented](https://github.com/YunoHost/yunohost/pull/1675#issuecomment-1839697743) on [issue #1675](https://github.com/YunoHost/yunohost/pull/1675) Magically keeping all custom-config panel settings during the conf upgrade via ynh_add_config without having to define all of them as yunohost settings: Related to https://github.com/YunoHost/issues/issues/1973
[23:24:37]
<Yunohost Git/Infra notifications> [issues] @zamentur [commented](https://github.com/YunoHost/issues/issues/1973#issuecomment-1839696812) on [issue #1973](https://github.com/YunoHost/issues/issues/1973) Magically keeping all custom-config panel settings during the conf upgrade via ynh_add_config without having to define all of them as yunohost settings: ## Solution 1: read values, upgrade the config and rewrite values yunohost app config --export --related-to="instal...
[23:26:54]
<Yunohost Git/Infra notifications> [issues] @zamentur [commented](https://github.com/YunoHost/issues/issues/1973#issuecomment-1839696812) on [issue #1973](https://github.com/YunoHost/issues/issues/1973) Magically keeping all custom-config panel settings during the conf upgrade via ynh_add_config without having to define all of them as yunohost settings: ## Solution 1: read values, upgrade the config and rewrite values yunohost app config --export --related-to="instal...
[23:28:31]
<Yunohost Git/Infra notifications> [issues] @zamentur [commented](https://github.com/YunoHost/issues/issues/1973#issuecomment-1839696812) on [issue #1973](https://github.com/YunoHost/issues/issues/1973) Magically keeping all custom-config panel settings during the conf upgrade via ynh_add_config without having to define all of them as yunohost settings: ## Solution 1: read values, upgrade the config and rewrite values yunohost app config --export --related-to="instal...
[23:30:02]
<Aleks (he/him/il/lui)> ogod my brain is melting trying to think about config panel and settings again
[23:30:07]
<Aleks (he/him/il/lui)> i'm not even sure to understand what's the issue anymore
[23:30:40]
<Aleks (he/him/il/lui)> especially since i discovered this line https://github.com/YunoHost/yunohost/blob/dev/helpers/config#L123 i'm like "uuuuh what's the issue then"
[23:31:46]
<Aleks (he/him/il/lui)> ljf:
> when we run yunohost app config --export --related-to="$install_dir/config/config.php" the config_panel.toml is the new config_panel, but the config.php is the old one...
uuuh is it ?
[23:32:52]
<Aleks (he/him/il/lui)> (the config_panel.toml is only overwritten after the upgrade here https://github.com/YunoHost/yunohost/blob/dev/src/app.py#L917 ?)
[23:37:09]
<Aleks (he/him/il/lui)> ah yes now i re-remember the issue
[23:37:24]
<Aleks (he/him/il/lui)> if the app config contains `foo: "bar"` yet is exposed via the config panel, it's gonna be overwritten by `ynh_add_config` unless we write `foo: __FOO__` and have `$foo` being a setting initialized with "bar"
[23:37:29]
<Aleks (he/him/il/lui)> #simple,basique