[11:30:34]
<Aleks (he/him/il/lui)> too many wishlist proposals x___x
[11:31:23]
<Aleks (he/him/il/lui)> https://i.kym-cdn.com/photos/images/original/000/439/825/7f8.gif
[11:44:24]
<lapineige> well, proposals are still proposals ^^
[11:44:44]
<lapineige> if we get tired we have a nice list to pick something to package 😂
[11:46:07]
<lapineige> Oh I see Piped… eric_G your old repo isn't here anymore, what was the reason of its deletion ?
(is it too hard to package so I shouldn't try ? 😅)
[11:57:54]
<Yunohost Git/Infra notifications> [apps] @alexAubin edited [pull request #1826](https://github.com/YunoHost/apps/pull/1826): Update app levels according to CI results
[11:57:54]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @retiolus [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/620#issuecomment-1784083067) on [issue #620](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/620) Cant open public shared links: That was the issue, the problem is that the app name "files_reader" doesnt correspond to the display name "Reader (OCR)...
[11:57:55]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @retiolus closed [issue #620](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/620): Cant open public shared links
[11:58:35]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to update_app_levels: Update apps.toml : simplytranslate back to level 7 ([ac8ea300](https://github.com/YunoHost/apps/commit/ac8ea300717018cb83da302210d0c8c8eb93ba5a))
[12:30:51]
<Yunohost Git/Infra notifications> [apps] @Thovi98 opened [pull request #1831](https://github.com/YunoHost/apps/pull/1831): Add Reiverr to catalog
[12:32:21]
<Yunohost Git/Infra notifications> [apps] @alexAubin [commented](https://github.com/YunoHost/apps/pull/1831#discussion_r1375428710) on pull request #1831 Add Reiverr to catalog: suggestion [remotestorage]
[12:34:22]
<Yunohost Git/Infra notifications> [apps] @Thovi98 [commented](https://github.com/YunoHost/apps/pull/1831#discussion_r1375429038) on pull request #1831 Add Reiverr to catalog: Yeaah thanks
[13:14:12]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1831](https://github.com/YunoHost/apps/pull/1831#pullrequestreview-1702959265) Add Reiverr to catalog
[13:14:22]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 3 commits to master ([46ccb9ab0c7f...a8071972dba1](https://github.com/YunoHost/apps/compare/46ccb9ab0c7f...a8071972dba1))
[13:14:22]
<Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1831](https://github.com/YunoHost/apps/pull/1831): Add Reiverr to catalog
[13:14:30]
<Yunohost Git/Infra notifications> [apps/master] Add Reiverr to catalog - Thomas
[13:14:32]
<Yunohost Git/Infra notifications> [apps/master] delete the one (how did it come? 😅) - Thomas
[13:14:36]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1831 from Thovi98/master Add Reiverr to catalog - eric_G
[14:06:10]
<lapineige> Aleks (he/him/il/lui): thanks for the commits on Yologen 🙂
Now the thing I did not anticipate : that my developement version would diverge from the "official" codebase, so it's making it harder to merge changes 😅 (for instance I'm still displaying the software version).
---
For all : the fixes I merged in yunohost repo uploaded to yologen.lapineige.fr 🙂
[14:07:44]
<Aleks (he/him/il/lui)> meh yeah i have a commit which i'm about to push which reworks the structure of the form because we really need to have some clear logic regarding the different blocks ... and i renamed / reworked some of the fields and it's gonna break the generation of the actual files which i didn't have a look at yet
[14:11:11]
<eric_G> > <@Alekswag:matrix.org> too many wishlist proposals x___x
We just need an automated merge process 😶🌫️
[14:16:21]
<lapineige> > <@Alekswag:matrix.org> meh yeah i have a commit which i'm about to push which reworks the structure of the form because we really need to have some clear logic regarding the different blocks ... and i renamed / reworked some of the fields and it's gonna break the generation of the actual files which i didn't have a look at yet
> because we really need to have some clear logic regarding the different blocks ...
I had struggle with that ^^, in particular because it was a WIP with more stuff added all the time.
I took a look at it a few days ago and I was not convinced… in particular because when packaging I'm not doing stuff in the order the form uses.
> and i renamed / reworked some of the fields and it's gonna break the generation of the actual files which i didn't have a look at yet
I can do that 🙂
[14:17:40]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited review [pull request #1799](https://github.com/YunoHost/apps/pull/1799#pullrequestreview-1702968897): Add a Yunohost App Generator (alias Yologen)
[14:17:41]
<Yunohost Git/Infra notifications> [apps] @ericgaspar [commented](https://github.com/YunoHost/apps/pull/1799#discussion_r1375445765) on pull request #1799 Add a Yunohost App Generator (alias Yologen): suggestion #### App file generated with YoloGen, the YunoHost app generator, version {{ parameters[GENERATOR_VERSION...
[14:17:41]
<Yunohost Git/Infra notifications> [apps] @ericgaspar [commented](https://github.com/YunoHost/apps/pull/1799#discussion_r1375445828) on pull request #1799 Add a Yunohost App Generator (alias Yologen): suggestion #### App file generated with YoloGen, the YunoHost app generator, version {{ parameters[GENERATOR_VERSION...
[14:17:41]
<Yunohost Git/Infra notifications> [apps] @ericgaspar [commented](https://github.com/YunoHost/apps/pull/1799#discussion_r1375445515) on pull request #1799 Add a Yunohost App Generator (alias Yologen): suggestion #### App file generated with YoloGen, the YunoHost app generator, version {{ parameters[GENERATOR_VERSION...
[14:17:42]
<Yunohost Git/Infra notifications> [apps] @ericgaspar [commented](https://github.com/YunoHost/apps/pull/1799#discussion_r1375445892) on pull request #1799 Add a Yunohost App Generator (alias Yologen): suggestion #### App file generated with YoloGen, the YunoHost app generator, version {{ parameters[GENERATOR_VERSION...
[14:26:45]
<Aleks (he/him/il/lui)> i won't have time to dig further today but i see some further improvements to work on such as :
- probably tweaking the Form class mechanism such that we don't have to write `{{ foo.label }} {{ foo() }} <br>` all over the place ... ideally only calling `{{ foo() }}` would render the form item with the label directly ...
- Ideally we should be able to call e.g. `{{ GeneralInfos.render_form() | safe }}` and that's it from the template, so as much form logic is handled on the python side (dunno if that's realistic but we'll see)
- we should render the "help" text of the option (which currently is named `"title"` i guess), which should also be handled by just calling `{{ foo() }}` somehow
- there are some bits of horrendous JS 😅 like the `showForm` with a wall of `if type == foo then form = "foob"` and can probably be factorized once I understand what it's all about
- similar stuff in the python code with wall of `if` that can be factorized ... and spooky `global`s 🎃
[14:28:28]
<Aleks (he/him/il/lui)> - i don't know if it is that much important to be able to see / download individual files from the web interface, i didnt have time to think a lot about it but it seems to result in a lot of technical complexity for something that may not be that much important ... or maybe i missed something, i need to dig deeper
[14:30:50]
<lapineige> > probably tweaking the Form class mechanism such that we don't have to write {{ foo.label }} {{ foo() }} <br> all over the place ... ideally only calling {{ foo() }} would render the form item with the label directly ...
Wouldn't it be overkill to make such a change for so little gain ? 😅
(also cometimes you needs both to apply some custom CSS directly)
[14:31:14]
<lapineige> > Ideally we should be able to call e.g. {{ GeneralInfos.render_form() | safe }} and that's it from the template, so as much form logic is handled on the python side (dunno if that's realistic but we'll see)
I don't get this one
[14:32:48]
<eric_G> For YoloGen, the NGINX config could be adapted according to the techo used PHP, NODE, GO...
[14:32:52]
<lapineige> > we should render the "help" text of the option (which currently is named "title" i guess), which should also be handled by just calling {{ foo() }} somehow
I'm wondering about that.
I believed "title" is read be screenreaders, and shown on over.
I was thinking about another "tutorial text", displayed on purpose, for settings requiring it. Maybe behind a dropdown to avoid too much visual complexity
[14:33:12]
<lapineige> > <@ericg:matrix.org> For YoloGen, the NGINX config could be adapted according to the techo used PHP, NODE, GO...
What do you mean ? (I've never done that)
[14:33:51]
<lapineige> > there are some bits of horrendous JS 😅 like the showForm with a wall of if type == foo then form = "foob" and can probably be factorized once I understand what it's all about
Yeah I would love that, it was painfull to write 😂, but I've no idea (there are some special cases)
[14:35:01]
<lapineige> > similar stuff in the python code with wall of if that can be factorized ... and spooky globals 🎃
The global thing comes from the fact that I cannot transfert walls of text (template content) from the JS side (using an XHR)… or I least I didn't find a better workaround during several days 😅… I saved my mental health ^^
[14:36:52]
<lapineige> > i don't know if it is that much important to be able to see / download individual files from the web interface, i didnt have time to think a lot about it but it seems to result in a lot of technical complexity for something that may not be that much important ... or maybe i missed something, i need to dig deeper
For now the technical complexity isn't a big deal as it's… done 🙂.
I think it's good to have a quite quick feedback loop when you make a change.
*(also it helps when the zip download is broken… 😅)*
I would prefer a nicer mechanism, such as displaying under each option a "diff" of what's changed, or at least the end result, for teaching purposes… but that's a lot of work and complexity.
[14:37:41]
<lapineige> ---
By the way if anyone wanna work with me on some improvements, we could make a call so I could help you to dig into the code faster 🙂
[14:37:58]
<lapineige> ---
By the way if anyone wanna work with me on some improvements on Yologen, we could make a call so I could help you to dig into the code faster 🙂
[14:38:14]
<Aleks (he/him/il/lui)> > > probably tweaking the Form class mechanism such that we don't have to write {{ foo.label }} {{ foo() }} <br> all over the place ... ideally only calling {{ foo() }} would render the form item with the label directly ...
>
> Wouldn't it be overkill to make such a change for so little gain ? 😅
> (also cometimes you needs both to apply some custom CSS directly)
eeer yeah but i think there is both css defined on the python side and some defined on the html template side, and overall it's just cumbersome to write and it will even more become cumbersome once we start adding title/"help" strings in the mix which as far as i've seen so far are not displayed nor shown on hover (didn't check everything, not 100% sure), so the idea is that adding/maintaining the whole field-rendering logic should be as painless as possible, and ultimately if the html template only consists of calling `{{ foo() }}` for every field in the form block, we could write something like `{{ block.render_all_fields() }}` and not having to touch the html at all everytime we tweak fields in the python side ... Dunno if that's doable or not, but ultimately my point is that we should have separation of concerns : either we handle the rendering logic from the python side, either we handle it from the html side, but having a little bit of both is hell (i should read more about flask-form thing, there are probably stuff to help us with all of this)
[14:38:49]
<lapineige> > title/"help" strings in the mix which as far as i've seen so far are not displayed nor shown on hover
They should !
[14:39:28]
<lapineige> > i should read more about flask-form thing, there are probably stuff to help us with all of this
I hope so !
[14:39:58]
<lapineige> > and ultimately if the html template only consists of calling {{ foo() }} for every field in the form block, we could write something like {{ block.render_all_fields() }}
There were cases where doing everything on the python side failed… ☹️
[14:41:07]
<lapineige> And I think most of that "by each option CSS code" could be removed by factorizing stuff in proper CSS class and better option that my crappy CSS. So most of it would disappear anyway, and maintenance should be a lot easier
[14:43:31]
<lapineige> > <@Alekswag:matrix.org> meh yeah i have a commit which i'm about to push which reworks the structure of the form because we really need to have some clear logic regarding the different blocks ... and i renamed / reworked some of the fields and it's gonna break the generation of the actual files which i didn't have a look at yet
Wow it's so hard to review such a code block in github or whatever… without seeing the end result 😆
[14:44:32]
<Yunohost Git/Infra notifications> [apps] @lapineige [commented](https://github.com/YunoHost/apps/pull/1799#discussion_r1375450767) on pull request #1799 Add a Yunohost App Generator (alias Yologen): Fix YunoHost typo
[14:44:39]
<Yunohost Git/Infra notifications> [apps] @lapineige deleted a [comment](https://github.com/YunoHost/apps/pull/1799#discussion_r1375450767) on pull request #1799 Add a Yunohost App Generator (alias Yologen): Fix YunoHost typo
[15:04:11]
<eric_G> any idea of the value expected here: https://github.com/YunoHost-Apps/proxigram_ynh/blob/6601cfdf774db5d49be128f13e56d89548cf9494/conf/.env#L17
[15:36:48]
<eric_G> ⭐️Sunday Challenge⭐️: Debug this code that is supposed to disable Mail app before upgrading Nextcloud. It is not working according to the number of user having an Nextcloud failed upgrade.
https://github.com/YunoHost-Apps/nextcloud_ynh/blob/bc59f53dbb91aa089bbade0bac6afa55dec4e6f3/scripts/upgrade#L269-L271
[16:01:50]
<tituspijean> I guess it's to save Instagram posts in the local feed, so I understand why there's the "don't set it too high" comment.
[16:01:59]
<tituspijean> Can they share their upgrade log, to see if that command is actually run?
[16:10:47]
<eric_G> > <@titus:pijean.ovh> Can they share their upgrade log, to see if that command is actually run?
https://paste.yunohost.org/raw/pijosegifo
[16:11:23]
<eric_G> from https://forum.yunohost.org/t/echec-mise-a-jour-nextcloud-26-0-3-ynh1-a-27-1-2-ynh3/26931/5
[16:11:45]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @tituspijean pushed 1 commit to testing: Implement phpflags and advertise command ([a37ef189](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/a37ef189fb3b8f701e47272e71ef43e0da4b8625))
[16:12:05]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @tituspijean pushed 1 commit to testing: Fix comments in upgrade ([27630282](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/2763028257eb6e45f9d963712b4f453a37f07ea5))
[16:13:05]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [comment](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/a37ef189fb3b8f701e47272e71ef43e0da4b8625#commitcomment-131182878) on commit a37ef18: Whats the use in the install script, out of curiosity ?
[16:14:25]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @tituspijean [comment](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/a37ef189fb3b8f701e47272e71ef43e0da4b8625#commitcomment-131182942) on commit a37ef18: None, its only here to set the setting. I thought symmetry was better. We could directly create the setting without declaring phpflags if thats found to be better.
[16:17:12]
<tituspijean> > <@ericg:matrix.org> https://paste.yunohost.org/raw/pijosegifo
As far as I can see, the Mail app is disabled, Nextcloud is upgraded then the upgrade script tries to enable Mail app, though it is not up-to-date (`sudo -u nextcloud php8.2 --define apc.enable_cli=1 occ --no-interaction --no-ansi app:enable mail
2023-10-29 15:21:06,856: DEBUG - App "Mail" cannot be installed because it is not compatible with this version of the server.`)
[16:17:34]
<tituspijean> Should there be a command to upgrade the Mail app?
[16:17:57]
<tituspijean> (or any other app actually)
[16:21:54]
<tituspijean> found `php occ app:update --all`
[16:24:05]
<tituspijean> erf sorry, it's already in the upgrade script for v2
[17:21:34]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @autra [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/617#issuecomment-1784174381) on [issue #617](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/617) Upgrade failed due to PHP version not compatible: I needed to disable he application phone sync (ocsms) (not maintained any more and doesnt supports php8.2), but apart ...
[17:31:55]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 73 commits to v2_PostgreSQL ([5383c04b714e...7b7657cc2733](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/5383c04b714e...7b7657cc2733))
[17:32:00]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Implement phpflags and advertise command - tituspijean
[17:32:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Fix comments in upgrade - tituspijean
[17:32:09]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Merge branch testing into v2_PostgreSQL - Éric Gaspar
[17:32:09]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#issuecomment-1784177046) on [issue #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597) V2 postgresql: testme
[17:32:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#issuecomment-1784177058) on [issue #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597) V2 postgresql: Alrighty
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/10688/badge)](h...
[17:33:10]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2_PostgreSQL: Update nginx.conf ([a0864f25](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/a0864f25b0d7a35dca9244e319d6c22562b2e27c))
[17:34:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to testing: Update nginx.conf ([19cb6388](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/19cb6388fc184c750f43d544ecd8ab30e3a36e4b))
[17:34:58]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to testing: Update DESCRIPTION.md ([bcbdb177](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/bcbdb177a6bac4ae9939b80fb90ab6313cef1cad))
[17:36:09]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to testing: Update install ([aab1e0b1](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/aab1e0b13a0f6f34ec238a1c62934fffb5abbdc8))
[17:37:02]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to testing: Update upgrade ([d7d24bd6](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/d7d24bd648f6b1bd5226db53925fa9b79800c8a1))
[17:37:05]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 5 commits to v2_PostgreSQL ([a0864f25b0d7...025a04018a58](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/a0864f25b0d7...025a04018a58))
[17:37:10]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Update install - Éric Gaspar
[17:37:14]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Update upgrade - Éric Gaspar
[17:37:19]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Merge branch testing into v2_PostgreSQL - Éric Gaspar
[17:38:23]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to testing: Update nginx.conf ([91c059ac](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/91c059acd73241f238e5ab34196311f3f6b78b51))
[17:38:39]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 2 commits to v2_PostgreSQL ([025a04018a58...1f9e0b8e2327](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/025a04018a58...1f9e0b8e2327))
[17:38:43]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Update nginx.conf - Éric Gaspar
[17:38:49]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Merge branch testing into v2_PostgreSQL - Éric Gaspar
[17:39:56]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2_PostgreSQL: Update _common.sh ([374819cb](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/374819cb1266de62d69c824200c8aec6e47cad9c))
[18:18:30]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to add-to-wishlist-passkey-support: Add Passkey Support to wishlist ([91855295](https://github.com/YunoHost/apps/commit/91855295f8714bfbf385c5e1536ee1635d177a5f))
[18:18:30]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #1832](https://github.com/YunoHost/apps/pull/1832): Add Passkey Support to wishlist
[18:25:14]
<eric_G> ^ euh...
[18:26:32]
<lapineige> > <@titus:pijean.ovh> As far as I can see, the Mail app is disabled, Nextcloud is upgraded then the upgrade script tries to enable Mail app, though it is not up-to-date (`sudo -u nextcloud php8.2 --define apc.enable_cli=1 occ --no-interaction --no-ansi app:enable mail
> 2023-10-29 15:21:06,856: DEBUG - App "Mail" cannot be installed because it is not compatible with this version of the server.`)
so a quick workaround is to tell them to upgrade their apps first ?
[18:28:23]
<eric_G> we are exprecting the upgrade script to handle that.
[18:28:49]
<eric_G> ...with little success sometime
[18:47:05]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2_PostgreSQL: Update nginx.conf ([1413ddf4](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/1413ddf49287f8d7a3961be79846fd162e89d152))
[18:58:56]
<Thomas> Does the readme updater bot still works?
[19:29:13]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch add-to-wishlist-feedbin
[19:29:14]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #1833](https://github.com/YunoHost/apps/pull/1833): Add Feedbin to wishlist
[19:29:14]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to add-to-wishlist-feedbin: Add Feedbin to wishlist ([ee1928f8](https://github.com/YunoHost/apps/commit/ee1928f80ad92ffe06a87d88ad9d1d22451e3ab6))
[19:40:37]
<Yunohost Git/Infra notifications> [apps] @Tagadda [commented](https://github.com/YunoHost/apps/pull/1827#discussion_r1375500830) on pull request #1827 Add Twake.app to wishlist: suggestion description = "Collaboration platform to improve organizational productivity"
[19:42:17]
<Yunohost Git/Infra notifications> [apps] @Tagadda [commented](https://github.com/YunoHost/apps/pull/1828#discussion_r1375501077) on pull request #1828 Add Exo platform to wishlist: suggestion description = "Modern intranet, social network, community management platform, collaboration platform and ...
[19:43:11]
<Yunohost Git/Infra notifications> [apps] @Tagadda [commented](https://github.com/YunoHost/apps/pull/1829#discussion_r1375501167) on pull request #1829 Add Automatisch to wishlist: suggestion description = "Workflow automation without coding"
[19:43:45]
<Yunohost Git/Infra notifications> [apps] @Tagadda [commented](https://github.com/YunoHost/apps/pull/1830#discussion_r1375501211) on pull request #1830 Add znc to wishlist: suggestion description = "IRC Bouncer"
[19:44:41]
<Yunohost Git/Infra notifications> [apps] @Tagadda [commented](https://github.com/YunoHost/apps/pull/1833#discussion_r1375501297) on pull request #1833 Add Feedbin to wishlist: suggestion description = "Simple, fast and nice looking RSS reader."
[19:44:44]
<Yunohost Git/Infra notifications> [apps] @Tagadda edited a [comment](https://github.com/YunoHost/apps/pull/1833#discussion_r1375501297) on pull request #1833 Add Feedbin to wishlist: suggestion description = "Simple, fast and nice looking RSS reader"
[19:55:06]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to add-to-wishlist-feedbin: Update wishlist.toml Co-authored-by: Tagada <36127788+Tagadda@users.noreply.github.com> ([a0326fd4](https://github.com/YunoHost/apps/commit/a0326fd428c4e87a72aca58c10d273ebda35b80e))
[19:55:10]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1833](https://github.com/YunoHost/apps/pull/1833#pullrequestreview-1703027993) Add Feedbin to wishlist
[19:55:34]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to add-to-wishlist-znc: Update wishlist.toml Co-authored-by: Tagada <36127788+Tagadda@users.noreply.github.com> ([86108538](https://github.com/YunoHost/apps/commit/8610853894ef594e647f65464f29d473c769e932))
[19:56:19]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1830](https://github.com/YunoHost/apps/pull/1830#pullrequestreview-1703028062) Add znc to wishlist
[19:56:19]
<Yunohost Git/Infra notifications> [apps] @Tagadda merged [pull request #1833](https://github.com/YunoHost/apps/pull/1833): Add Feedbin to wishlist
[19:56:19]
<Yunohost Git/Infra notifications> [apps] @Tagadda pushed 3 commits to master ([a8071972dba1...49e6c95df2a6](https://github.com/YunoHost/apps/compare/a8071972dba1...49e6c95df2a6))
[19:56:20]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to add-to-wishlist-automatisch: Update wishlist.toml Co-authored-by: Tagada <36127788+Tagadda@users.noreply.github.com> ([04f45566](https://github.com/YunoHost/apps/commit/04f455660799168561b2c93907462b70c423a7b9))
[19:56:20]
<Yunohost Git/Infra notifications> [apps] @Tagadda merged [pull request #1830](https://github.com/YunoHost/apps/pull/1830): Add znc to wishlist
[19:56:20]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1829](https://github.com/YunoHost/apps/pull/1829#pullrequestreview-1703028138) Add Automatisch to wishlist
[19:56:20]
<Yunohost Git/Infra notifications> [apps/master] Update wishlist.toml Co-authored-by: Tagada <36127788+Tagadda@users.noreply.github.com> - eric_G
[19:56:20]
<Yunohost Git/Infra notifications> [apps] @Tagadda pushed 3 commits to master ([49e6c95df2a6...7daedced209f](https://github.com/YunoHost/apps/compare/49e6c95df2a6...7daedced209f))
[19:56:20]
<Yunohost Git/Infra notifications> [apps/master] Add znc to wishlist - yunohost-bot
[19:56:20]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1833 from YunoHost/add-to-wishlist-feedbin Add Feedbin to wishlist - Tagada
[19:56:20]
<Yunohost Git/Infra notifications> [apps] @Tagadda deleted branch add-to-wishlist-feedbin
[19:56:20]
<Yunohost Git/Infra notifications> [apps/master] Add Feedbin to wishlist - yunohost-bot
[19:56:25]
<Yunohost Git/Infra notifications> [apps/master] Update wishlist.toml Co-authored-by: Tagada <36127788+Tagadda@users.noreply.github.com> - eric_G
[19:56:26]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1830 from YunoHost/add-to-wishlist-znc Add znc to wishlist - Tagada
[19:56:38]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to add-to-wishlist-twake-app: Update wishlist.toml Co-authored-by: Tagada <36127788+Tagadda@users.noreply.github.com> ([86c52236](https://github.com/YunoHost/apps/commit/86c522361a498b7bcd619740d963821631ce4f6f))
[19:56:45]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1827](https://github.com/YunoHost/apps/pull/1827#pullrequestreview-1703028199) Add Twake.app to wishlist
[19:57:09]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1825](https://github.com/YunoHost/apps/pull/1825#pullrequestreview-1703028229) Add LocalAI to wishlist
[19:57:10]
<Yunohost Git/Infra notifications> [apps] @Tagadda merged [pull request #1829](https://github.com/YunoHost/apps/pull/1829): Add Automatisch to wishlist
[19:57:10]
<Yunohost Git/Infra notifications> [apps] @Tagadda pushed 3 commits to master ([7daedced209f...d851f067263b](https://github.com/YunoHost/apps/compare/7daedced209f...d851f067263b))
[19:57:11]
<Yunohost Git/Infra notifications> [apps] @Tagadda deleted branch add-to-wishlist-automatisch
[19:57:14]
<Yunohost Git/Infra notifications> [apps/master] Add Automatisch to wishlist - yunohost-bot
[19:57:15]
<Yunohost Git/Infra notifications> [apps/master] Update wishlist.toml Co-authored-by: Tagada <36127788+Tagadda@users.noreply.github.com> - eric_G
[19:57:19]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1829 from YunoHost/add-to-wishlist-automatisch Add Automatisch to wishlist - Tagada
[19:57:32]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1824](https://github.com/YunoHost/apps/pull/1824#pullrequestreview-1703028278) Add PrivateGPT to wishlist
[19:57:37]
<Yunohost Git/Infra notifications> [apps] @Tagadda pushed 3 commits to master ([d851f067263b...f90314c1cd66](https://github.com/YunoHost/apps/compare/d851f067263b...f90314c1cd66))
[19:57:37]
<Yunohost Git/Infra notifications> [apps] @Tagadda merged [pull request #1827](https://github.com/YunoHost/apps/pull/1827): Add Twake.app to wishlist
[19:57:38]
<Yunohost Git/Infra notifications> [apps] @Tagadda deleted branch add-to-wishlist-twake-app
[19:57:41]
<Yunohost Git/Infra notifications> [apps/master] Add Twake.app to wishlist - yunohost-bot
[19:57:45]
<Yunohost Git/Infra notifications> [apps/master] Update wishlist.toml Co-authored-by: Tagada <36127788+Tagadda@users.noreply.github.com> - eric_G
[19:57:49]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1827 from YunoHost/add-to-wishlist-twake-app Add Twake.app to wishlist - Tagada
[19:57:58]
<Yunohost Git/Infra notifications> [apps] @Tagadda pushed 2 commits to master ([f90314c1cd66...05ae2c6935d2](https://github.com/YunoHost/apps/compare/f90314c1cd66...05ae2c6935d2))
[19:57:58]
<Yunohost Git/Infra notifications> [apps] @Tagadda merged [pull request #1824](https://github.com/YunoHost/apps/pull/1824): Add PrivateGPT to wishlist
[19:58:00]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1816](https://github.com/YunoHost/apps/pull/1816#pullrequestreview-1703028339) Add tryton to wishlist
[19:58:03]
<Yunohost Git/Infra notifications> [apps] @Tagadda deleted branch add-to-wishlist-privategpt
[19:58:06]
<Yunohost Git/Infra notifications> [apps/master] Add PrivateGPT to wishlist - yunohost-bot
[19:58:09]
<Yunohost Git/Infra notifications> [apps] @Tagadda merged [pull request #1825](https://github.com/YunoHost/apps/pull/1825): Add LocalAI to wishlist
[19:58:09]
<Yunohost Git/Infra notifications> [apps] @Tagadda pushed 2 commits to master ([05ae2c6935d2...fafdb16fcc86](https://github.com/YunoHost/apps/compare/05ae2c6935d2...fafdb16fcc86))
[19:58:10]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1824 from YunoHost/add-to-wishlist-privategpt Add PrivateGPT to wishlist - Tagada
[19:58:21]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1825 from YunoHost/add-to-wishlist-localai Add LocalAI to wishlist - Tagada
[19:58:22]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to add-to-wishlist-exo-platform: Update wishlist.toml Co-authored-by: Tagada <36127788+Tagadda@users.noreply.github.com> ([9ae10709](https://github.com/YunoHost/apps/commit/9ae107097f6635cef9b67fbeca4342f516f70756))
[19:58:29]
<Yunohost Git/Infra notifications> [apps] @Tagadda merged [pull request #1816](https://github.com/YunoHost/apps/pull/1816): Add tryton to wishlist
[19:58:30]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1828](https://github.com/YunoHost/apps/pull/1828#pullrequestreview-1703028395) Add Exo platform to wishlist
[19:58:36]
<Yunohost Git/Infra notifications> [apps] @Tagadda deleted branch add-to-wishlist-tryton
[19:58:43]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1816 from YunoHost/add-to-wishlist-tryton Add tryton to wishlist - Tagada
[19:59:02]
<Yunohost Git/Infra notifications> [apps] @Tagadda merged [pull request #1828](https://github.com/YunoHost/apps/pull/1828): Add Exo platform to wishlist
[19:59:05]
<Yunohost Git/Infra notifications> [apps] @Tagadda pushed 3 commits to master ([f38e2479c181...ee498a7c9cc1](https://github.com/YunoHost/apps/compare/f38e2479c181...ee498a7c9cc1))
[19:59:05]
<Yunohost Git/Infra notifications> [apps] @Tagadda deleted branch add-to-wishlist-exo-platform
[19:59:09]
<Yunohost Git/Infra notifications> [apps/master] Add Exo platform to wishlist - yunohost-bot
[19:59:10]
<Yunohost Git/Infra notifications> [apps/master] Update wishlist.toml Co-authored-by: Tagada <36127788+Tagadda@users.noreply.github.com> - eric_G
[19:59:13]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1828 from YunoHost/add-to-wishlist-exo-platform Add Exo platform to wishlist - Tagada
[20:19:53]
<lapineige> > <@ericg:matrix.org> we are exprecting the upgrade script to handle that.
yeah I mean before it gets ready
[20:50:37]
<orhtej2> wasn't `purge data dir' checkbox released yet?
[20:50:39]
<orhtej2> in webadmin that is
[20:51:14]
<eric_G> it is
[22:03:13]
<orhtej2> > also, if `sudo -u www-data ls some_crappy_path` gives permission denied nginx is not going to serve that
proper permissions are `$app:www-data 750`
[22:03:57]
<orhtej2> proper order of testing fix is first `git push`, then `yunohost app upgrade` x_x
[22:43:56]
<Yunohost Git/Infra notifications> [apps] @orhtej2 opened [pull request #1834](https://github.com/YunoHost/apps/pull/1834): Rename SyncStorage-rs logo.
[22:47:20]
<Yunohost Git/Infra notifications> [apps] @Tagadda approved [pull request #1834](https://github.com/YunoHost/apps/pull/1834#pullrequestreview-1703052296) Rename SyncStorage-rs logo.
[22:47:27]
<Yunohost Git/Infra notifications> [apps] @Tagadda merged [pull request #1834](https://github.com/YunoHost/apps/pull/1834): Rename SyncStorage-rs logo.
[22:47:28]
<Yunohost Git/Infra notifications> [apps] @Tagadda pushed 2 commits to master ([ee498a7c9cc1...a922efea36cc](https://github.com/YunoHost/apps/compare/ee498a7c9cc1...a922efea36cc))
[22:47:33]
<Yunohost Git/Infra notifications> [apps/master] Rename SyncStorage-rs logo. - orhtej2
[22:47:37]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1834 from orhtej2/sync Rename SyncStorage-rs logo. - Tagada