[07:54:29]
<isAAAc> > yakforms 7.x-1.1-rc1 Posted by theolem on 11 May 2021 `First Yakforms release! Including: New identity: revised HTML templates, removed remaining old ... external file from within the module. yakforms-7.x-1.1-rc1.tar.gz `
in addition, https://yakforms.org/pages/install.html speaks about yakforms modules , not more about framaforms modules
[07:54:53]
<isAAAc> (hi o/)
[14:23:38]
<Yunohost Git/Infra notifications> [apps] @tituspijean pushed 1 commit to add-to-wishlist-convos: Improve Convos description Co-authored-by: Tagada <36127788+Tagadda@users.noreply.github.com> ([fe38bb2a](https://github.com/YunoHost/apps/commit/fe38bb2aa6bd757cf430756e07bd6c68056e183b))
[14:23:41]
<Yunohost Git/Infra notifications> [apps] @tituspijean approved [pull request #1798](https://github.com/YunoHost/apps/pull/1798#pullrequestreview-1671385785) Add Convos to wishlist
[14:25:27]
<Yunohost Git/Infra notifications> [apps] @tituspijean edited [pull request #1798](https://github.com/YunoHost/apps/pull/1798): Add Convos to wishlist
[14:25:28]
<Yunohost Git/Infra notifications> [apps] @tituspijean edited [pull request #1798](https://github.com/YunoHost/apps/pull/1798): Add Convos to wishlist
[14:25:29]
<Yunohost Git/Infra notifications> [apps] @tituspijean edited [pull request #1798](https://github.com/YunoHost/apps/pull/1798): Add Convos to wishlist
[14:25:31]
<Yunohost Git/Infra notifications> [apps] @tituspijean pushed 3 commits to master ([f018129bae54...5d340983b283](https://github.com/YunoHost/apps/compare/f018129bae54...5d340983b283))
[14:25:32]
<Yunohost Git/Infra notifications> [apps] @tituspijean merged [pull request #1798](https://github.com/YunoHost/apps/pull/1798): Add Convos to wishlist
[14:25:36]
<Yunohost Git/Infra notifications> [apps/master] Add Convos to wishlist - yunohost-bot
[14:25:41]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1798 from YunoHost/add-to-wishlist-convos Add Convos to wishlist - tituspijean
[14:25:45]
<Yunohost Git/Infra notifications> [apps] @tituspijean deleted branch add-to-wishlist-convos
[14:25:48]
<Yunohost Git/Infra notifications> [apps] @tituspijean deleted branch add-to-wishlist-immich
[14:25:48]
<Yunohost Git/Infra notifications> [apps] @tituspijean deleted branch pairdrop
[17:38:20]
<Yunohost Git/Infra notifications> [wordpress_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/wordpress_ynh/pull/229#issuecomment-1758171090) on [issue #229](https://github.com/YunoHost-Apps/wordpress_ynh/pull/229) Allow specifying PHP version #223: May the CI gods be with you
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/...
[19:14:03]
<tituspijean> Is the value of `init_{perm}_permission` from manifest available in the scripts?
[19:15:04]
<tituspijean> looks like neither as $init_{perm}_permission or $perm
[20:20:21]
<Aleks (he/him/il/lui)> Eeh nope but it's used for provisionning the permission so you could check the list of permission for the corresponding app
[20:21:16]
<Aleks (he/him/il/lui)> I'm curious about the use case ?
[20:23:14]
<tituspijean> packaging v2 of dotclear2: https://github.com/YunoHost-Apps/dotclear2_ynh/pull/66
the LDAP config of the app has a hardcoded super_user variable that I want to be taken from the admin permission
[20:24:17]
<tituspijean> your suggestion to take the user from the permission seems fine thanks 🙂
'coz I'm already getting a "Using helper ynh_permission_update is deprecated when using packaging v2 ... It is replaced by: the permission resource" warning if I go the other way around
[20:30:03]
<tituspijean> And I think I have yet again created a monster with https://github.com/YunoHost-Apps/dotclear2_ynh/pull/70
[20:31:55]
<tituspijean> > so you could check the list of permission for the corresponding app
mmh wait how do I do that?
[20:37:04]
<tituspijean> oh a permission is also a debian group, might use that
[20:39:55]
<tituspijean> (not usable, ldap groups are not in /etc/groups or whatever)
[20:44:41]
<tituspijean> `yunohost user permission info "dotclear2.admin" --output-as json | jq -r '.allowed | .[]'` should do the trick. Testing.
[20:45:08]
<Tag> Don't you need a user list and not a group list ?
[20:45:21]
<tituspijean> I request a user for the init_admin_permission question
[20:45:43]
<Tag> the init_admin_permission is a group IIRC
[20:46:01]
<Tag> or can it be a user ?
[20:46:10]
<tituspijean> in my tests the manifest is not complaining if I put a user type
[20:47:04]
<Tag> 🤔 but it's a select in the webadmin ?
[20:47:05]
<Tag> oh wait
[20:47:22]
<tituspijean> CLI:
```
~/apps/dotclear2_ynh multiple-blogs # yunohost app install . -f
Choose the domain where this app should be installed [castopod.local | commento.local | conduitwsl.local | dendrite.local | dex-headscale.
local | dexoutline.local | element.local | headscale.local | hedgedoc.local | lemmywsl.local | minioconsole.local | minio.local | n8n.loca
l | nocodb.local | ntfy.local | outline.local | overseerr.local | peertube.local | pleroma.local | site.local | ... and 10 other available
options not shown]: wsl.local
Choose the URL path (after the domain) where this app should be installed: /dotclear2
Who should have access to this app? (This can later be changed) [all_users | visitors | admins]: visitors
init_admin_permission [titus | test]: titus
```
[20:47:37]
<Tag> ooooh there's a type attribute for user or groups :o
[20:47:40]
<tituspijean> it loses the automatic ask key, but it works
[20:48:08]
<tituspijean> dang I should cleanup my test domains
[20:49:35]
<Tag> ahah (I DMed you, don't know if you got it)
[20:51:05]
<tituspijean> nope (not yet, I guess, *just matrix things*)
[20:52:02]
<Tag> yeah, matrix things... I mean you didn't get the last one (from 26/09) either I think :x
[20:53:50]
<tituspijean> Can you try inviting me again into the dm room?
[20:55:25]
<Tag> L'utilisateur est déjà dans ce salon >\<
Let's create a new one I guess (si je trouve comment...)
[21:12:15]
<Yunohost Git/Infra notifications> [wordpress_ynh] @fflorent [commented](https://github.com/YunoHost-Apps/wordpress_ynh/pull/229#issuecomment-1758549673) on [issue #229](https://github.com/YunoHost-Apps/wordpress_ynh/pull/229) Allow specifying PHP version #223: testme
[21:12:16]
<Yunohost Git/Infra notifications> [wordpress_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/wordpress_ynh/pull/229#issuecomment-1758549868) on [issue #229](https://github.com/YunoHost-Apps/wordpress_ynh/pull/229) Allow specifying PHP version #223: :stuck_out_tongue_winking_eye:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/ap...
[22:41:08]
<Yunohost Git/Infra notifications> [wordpress_ynh] @alexAubin changes_requested [pull request #229](https://github.com/YunoHost-Apps/wordpress_ynh/pull/229#pullrequestreview-1672700396) Allow specifying PHP version #223: Personally, as expressed in the corresponding issue if i recall correctly, Im against this change This is against Yu...
[22:59:17]
<Yunohost Git/Infra notifications> [wordpress_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/wordpress_ynh/pull/229#discussion_r1355861857) on pull request #229 Allow specifying PHP version #223: Or i dunno, we should at least add an help key explaining that the default value is fine for regular use case and that...
[23:00:02]
<Yunohost Git/Infra notifications> [wordpress_ynh] @Tagadda [commented](https://github.com/YunoHost-Apps/wordpress_ynh/pull/229#issuecomment-1758660072) on [issue #229](https://github.com/YunoHost-Apps/wordpress_ynh/pull/229) Allow specifying PHP version #223: I think that we shouldnt ask PHP version when installing Wordpress. If we need to change the PHP version, I think the c...