[01:07:18]
<Yunohost Git/Infra notifications> App ethercalc failed all tests in job [#20935](https://ci-apps.yunohost.org/ci/job/20935) :(
[12:41:05]
<azahto>
For the record, `try_files $uri $uri/ __PATH__/index.php;` instead of `try_files $uri $uri/ index.php;`
[12:41:45]
<azahto> I think it now works properly so I will have to prepare a proper package to submit to the official list of packages :D
[12:42:30]
<azahto> Hi there! I am slowly getting to a working state for HotCRP_ynh: https://github.com/Azzaare/hotcrp_ynh/tree/newskin
I've been stuck for a while on (I assume) an NGINX issue. If I install the package, it works fine and I can access the `index.php`. However, if I try to do anything else (such as opening a link to another file from index.php) I am redirect to the SSO of yunohost.
```
116.82.41.27 - - [27/Nov/2023:05:53:29 +0000] "GET /crp6/newaccount HTTP/2.0" 302 138 "https://vps-a5f7d75d.vps.ovh.net/crp6/" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/119.0.0.0 Safari/537.36 Edg/119.0.0.0"
116.82.41.27 - - [27/Nov/2023:05:53:30 +0000] "GET /yunohost/sso/?r=aHR0cHM6Ly92cHMtYTVmN2Q3NWQudnBzLm92aC5uZXRpbmRleC5waHA= HTTP/2.0" 200 3122 "https://vps-a5f7d75d.vps.ovh.net/crp6/" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/119.0.0.0 Safari/537.36 Edg/119.0.0.0"
```
I can't really get what are the things I need to do to force NGINX to not redirect to the SSO. Any idea?
[12:42:46]
<azahto> As far as I understand the systemd thingy, I don't think I need for a webapp that is run through php, am I right?
[12:42:47]
<azahto> If the server is reboot, the app should be available anyway (in the sense that nginx and php are the systemd services)?
[12:42:48]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @Crashone [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/625#issuecomment-1827441861) on [issue #625](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/625) 502 Gateway Time-out: Thank you for this workaround.
[12:42:50]
<azahto> Thanks Mateusz Szymański
[12:42:52]
<Mateusz Szymański> > If the server is reboot, the app should be available anyway (in the sense that nginx and php are the systemd services)?
It should not need an additional service, no
[12:42:55]
<azahto> OK, I will. So master is only for release ?
One more question (sorry), regarding LICENSE, as I package an existing source, I can use my own OSS LICENSE I suppose (was planning to use MIT).
[12:42:57]
<Mateusz Szymański> > If the server is reboot, the app should be available anyway (in the sense that nginx and php are the systemd services)?
yes
[12:42:57]
<Yunohost Git/Infra notifications> App question2answer stays at level 2 in job [#20945](https://ci-apps.yunohost.org/ci/job/20945)
[12:43:04]
<lapineige> Yes master is the released version
[12:43:14]
<Mateusz Szymański> everyone seems to use GNU Affero General Public License v3.0
[12:43:46]
<azahto> Fair enough, so that will be a task for tomorrow! Thank you all for your help
[12:43:46]
<azahto> > <@orhtej2:matrix.org> if you feel super fancy you can run the generator yourself: https://github.com/YunoHost/apps/tree/master/tools/README-generator
I did enough of fancy for me in the last 2 weeks, maybe for my next package! (which is already in the pipeline)
[12:43:46]
<Aleks (he/him/il/lui)> no need to remove it. yunohost-bot will probably overwrite it later yes
[12:43:50]
<Aleks (he/him/il/lui)> or you can run the autoreadme generator manually i suppose but it's a pain in the ass to redo it manually everytime
[12:43:50]
<azahto> Well, the readme from the original app is pretty short, most of it is about configuration, which is automatically handled with the ynh package!
OK, so I think I reached a point where I can publish the app. Should I try to move the to the yunohost org now?
[12:43:50]
<Aleks (he/him/il/lui)> yeah if the point is just having something that works then no need to care about the README 😉
[12:43:50]
<azahto> I see. I might go with the flow then :D
[12:44:07]
<azahto> Here we go with another question! Based on the ouput of package-linter, should I remove the readme and let the yunohost bot create it later?
```
! It looks like the README was not generated automatically by https://github.com/YunoHost/apps/tree/master/tools/README-generator. Note that nowadays you are not suppose to edit README.md, the yunohost bot will usually automatically update it if your app is hosted in the YunoHost-Apps org ... or you can also generate it by running the README-generator yourself.
```
[12:44:07]
<Mateusz Szymański> > Well, the readme from the original app is pretty short, most of it is about configuration, which is automatically handled with the ynh package!
> OK, so I think I reached a point where I can publish the app. Should I try to move the to the yunohost org now?
I would advise to mark `master` and `testing` as protected branches with PR/MR required prior to that
[12:44:07]
<azahto> That was the plan, I would not take so much time to understand the packaging system here just to have one instance running on my server :D
[12:44:07]
<Aleks (he/him/il/lui)> assuming your repo is moved to the yunohost-apps org
[12:44:12]
<Mateusz Szymański> > That was the plan, I would not take so much time to understand the packaging system here just to have one instance running on my server :D
it was moved apparently already: https://github.com/YunoHost-Apps/mlmmj_ynh
[12:44:35]
<Mateusz Szymański> btw can you clone `master` branch to `testing`? This is a standard YNH repo layout
[12:44:43]
<Mateusz Szymański> > Here we go with another question! Based on the ouput of package-linter, should I remove the readme and let the yunohost bot create it later?
> ```
> ! It looks like the README was not generated automatically by https://github.com/YunoHost/apps/tree/master/tools/README-generator. Note that nowadays you are not suppose to edit README.md, the yunohost bot will usually automatically update it if your app is hosted in the YunoHost-Apps org ... or you can also generate it by running the README-generator yourself.
> ```
if you feel super fancy you can run the generator yourself: https://github.com/YunoHost/apps/tree/master/tools/README-generator
[12:44:49]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @Crashone [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/625#issuecomment-1827441861) on [issue #625](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/625) 502 Gateway Time-out: Thank you for this solution.
[12:44:49]
<Mateusz Szymański> > OK, I will. So master is only for release ?
> One more question (sorry), regarding LICENSE, as I package an existing source, I can use my own OSS LICENSE I suppose (was planning to use MIT).
I never understood that part TBH and just went with copying some random license from some random package
[12:45:00]
<Mateusz Szymański> and create basic `tests.toml`, otherwise CI will fail and the app will be marked as broken
[13:31:37]
<crashone> Hi. What is the right way to advert maintainers that an app is not up-to-date ? Should I open a bug report ?
[13:47:06]
<Aleks (he/him/il/lui)> yes, on the app repo
[13:47:43]
<Aleks (he/him/il/lui)> and possibly check that a pull request doesnt already exist to update the app
[13:50:59]
<Aleks (he/him/il/lui)> tellmewhy.mp3
[13:51:02]
<Aleks (he/him/il/lui)> zzz ci-apps-bookworm getting even more stucked than before T.T
[14:06:52]
<crashone> > <@Alekswag:matrix.org> yes, on the app repo
like that ? https://github.com/YunoHost-Apps/dolibarr_ynh/issues/106
[15:46:27]
<Yunohost Git/Infra notifications> [test_apps] @alexAubin deleted branch cp-fix
[15:46:27]
<Yunohost Git/Infra notifications> [test_apps] @alexAubin pushed 2 commits to master ([3120381ec167...6e1a3ebb2190](https://github.com/YunoHost/test_apps/compare/3120381ec167...6e1a3ebb2190))
[15:46:27]
<Yunohost Git/Infra notifications> [test_apps] @alexAubin merged [pull request #10](https://github.com/YunoHost/test_apps/pull/10): config_app_ynh: adapt to config panel changes
[16:09:12]
<Wylel> Is anyone maintaining Firefly III?
[16:16:34]
<Aleks (he/him/il/lui)> no
[16:25:21]
<eric_G> I am.
[16:27:20]
<eric_G> no, I am kidding 😶🌫️
[16:37:45]
<Yunohost Git/Infra notifications> App dolibarr goes down from level 8 to 6 in job [#20953](https://ci-apps.yunohost.org/ci/job/20953)
[17:07:58]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch add-to-wishlist-windmill
[17:07:59]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to add-to-wishlist-windmill: Add Windmill to wishlist ([639bb844](https://github.com/YunoHost/apps/commit/639bb844f02bc0d75309213b45945a8ca8af8855))
[17:07:59]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #1886](https://github.com/YunoHost/apps/pull/1886): Add Windmill to wishlist
[17:13:04]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @leonarf opened [issue #132](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/132): problem with html relative path using ../ and application configured with a subpath
[18:32:22]
<Yunohost Git/Infra notifications> [apps] @Tagadda approved [pull request #1886](https://github.com/YunoHost/apps/pull/1886#pullrequestreview-1751105062) Add Windmill to wishlist
[20:10:06]
<Yunohost Git/Infra notifications> [apps] @alexAubin created new branch alexAubin-patch-1
[20:10:07]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to alexAubin-patch-1: not_working -> notworking ([78cc8033](https://github.com/YunoHost/apps/commit/78cc8033118e5de6122c7d16743d09d239ef714c))
[20:28:17]
<Yunohost Git/Infra notifications> [apps] @alexAubin opened [pull request #1887](https://github.com/YunoHost/apps/pull/1887): not_working -> notworking
[20:29:18]
<Yunohost Git/Infra notifications> [apps] @Tagadda approved [pull request #1887](https://github.com/YunoHost/apps/pull/1887#pullrequestreview-1751300408) not_working -> notworking
[20:31:12]
<Aleks (he/him/il/lui)> i misclicked "create a new branch" instead of comitting to master and now for some reason the PR is stuck at "Checking for ability to merge automatically…" x_x
[21:35:07]
<azahto> > <@orhtej2:matrix.org> it was moved apparently already: https://github.com/YunoHost-Apps/mlmmj_ynh
Not my package though!
[21:35:53]
<Yunohost Git/Infra notifications> [apps] @alexAubin deleted branch alexAubin-patch-1
[21:35:53]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to master: not_working -> notworking (#1887) ([de8bd299](https://github.com/YunoHost/apps/commit/de8bd299fa315d027a13f8f5ef868e27d1399753))
[21:35:54]
<Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #1887](https://github.com/YunoHost/apps/pull/1887): not_working -> notworking