[00:27:40]
<lapineige> > Allowing user to choose between dB engines seems like an overkill, just use postgresql if support is golden
In reply to @orhtej2:circledsquareroot.ovh
Allowing user to choose between dB engines seems like an overkill, just use postgresql if support is golden
Yes, except if they add some important specific feature to the app, just pick one of them
[00:28:06]
<lapineige> Good to see someone packaging this software ! :)
[07:57:49]
<PapaDragon> Well, I just saw this in https://codeberg.org/streams/streams/src/branch/dev/install/INSTALL.txt : "Case-insensitive search is not supported on postgres. This is not harmful, but postgres nodes should probably not be used as directory servers due to this limitation."
[07:58:10]
<PapaDragon> Mysql it shall be, then.
[09:02:50]
<Yunohost Git/Infra notifications> [apps] @oufmilo opened [pull request #1702](https://github.com/YunoHost/apps/pull/1702): Add firefish to catalog
[09:11:59]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited review [pull request #1702](https://github.com/YunoHost/apps/pull/1702#pullrequestreview-1560597233): Add firefish to catalog
[09:12:00]
<Yunohost Git/Infra notifications> [apps] @ericgaspar [commented](https://github.com/YunoHost/apps/pull/1702#discussion_r1282890061) on pull request #1702 Add firefish to catalog: suggestion potential_alternative_to = [ "X", "Mastodon", "Pleroma", "Misskey" ]
[09:15:44]
<oufmilo[m]> > <@yunohostinfra:matrix.org> [apps] @ericgaspar [commented](https://github.com/YunoHost/apps/pull/1702#discussion_r1282890061) on pull request #1702 Add firefish to catalog: suggestion potential_alternative_to = [ "X", "Mastodon", "Pleroma", "Misskey" ]
Threads ? ^^
[09:30:46]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1702](https://github.com/YunoHost/apps/pull/1702#pullrequestreview-1560638020) Add firefish to catalog
[09:30:46]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited review [pull request #1702](https://github.com/YunoHost/apps/pull/1702#pullrequestreview-1560638020): Add firefish to catalog
[09:30:46]
<Yunohost Git/Infra notifications> [apps] @ericgaspar [commented](https://github.com/YunoHost/apps/pull/1702#discussion_r1282917166) on pull request #1702 Add firefish to catalog: suggestion potential_alternative_to = [ "X", "Threads", "Mastodon", "Pleroma", "Misskey" ]
[09:30:54]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 4 commits to master ([6e8a7acee3f3...8c6aba65a852](https://github.com/YunoHost/apps/compare/6e8a7acee3f3...8c6aba65a852))
[09:30:54]
<Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1702](https://github.com/YunoHost/apps/pull/1702): Add firefish to catalog
[09:30:58]
<Yunohost Git/Infra notifications> [apps/master] Update apps.toml Co-authored-by: Éric Gaspar <46165813+ericgaspar@users.noreply.github.com> - oufmilo
[09:31:08]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1702 from oufmilo/patch-1 Add firefish to catalog - Éric Gaspar
[09:31:08]
<Yunohost Git/Infra notifications> [apps/master] Update apps.toml - Éric Gaspar
[09:50:52]
<orhtej2> > <PapaDragon> I gave try to v2. The install worked but the log file got flooded with errors like this:
> - Every ten seconds - https://privatebin.io/?36b34e71f785b212#EF2qujmyopmxqiGRZVLidu7K1KyyzF5f4j9d1QbFAf85
> - Every ten minutes - https://privatebin.io/?ffb1ed8fb8631b44#5hBzzAnzhvMzrQVrq2Se3wPBPHcnrt3n9vpBBM3R5ucf
> If there's a simple way to get rid of those PHP warnings, I'll gladly switch to v2 🙂
what log is this? I installed from your `format2` branch and everything seems to be working just fine (other than me having to create user first, why are there open registrations enabled by default and why first user gets admin rights?)
[12:22:04]
<PapaDragon> In the Streams Admin section you can enable logging, which is stored in /var/www/streams/php.log. You have to manually enter the log file name "php.log". The logs I pasted are those which appear when logging is eabled and set to debug.
I first asked the app main developer about the messages I got using format v2, he told me that PHP warning logging is disabled in Streams and that somehow the messages were most likely be generated by a YunoHost component.
[12:26:57]
<PapaDragon> As for the registration settings, I'll have a look. LDAP authentication is not fully usable for the moment (Streams ldapauth plugin could need some tweaking before being really usable). The admin is not the first registered user, but the user that will use the email of the YunoHost user chosen as admin prior to installation.
[12:40:22]
<orhtej2> > <PapaDragon> As for the registration settings, I'll have a look. LDAP authentication is not fully usable for the moment (Streams ldapauth plugin could need some tweaking before being really usable). The admin is not the first registered user, but the user that will use the email of the YunoHost user chosen as admin prior to installation.
ah, true that I registered with the same email, nice
[13:11:04]
<eric_G> I see this
ⓘ manifest
```
- Error validating manifest using schema: in key install > domain
Additional properties are not allowed ('full_domain' was unexpected)
```
in
https://ci-apps-dev.yunohost.org/ci/job/8596
is `full_domain = true` not valide in `[install.domain]`?
[13:34:12]
<Tag> IIRC it's not used in the core. The "require a full domain" thing is deduced from the fact that there's no `path` asked in the manifest
[13:39:27]
<Tag> Oh, the tool to convert package from v1 to v2 add this properties to the manifest.toml ^^ (but it shouldn't)
https://github.com/YunoHost/apps/blob/master/tools/packaging_v2/convert_app_to_packaging_v2.py#L148-L149
[13:42:33]
<Yunohost Git/Infra notifications> [apps] @Tagadda created new branch Tagadda-patch-1
[13:42:36]
<Yunohost Git/Infra notifications> [apps] @Tagadda pushed 1 commit to Tagadda-patch-1: convert_app_to_packaging_v2.py: remove full_domain This properties is not used by the core. Its deduced from the fact ... ([920e9633](https://github.com/YunoHost/apps/commit/920e9633953f358eec5ca54df029acaf3e60636d))
[13:43:03]
<Yunohost Git/Infra notifications> [apps] @Tagadda opened [pull request #1703](https://github.com/YunoHost/apps/pull/1703): convert_app_to_packaging_v2.py: remove full_domain
[13:46:52]
<eric_G> Quick search on GitHub shows 72 apps using it. 😱
[15:28:03]
<Yunohost Git/Infra notifications> App funkwhale rises from level 6 to 8 in job [#17211](https://ci-apps.yunohost.org/ci/job/17211) !
[15:39:04]
<Aleks (he/him/il/lui)> > <@ericg:matrix.org> Quick search on GitHub shows 72 apps using it. 😱
Yeah we could probably write an autopatch for this like for http_host recently
[16:22:35]
<Yunohost Git/Infra notifications> [apps] @ericgaspar created new branch Twitter---X
[16:22:36]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to Twitter---X: Update apps.toml ([1b46f97e](https://github.com/YunoHost/apps/commit/1b46f97e5fa3ed94262815c77cd2e3b882d5b9a6))
[16:23:06]
<Yunohost Git/Infra notifications> [apps] @ericgaspar opened [pull request #1704](https://github.com/YunoHost/apps/pull/1704): Replace Twitter by X
[16:24:04]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1704](https://github.com/YunoHost/apps/pull/1704): Replace Twitter by X
[18:40:58]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 2 commits to master ([8c6aba65a852...bb84ce406702](https://github.com/YunoHost/apps/compare/8c6aba65a852...bb84ce406702))
[18:40:59]
<Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #1703](https://github.com/YunoHost/apps/pull/1703): convert_app_to_packaging_v2.py: remove full_domain
[18:40:59]
<Yunohost Git/Infra notifications> [apps] @alexAubin deleted branch Tagadda-patch-1
[18:41:00]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1703 from YunoHost/Tagadda-patch-1 convert_app_to_packaging_v2.py: remove full_domain - Alexandre Aubin
[18:41:02]
<Yunohost Git/Infra notifications> [apps/master] convert_app_to_packaging_v2.py: remove full_domain This properties is not used by the core. Its deduced from the fact ... - Tagada
[22:47:55]
<oufmilo[m]> Hi 👋
I’ve got a warning ⚠️ with readme but I don’t understand because I don’t modify it myself
https://ci-apps-dev.yunohost.org/ci/job/8592
If I remove readme yunohost-bot will reinstall them ?
[22:56:34]
<PapaDragon> > > <PapaDragon> I gave try to v2. The install worked but the log file got flooded with errors like this:
> > - Every ten seconds - https://privatebin.io/?36b34e71f785b212#EF2qujmyopmxqiGRZVLidu7K1KyyzF5f4j9d1QbFAf85
> > - Every ten minutes - https://privatebin.io/?ffb1ed8fb8631b44#5hBzzAnzhvMzrQVrq2Se3wPBPHcnrt3n9vpBBM3R5ucf
> > If there's a simple way to get rid of those PHP warnings, I'll gladly switch to v2 🙂
>
> what log is this? I installed from your `format2` branch and everything seems to be working just fine (other than me having to create user first, why are there open registrations enabled by default and why first user gets admin rights?)
My bad, the config file handling PHP logs had PHP warnings enabled. It's now corrected.
[22:57:58]
<PapaDragon> Also I switched to packaging format v2 on my repo (https://github.com/dragondaddy/streams_ynh/). Install, backup, remove, restore and upgrade script seem to work as expected.
[23:33:31]
<PapaDragon> Now I just need to find out what values I should in the [integration] section of manifest.toml. If anyone has an idea on how to proceed to find relevant values, I'd really appreciate
[23:36:22]
<PapaDragon> Same for fpm related values in script/install, these are things I'm really not familiar with
[23:51:42]
<PapaDragon> > Now I just need to find out what values I should in the [integration] section of manifest.toml. If anyone has an idea on how to proceed to find relevant values, I'd really appreciate
By values, I mean "disk" "ram.build" & "ram.runtime"