[09:20:16]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @mateMathieu closed issue #518: Cron not functioning after restore https://github.com/YunoHost-Apps/nextcloud_ynh/issues/518
[09:20:16]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @mateMathieu commented on issue #518 Cron not functioning after restore: Fixed by updating the instance. https://github.com/YunoHost-Apps/nextcloud_ynh/issues/518#issuecomment-1275854919
[12:42:38]
<nicofrand> Salut yalh76 ! Est-ce que je pourrais avoir le droit sur la branche testing de kresus_ynh ? Je voudrais force push dessus pour la rebase avec master
[12:42:42]
<nicofrand> j'en ai marre des conflits
[13:15:57]
<Aleks (he/him/il/lui)> nicofrand: est-ce que c'est à cause du README ? normalement on a réglé ça au meeting de la semaine dernière
[13:18:50]
<nicofrand> j'en sais rien mais testing devient tout le temps désynchronisé de master et c'est relou pour juste fusionner
[13:19:44]
<nicofrand> les hashs de commits sont différents etc
[13:20:03]
<nicofrand> si on pouvait juste avoir testing iso avec master et ensuite fusionner les branches dans testing puis testing dans master ce serait quand même + simple
[13:20:28]
<nicofrand> là j'ai corrigé testing et pour mettre à jour master il essaie d'inclure plein de trucs déjà fusionnés
[13:25:01]
<Aleks (he/him/il/lui)> bah du coup pour te faire le résumé de la discussion / conclusion, il a été décidé (je crois) :
- que pour la vaste majorité des apps, on oublie le fait de "forcément créer une testing et de faire des pulls requests qui ciblent testing, puis merger testing dans master"
- on remet master/main en branche par défaut
- les PR sont par défaut faite sur master/main
- pour certaines apps complexes ou importantes (par ex. Nextcloud) où c'est important d'avoir une testing intermédiaire, on garde la testing
[13:26:17]
<Aleks (he/him/il/lui)> et du coup il me semble que yalh était passé sur un peu tout les apps pour remettre master/main en branche par défaut, et du coup normalement la prochaine fois que tu fais une PR sur kresus, tu pars de master et donc y'aura pas d'histoire de conflit ?
[13:28:54]
<nicofrand> la MR est déjà faite et fusionnée dans testing
[13:29:06]
<nicofrand> 'fin bon, je peux la refaire sur master et basta OK
[13:29:36]
<Aleks (he/him/il/lui)> non c'est pas nécessaire, c'est "juste" que à la fin de ce "cycle" de PR / testing / merge dans master, y'aura plus de problème en utilisant le nouveau workflow
[13:29:54]
<nicofrand> bah, ce sera rapide et on qu'à supprimer testing et voilà
[13:29:55]
<Aleks (he/him/il/lui)> mais en tout cas si les conflits venaient du README normalement ça c'est de toute façon réglé aussi
[13:30:58]
<nicofrand> pas que, testign est en .6 et master en .7 je sais pas pourquoi
[13:30:58]
<Aleks (he/him/il/lui)> oui tu peux aussi direct recréer le truc sur master, comme tu le sens
[13:30:58]
<Aleks (he/him/il/lui)> tu parles de ~ynh7 ?
[13:32:19]
<Aleks (he/him/il/lui)> hmouai effectivement
[13:32:20]
<Aleks (he/him/il/lui)> sans doute un truc qui a été mergé dans master directement
[13:34:23]
<nicofrand> https://github.com/YunoHost-Apps/kresus_ynh/pull/142/files du coup je mets pas à jour les Readme c'est ça ?
[13:34:23]
<Aleks (he/him/il/lui)> (ceci dit c'est vrai j'y pense maintenant, l'avantage que y'avais à utiliser "testing" c'était que le bump de version était "centralisé" sur testing ... alors que là si deux branches modifient le numéro de version il est possible que le machin soit pas d'équerre blblbl ..)
[13:34:48]
<Aleks (he/him/il/lui)> le README est de toute manière généré et updaté automagiquement par yunohost-bot oui
[13:36:14]
<Aleks (he/him/il/lui)> and now that I read the README, I realize this looks like a pretty cool app i / we might want to use 👀
[13:38:55]
<Aleks (he/him/il/lui)> a day may come when I'll actually read carefully Yunohost's app catalog 👀
[14:01:51]
<nicofrand> :D
[16:50:21]
<yalh76> > <@Alekswag:matrix.org> a day may come when I'll actually read carefully Yunohost's app catalog 👀
There is too much apps to know them all ^^
[18:26:15]
<tituspijean> There are more than 400 now...it's actually still less than the number of Pokémon. 🙃
[18:26:39]
<tituspijean> > <@Alekswag:matrix.org> and now that I read the README, I realize this looks like a pretty cool app i / we might want to use 👀
It's a nice one indeed 👌
[18:41:51]
<yalh76> > Salut yalh76 ! Est-ce que je pourrais avoir le droit sur la branche testing de kresus_ynh ? Je voudrais force push dessus pour la rebase avec master
est ce qu'on pourrait se parler oralement ? j'ai trouvé le bug lié à https://github.com/YunoHost-Apps/kresus_ynh/issues/140, mais c'est trop chiant à expliquer par ecrit
[18:43:47]
<nicofrand> > <@yalh:matrix.yalh.net> est ce qu'on pourrait se parler oralement ? j'ai trouvé le bug lié à https://github.com/YunoHost-Apps/kresus_ynh/issues/140, mais c'est trop chiant à expliquer par ecrit
J'avoue ne pas être un grand fan des appels/visio :s. Mais c'est corrigé non ?
[18:44:22]
<nicofrand> C'est ce que j'ai fait tout à l'heure
[18:47:15]
<nicofrand> En gros t'avais rajouté une étape de build non nécessaire
[18:47:55]
<nicofrand> Et qui faisait planter l'installation (car les paquets npm nécessaires au build ne sont pas installés en mode production)
[18:48:35]
<nicofrand> J'ai testé l'upgrade et reproduit le bug, et réussi l'upgrade avec ma correction
[18:54:03]
<yalh76> https://aria.im/_matrix/media/v1/download/matrix.yalh.net/RwqALwZxBYmKGZZgVAkxeThE
[18:54:28]
<yalh76> c'est vraiment long à expliquer ...
[18:55:35]
<yalh76> nicofrand: en gros le bug est:
il a encore dans son dossier /var/www/kresus des fichiers sources antérieurs à la 0.18 de kresus
[18:56:14]
<yalh76> et la PR que j'ai faite ne remet les sources à zéro que si on vient d'un version strictement inférieure à la 0.18.1
[18:58:56]
<nicofrand> Logiquement c'était le but précédent, que j'avais réglé en supprimant le dossier build précédent justement
[18:59:22]
<nicofrand> Bug*
[18:59:22]
<yalh76> bon ben j'ai fini mon commentaire ...
[19:00:18]
<nicofrand> https://github.com/YunoHost-Apps/kresus_ynh/issues/134
[19:00:18]
<yalh76> oauip mais je pense qu'il a un fichier /var/www/kresus/webpack.config.js qui fout le bronx et qui vient d'un verison avant la 0.18.0
[19:01:22]
<yalh76> c'est entre autre à cause de ça que j'ai fait https://github.com/YunoHost/yunohost/pull/1509
[19:02:34]
<yalh76> aprés si y'a pas besoin de buildé `yarn build:prod ` , y'a plus de problémes.
Faut juste garder en tête que jusque là les fichiers supprimés dans kresus était encore présent dans kresus_ynh
[19:02:52]
<yalh76> tu peux refermer https://github.com/YunoHost-Apps/kresus_ynh/issues/140 si tu veux
[19:03:18]
<nicofrand> > <@yalh:matrix.yalh.net> oauip mais je pense qu'il a un fichier /var/www/kresus/webpack.config.js qui fout le bronx et qui vient d'un verison avant la 0.18.0
Ça ne devrait pas avoir d'importance à moins que le package.json ne soit pas non plus à jour
[19:03:18]
<nicofrand> On verra son retour
[19:04:04]
<nicofrand> Mais avec ton répertoire temporaire c'est réglé dans tous les cas a priori
[19:05:55]
<yalh76> ouaip sauf pour les gens qui ont eu une version < 0.18.1 , qui ont upgradé pour une des version 0.18.1 pour ensuite upgrader en version 0.18.1~ynh7 ^^
ça sera définitivement réglé quand il y aura une version 0.18.2
[19:06:11]
<nicofrand> Pour finir, le dockerfile nightly fait en effet un yarn build car il se base sur les sources nightly, tandis que les tags sont pré-buildés
[19:08:02]
<yalh76> > Pour finir, le dockerfile nightly fait en effet un yarn build car il se base sur les sources nightly, tandis que les tags sont pré-buildés
ouaip c'est ça que j'avais pas vu donc oui pas besoin du `yarn build:prod ` ^^ ça va économiser du temps de build :)
[19:08:08]
<nicofrand> > <@yalh:matrix.yalh.net> ouaip sauf pour les gens qui ont eu une version < 0.18.1 , qui ont upgradé pour une des version 0.18.1 pour ensuite upgrader en version 0.18.1~ynh7 ^^
> ça sera définitivement réglé quand il y aura une version 0.18.2
C'est pas pour demain a priori ^^
[19:09:32]
<yalh76> > C'est pas pour demain a priori ^^
sauf si j'ouvre une issue sur les problème de double authentification de Boursorama ^^
que j'ai pas encore eu le temps de trop regarder ^^
[19:09:40]
<nicofrand> > <@yalh:matrix.yalh.net> ouaip c'est ça que j'avais pas vu donc oui pas besoin du `yarn build:prod ` ^^ ça va économiser du temps de build :)
On devrait peut-être le préciser dans les tags en fait...
[19:10:03]
<nicofrand> > <@yalh:matrix.yalh.net> sauf si j'ouvre une issue sur les problème de double authentification de Boursorama ^^
> que j'ai pas encore eu le temps de trop regarder ^^
Ça dépend si c'est un souci woob ou Kresus !
[19:10:12]
<nicofrand> Mais on est un peu au ralenti là
[19:11:42]
<nicofrand> Pour Boursorama framasky l'utilise , il a pas remonté de souci de 2FA a priori
[19:13:38]
<yalh76> > On devrait peut-être le préciser dans les tags en fait...
disons que comme je sais pas trop comment le node/typescript ça fonctionne, l'analyse de ce qu'il faut faire pour installer le package c'est compliqué ^^
[19:13:38]
<nicofrand> Ou alors à woob directement
[19:13:39]
<nicofrand> > <@yalh:matrix.yalh.net> disons que comme je sais pas trop comment le node/typescript ça fonctionne, l'analyse de ce qu'il faut faire pour installer le package c'est compliqué ^^
En fait on documente l'installation depuis les sources ou npm mais pas depuis les tags (utilisés par npm ou le package arch)
[19:14:03]
<yalh76> > Mais on est un peu au ralenti là
je disais ça pour taquiner,
j'ai pas analysé le probléme juste que le 2FA boursorama fonctionne trés bien sur un serveur à la maison mais ne passe pas sur un serveur chez OVH ... Donc rien de bloquant
[19:14:43]
<yalh76> > En fait on documente l'installation depuis les sources ou npm mais pas depuis les tags (utilisés par npm ou le package arch)
pour ça y'a juste à dire au genre de lire le code d'install de kresus_ynh :p
[19:16:19]
<nicofrand> > <@yalh:matrix.yalh.net> je disais ça pour taquiner,
> j'ai pas analysé le probléme juste que le 2FA boursorama fonctionne trés bien sur un serveur à la maison mais ne passe pas sur un serveur chez OVH ... Donc rien de bloquant
Ah oui étrange. Faudrait supprimer la session (cookies stockés) et retenter et sinon regarder les logs woob pour voir si y'a pas une histoire d'ip bloquée
[19:17:54]
<yalh76> ouaip c'est à ça que je pense, certaines IP OVH bloquées chez Boursorama
[21:22:33]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @dalgwen commented on issue #478 Folders shared publicly empty when connected with SSO: Exact same issue here. Seems to big to be an issue with Nextcloud. Especially because it is several months old. So I thi... https://github.com/YunoHost-Apps/nextcloud_ynh/issues/478#issuecomment-1276750263