Wednesday, August 16, 2023
apps@conference.yunohost.org
August
Mon Tue Wed Thu Fri Sat Sun
  1
2
3
4
5
6
7
8
9 10
11
12
13
14
15
16
17
18
19
20
21
22 23 24
25
26
27
28
29
30
31
     
             

[07:55:37] <Yunohost Git/Infra notifications> App z-push goes down from level 8 to 4 in job [#17849](https://ci-apps.yunohost.org/ci/job/17849)
[08:41:24] <Yunohost Git/Infra notifications> [my_webapp_ynh] @stilobique edited a [comment](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/119#discussion_r1295573309) on pull request #119 Setup 404 error code: True, set with a default value and only a setup available with the config panel 👍
[08:41:24] <Yunohost Git/Infra notifications> [my_webapp_ynh] @stilobique [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/119#discussion_r1295573309) on pull request #119 Setup 404 error code: True, set with a default value and only a setup available with the config panel 👍
[10:12:37] <Yunohost Git/Infra notifications> App osticket failed all tests in job [#17853](https://ci-apps.yunohost.org/ci/job/17853) :(
[13:30:53] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean opened [issue #120](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/120): Displayed database password is incorrect
[14:23:56] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean assigned tituspijean on [issue #120](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/120): Displayed database password is incorrect
[14:25:37] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean created new branch fix-db
[14:25:39] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean pushed 2 commits to fix-db ([494db3a91eec^...347546ab04f9](https://github.com/YunoHost-Apps/my_webapp_ynh/compare/494db3a91eec^...347546ab04f9))
[14:25:44] <Yunohost Git/Infra notifications> [my_webapp_ynh/fix-db] Actually use the generated db_pwd - tituspijean
[14:25:44] <Yunohost Git/Infra notifications> [my_webapp_ynh] @yunohost-bot pushed 1 commit to fix-db: Auto-update README ([043c417a](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/043c417a4504a9d45bc8b059bb40f098109d8c76))
[14:25:45] <Yunohost Git/Infra notifications> [my_webapp_ynh/fix-db] Bump package version and display bugfix - tituspijean
[14:26:10] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean opened [pull request #121](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/121): Fix db_pwd
[14:27:47] <Aleks (he/him/il/lui)> hmpf and also we really need to address the issue of "the mysql info are shown in the postinstall even when not enabling mysql
[14:28:22] <Aleks (he/him/il/lui)> i was thinking about maybe some `[if foobar]` `[/if]` syntax but that's custom and will be a pain to implement
[14:28:46] <Yunohost Git/Infra notifications> [my_webapp_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/121#issuecomment-1680724946) on [issue #121](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/121) Fix db_pwd: Fingers crossed
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/8869/bad...
[14:28:46] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/121#issuecomment-1680724872) on [issue #121](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/121) Fix db_pwd: testme
[14:28:47] <Aleks (he/him/il/lui)> but now that a think again about it we could just go with regular jinja syntax (just on top of the existing `__FOOBAR__` thing for var)
[14:28:55] <Aleks (he/him/il/lui)> like `{% if foobar %}` `{% endif %}`
[14:28:57] <tituspijean> > <@Alekswag:matrix.org> i was thinking about maybe some `[if foobar]` `[/if]` syntax but that's custom and will be a pain to implement

why not jinja? 🙃
[14:29:14] <Aleks (he/him/il/lui)> it's pretty easy to implement, we just call jinja before replacing variables like we already do, etc
[14:31:20] <Aleks (he/him/il/lui)> so like everything is "simple" in the nominal case, we have the usual `__FOOBAR__` syntax, but we can use jinja syntax for advanced stuff
[14:31:29] <Aleks (he/him/il/lui)> that's kinda frankensteinish but hmf
[14:32:30] <Aleks (he/him/il/lui)> (definitely not as frankensteinish as the accounting stuff i'm working on for $dayjob right now 👀)
[15:12:40] <Yunohost Git/Infra notifications> [my_webapp_ynh] @alexAubin approved [pull request #121](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/121#pullrequestreview-1580863409) Fix db_pwd
[15:13:32] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean edited [pull request #121](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/121): Fix db_pwd
[15:13:33] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean edited [pull request #121](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/121): Fix db_pwd
[15:14:01] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean merged [pull request #121](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/121): Fix db_pwd
[15:14:01] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean pushed 4 commits to testing ([03cc1fbd9530...9e25b1d0a0c4](https://github.com/YunoHost-Apps/my_webapp_ynh/compare/03cc1fbd9530...9e25b1d0a0c4))
[15:14:05] <Yunohost Git/Infra notifications> [my_webapp_ynh/testing] Bump package version and display bugfix - tituspijean
[15:14:09] <Yunohost Git/Infra notifications> [my_webapp_ynh/testing] Auto-update README - yunohost-bot
[15:14:10] <Yunohost Git/Infra notifications> [my_webapp_ynh/testing] Merge pull request #121 from YunoHost-Apps/fix-db Fix db_pwd - tituspijean
[15:15:10] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/121#issuecomment-1680804435) on [issue #121](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/121) Fix db_pwd: (dont delete the branch just yet)
[15:15:52] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean pushed 5 commits to enh-postgresql ([1d83be3e56ab...cf1e857017b5](https://github.com/YunoHost-Apps/my_webapp_ynh/compare/1d83be3e56ab...cf1e857017b5))
[15:15:55] <Yunohost Git/Infra notifications> [my_webapp_ynh/enh-postgresql] Auto-update README - yunohost-bot
[15:16:02] <Yunohost Git/Infra notifications> [my_webapp_ynh/enh-postgresql] Merge pull request #121 from YunoHost-Apps/fix-db Fix db_pwd - tituspijean
[15:16:04] <Yunohost Git/Infra notifications> [my_webapp_ynh/enh-postgresql] Merge branch testing into enh-postgresql - tituspijean
[15:28:57] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean edited [pull request #118](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/118): Implement postgresql database choice
[15:29:17] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean approved [pull request #118](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/118#pullrequestreview-1580897025) Implement postgresql database choice: Successfully tested with Joomla
[15:31:13] <Yunohost Git/Infra notifications> [my_webapp_ynh] @alexAubin approved [pull request #118](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/118#pullrequestreview-1580901004) Implement postgresql database choice
[15:38:07] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean pushed 1 commit to enh-postgresql: Improve POST_INSTALL documentation for databases ([e3599fe6](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/e3599fe60c2ae982f8cc9539bb9063ed45e73770))
[15:38:10] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean dismissed review [pull request #118](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/118#pullrequestreview-1580901004): Implement postgresql database choice
[15:38:10] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean dismissed review [pull request #118](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/118#pullrequestreview-1580897025): Implement postgresql database choice
[15:40:49] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean pushed 10 commits to testing ([9e25b1d0a0c4...4ebc7e27d5de](https://github.com/YunoHost-Apps/my_webapp_ynh/compare/9e25b1d0a0c4...4ebc7e27d5de))
[15:40:49] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean merged [pull request #118](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/118): Implement postgresql database choice
[15:40:53] <Yunohost Git/Infra notifications> [my_webapp_ynh/testing] Merge branch testing into enh-postgresql - tituspijean
[15:40:54] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean deleted branch enh-postgresql
[15:40:57] <Yunohost Git/Infra notifications> [my_webapp_ynh/testing] Improve POST_INSTALL documentation for databases - tituspijean
[15:41:01] <Yunohost Git/Infra notifications> [my_webapp_ynh/testing] Merge pull request #118 from YunoHost-Apps/enh-postgresql Implement postgresql database choice - tituspijean
[15:43:38] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean opened [pull request #122](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/122): Improving databases: password fix and type selection
[15:45:45] <Aleks (he/him/il/lui)> (i'll be working on supporting jinja syntax in notifications right after i give up on my frankensteinaccounting)
[15:47:04] <Yunohost Git/Infra notifications> App jellyfin goes down from level 8 to 6 in job [#17864](https://ci-apps.yunohost.org/ci/job/17864)
[15:53:12] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean pushed 1 commit to testing: Fix phpversion question type ([ce931807](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/ce931807718db9b4e5679185bb04b55344286cea))
[15:54:17] <tituspijean> What do you think about granting SSH permission to my_webapp along with SFTP? I think it makes sense.
[15:54:37] <tituspijean> I'll document the `yunohost app shell` command too
[15:56:42] <Aleks (he/him/il/lui)> > <@titus:pijean.ovh> What do you think about granting SSH permission to my_webapp along with SFTP? I think it makes sense.

hmmm maybe, can you elaborate ?
[15:56:56] <Aleks (he/him/il/lui)> you mean for people to upload stuff with SCP for example ?
[15:57:16] <tituspijean> and SCP too indeed
[15:57:33] <tituspijean> I'm thinking about apps needing to run some php commands
[15:58:02] <Aleks (he/him/il/lui)> not sure to understand the relation between running PHP commands and the SSH permission 😅
[15:58:08] <Aleks (he/him/il/lui)> ah you mean for maintenance
[15:58:29] <tituspijean> to allow admins to log in as `my_webapp` and run commands, depending on the installed app
[15:58:30] <tituspijean> yup
[15:59:11] <Aleks (he/him/il/lui)> what about a custom action in the config panel like we did for kiwix during the camp where you input a command and it's ran using `yunohost app shell` 🤓
[15:59:40] <tituspijean> *touché*
[16:00:00] <tituspijean> I might implement that for Flarum's extension installation too
[16:00:39] <Aleks (he/him/il/lui)> noice
[16:01:52] <Aleks (he/him/il/lui)> (regarding my webapp, i can also see this being useful for people that want to run a git pull)
[16:02:17] <Aleks (he/him/il/lui)> (we could even hide/show a "Git pull" button automagically if we detect a `.git` 😅)
[16:07:07] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean pushed 1 commit to testing: Enhance login documentation ([7a2eba44](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/7a2eba44db560235b91a800d898bb6cabe47c817))
[16:07:41] <Aleks (he/him/il/lui)> > <@Alekswag:matrix.org> (we could even hide/show a "Git pull" button automagically if we detect a `.git` 😅)

and even display "You are X commit behind" in an alertbox 🤓
[16:10:00] <tituspijean> _moments later, they ended up inadvertently coding a whole git suite_
[16:10:11] <Aleks (he/him/il/lui)> oopsies
[16:10:18] <tituspijean> *I did it again* 🎶
[16:11:26] <Aleks (he/him/il/lui)> "The date is August 19th, 2023, and Config Panel just attained consciousness and the US government wants to use YunoHost config panel to manage their nuclear arsenal"
[16:14:38] <tituspijean> *I'm afraid I cannot `option phpversion has 'choices' but has type 'string', use 'select' instead`, Dave* 🔴
[16:14:59] <Aleks (he/him/il/lui)> xD
[16:18:52] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean pushed 1 commit to testing: Enhance FR login documentation ([5a6756a8](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/5a6756a8c49f38a569d199a365463bb2cd82a52f))
[16:18:59] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/122#issuecomment-1680907779) on [issue #122](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/122) Improving databases: password fix and type selection: testme
[16:19:01] <Yunohost Git/Infra notifications> [my_webapp_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/122#issuecomment-1680907822) on [issue #122](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/122) Improving databases: password fix and type selection: :sunflower:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/8871/badge)](...
[17:29:36] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean created new branch jinja-doc
[17:29:36] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean pushed 1 commit to jinja-doc: Jinja template for the POST_INSTALL doc ([b6f86b2f](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/b6f86b2f4eaba45f94d42898773f00401a8da52f))
[17:31:52] <Aleks (he/him/il/lui)> <whisper>should be {% if database != 'none' %}</whisper>
[17:33:00] <tituspijean> yup haha
[17:33:05] <tituspijean> I've just realized that
[17:35:16] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean pushed 1 commit to jinja-doc: Fix jinja template Co-authored-by: Alexandre Aubin <alex.aubin@mailoo.org> ([0ba29961](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/0ba29961e5cda69eb9ef9b46416dde5b8357a14a))
[17:35:44] <Aleks (he/him/il/lui)> u wot m8
[17:35:54] <Aleks (he/him/il/lui)> how do you co-author stuff without a suggestion xD
[17:36:18] <tituspijean> Manually, in the commit message 😅
[17:36:30] <Aleks (he/him/il/lui)> ah xD
[17:36:45] <tituspijean> It workz
[17:36:49] <tituspijean> ```
yunohost app install https://github.com/YunoHost-Apps/my_webapp_ynh/tree/jinja-doc -f
Choose the domain where this app should be installed [dev.local | ynhdev.pijean.ovh | cryptpad.ynhdev.pijean.ovh | element.ynhdev.pijean.ovh | synapse.ynhde
v.pijean.ovh]: ynhdev.pijean.ovh
Choose the URL path (after the domain) where this app should be installed: /site
Do you need a SFTP access? [yes | no]: no
Choose an administration password for this app: **********
Who should have access to this app? (This can later be changed) [all_users | visitors | admins]: visitors
Choose a PHP version you want to use for your app [none | 7.4 | 8.0 | 8.1 | 8.2]: none
Do you need a database? [none | mysql | postgresql]: none
Info: Installing my_webapp...
Info: Provisionning system_user...
Info: Provisionning install_dir...
Info: Provisionning permissions...
Info: Provisionning apt...
Info: [+++.................] > Validating installation parameters...
Info: [###++...............] > Storing installation settings...
Info: [#####++++...........] > Configuring NGINX web server...
Info: [#########+..........] > Configuring system user...
Info: [####################] > Installation of my_webapp completed
Success! Installation completed
==========
The admin documentation below contains information on how to connect using SFTP to edit the website content.
==========
```
[17:39:36] <tituspijean> Double workz
```
yunohost app install https://github.com/YunoHost-Apps/my_webapp_ynh/tree/jinja-doc -f
Choose the domain where this app should be installed [dev.local | ynhdev.pijean.ovh | cryptpad.ynhdev.pijean.ovh | element.ynhdev.pijean.ovh | synapse.ynhde
v.pijean.ovh]: ynhdev.pijean.ovh
Choose the URL path (after the domain) where this app should be installed: /sites
Do you need a SFTP access? [yes | no]: no
Choose an administration password for this app: **********
Who should have access to this app? (This can later be changed) [all_users | visitors | admins]: visitors
Choose a PHP version you want to use for your app [none | 7.4 | 8.0 | 8.1 | 8.2]: none
Do you need a database? [none | mysql | postgresql]: postgresql
Info: Installing my_webapp...
Info: Provisionning system_user...
Info: Provisionning install_dir...
Info: Provisionning permissions...
Info: Provisionning apt...
Info: [+++.................] > Validating installation parameters...
Info: [###++...............] > Storing installation settings...
Info: [#####++++...........] > Creating a database...
Info: [#########+++........] > Configuring NGINX web server...
Info: [############++......] > Configuring system user...
Info: [####################] > Installation of my_webapp__2 completed
Success! Installation completed
==========
Here are information to log into the database:

- Type: postgresql
- Database user: my_webapp__2
- Database name: my_webapp__2
- Password: XXX



The admin documentation below contains information on how to connect using SFTP to edit the website content.
==========
```
[18:49:07] <Yunohost Git/Infra notifications> App misskey goes down from level 8 to 1 in job [#17868](https://ci-apps.yunohost.org/ci/job/17868)
[20:48:58] <Yunohost Git/Infra notifications> [my_webapp_ynh] @tituspijean pushed 1 commit to testing: Add password in tests ([fd0fe510](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/fd0fe51048670a7c61d5c28909b9c3abc43c23f6))