Time |
S |
Nick |
Message |
00:24 |
|
|
cait joined #koha |
00:35 |
|
|
cait joined #koha |
05:39 |
|
|
fridolin joined #koha |
06:14 |
|
|
thibaud_g joined #koha |
06:38 |
|
|
cait joined #koha |
07:03 |
|
|
marcelr joined #koha |
07:04 |
|
marcelr |
hi #koha |
07:08 |
|
cait |
hi marcelr |
07:26 |
|
PedroAmorim[m] |
good morning |
07:29 |
|
|
magnuse__ joined #koha |
07:39 |
|
|
cait1 joined #koha |
07:41 |
|
|
cait2 joined #koha |
07:42 |
|
cait2 |
good morning #koha |
07:51 |
|
|
alex_ joined #koha |
07:52 |
|
alex_ |
Bonjour |
08:13 |
|
|
cait3 joined #koha |
08:16 |
|
marcelr |
cait3 ? |
08:16 |
|
|
marcelr2 joined #koha |
08:17 |
|
marcelr2 |
hey |
08:18 |
|
cait |
too many caits, working on it |
08:21 |
|
|
cait joined #koha |
08:25 |
|
magnuse__ |
there can never be too many caits |
08:26 |
|
cait3 |
hm still 2 caits |
08:28 |
|
ashimema |
3 caits now |
08:30 |
|
cait3 |
the other laptop is suffering insomnia |
08:31 |
|
cait3 |
ah one cait less, only seeing one now |
08:31 |
|
cait |
ashimema: i think your 3rd cait was the one when I tried matrix, you should remove that one |
08:31 |
|
ashimema |
did I miss a git server change? |
08:31 |
|
cait |
hm? |
08:31 |
|
ashimema |
lol |
08:31 |
|
|
cait1 joined #koha |
08:31 |
|
cait |
and laptop is awake again gr. |
08:31 |
|
ashimema |
I just pulled and got a raft of security key change errors |
08:32 |
|
cait |
hm i can try too one sec |
08:33 |
|
ashimema |
oh.. it's github not ours |
08:33 |
|
ashimema |
https://github.blog/2023-03-23[…]rsa-ssh-host-key/ |
08:34 |
|
cait |
yes, git pull from kc worked without anyhing unusual for me |
08:35 |
|
Joubu |
ashimema: I pushed to master 15min ago and got an error about that! |
08:37 |
|
Joubu |
"we discovered that GitHub.com’s RSA SSH private key was briefly exposed in a public GitHub repository"...outch! |
08:41 |
|
ashimema |
Yikes |
08:43 |
|
Joubu |
ashimema: should be fixed now |
08:43 |
|
Joubu |
well, I mean the push hook on git.k-c.org |
08:49 |
|
cait |
marcelr: do you want me to file a new bug for the POD update? get_marc_host |
08:50 |
|
marcelr |
unit test is more important and could go together |
08:50 |
|
marcelr |
it is not completely missing but a bit unclear |
08:50 |
|
ashimema |
Nice one |
08:50 |
|
marcelr |
in POD |
08:52 |
|
cait |
I'll wait until you are done with checking the code |
08:52 |
|
marcelr |
ok |
08:52 |
|
marcelr |
seeing funny things |
08:54 |
|
marcelr |
cait: not sure if we are heading for a direct PQA ;) |
08:59 |
|
marcelr |
i might add a followup though |
09:02 |
|
|
fridolin left #koha |
09:09 |
|
marcelr |
de soep was toch niet zo heet |
09:09 |
|
marcelr |
that might be the first time that was said in #koha |
09:13 |
|
cait |
keep in mind... that is only a tempalte fix and not to blame! |
09:13 |
|
cait |
.. for the unit tests and POD at least |
09:16 |
|
|
astyles joined #koha |
09:16 |
|
cait |
hi astyles :) |
09:16 |
|
astyles |
hi 8-) |
09:19 |
|
astyles |
Would any happen to know if there is kinda a limit on the amount of bug patches to put in at anyone time? I am new the community so i don't wont to go to hard. ? |
09:19 |
|
magnuse__ |
no limit :-) |
09:20 |
|
astyles |
cool thanks. |
09:20 |
|
cait |
we are happy to have you |
09:20 |
|
astyles |
can anyone go ahead and test bugs with patches that have needs sign off? |
09:21 |
|
cait |
but we always have not enough testers, so if you try to balanc out testing and patch writing a little, that would be really helpful |
09:21 |
|
cait |
yes :) |
09:21 |
|
astyles |
I am happy to test and patch |
09:21 |
|
cait |
Needs signoff ones can be tested by anyone |
09:21 |
|
Joubu |
cait: bug 33331 FYI |
09:21 |
|
huginn |
Bug https://bugs.koha-community.or[…]_bug.cgi?id=33331 normal, P5 - low, ---, oleonard, NEW , Qwant VIPrivacy is hidding "Share list" link |
09:22 |
|
cait |
the next step, QA is an elected role and we dig a little deeper, looking at coding guidelines and such |
09:22 |
|
astyles |
ok cool good to know |
09:23 |
|
cait |
Joubu: we once had a antivirus tool block a single letter in the a-z in patron search... but only one letter |
09:23 |
|
cait |
took forever to figure that one out |
09:25 |
|
marcelr |
cait: qa is not about blaming :) |
09:25 |
|
astyles |
today at work I was staring at this if statement wondering why the condition did not work, so I did a bunch of testing, then realised, both if and else has some print message. 8-) |
09:25 |
|
cait |
yes... just got a little worried for my small tiny patch |
09:25 |
|
astyles |
the same print message. always little things |
09:26 |
|
cait |
true |
09:31 |
|
astyles |
is 'Master' branch generally the default best option for bugs in new bug reports? |
09:35 |
|
Joubu |
ashimema: talking about "Version" on bugzilla? |
09:35 |
|
ashimema |
? |
09:35 |
|
astyles |
ah yes I am |
09:37 |
|
Joubu |
ashimema: "Unspecified" if you don't know, "Master" if it's a bug you recreate on master, "XX.YY" if it's a bug that only appears on (/from) a specific stable branch |
09:37 |
|
astyles |
ok cool I will make a note of this. thanks. |
09:38 |
|
* ashimema |
assumes Joubu has my nick auto-completing and he really means astyles. |
09:38 |
|
Joubu |
yes, sorry! Didn't realise! |
09:39 |
|
astyles |
Thanks Joubu. 8-) |
09:40 |
|
astyles |
regarding bugs needing sign off, can I hands pick or do people prefer to get older bugs done, assuming still relevant? |
09:40 |
|
Joubu |
if you are starting with Koha I would pick the easy ones first |
09:40 |
|
ashimema |
pick what your comfortable with |
09:41 |
|
ashimema |
it's awesome to get older ones closed out.. but everyone has their own expertise area's too.. I'd pick something your familiar with to start on |
09:42 |
|
astyles |
I am mainly front end some some back end, dusting off my perl for sure. |
09:42 |
|
ashimema |
easy starting ones are marked with 'Academy' as a keyword.. or here's loads of string patches.. |
09:42 |
|
ashimema |
those are great for a quick achievement as they're quick to test and help with just getting the feel for where stuff is |
09:42 |
|
ashimema |
cool 🙂 |
09:42 |
|
cait |
oleonard-away: hm ending up with this in the translations: %s [%%# BOOTSTRAP BREADCRUMBS WRAPPER USAGE [ WRAPPER breadcrumbs ] [ Home breadcrumb automatically included ] [ WRAPPER breadcrumb_item ] |
09:43 |
|
astyles |
ok cool i will check out some academy items, thanks |
09:46 |
|
marcelr |
astyles++ |
09:48 |
|
cait |
astyles: we always need more hands everywhere - all contributions are valued :) |
09:56 |
|
astyles |
it's nice to contribute just for the fun of it. Really enjoying it. |
09:57 |
|
ashimema |
😄 |
09:57 |
|
ashimema |
you'll be a Koha addict in no time.. welcome |
09:57 |
|
* ashimema |
my names Martin and I'm an addict to contributing to Koha ILS |
09:58 |
|
astyles |
lol |
09:59 |
|
cait |
it's a little addictive |
10:00 |
|
ashimema |
have you seen the dashboard astyles? |
10:00 |
|
marcelr |
there you go |
10:00 |
|
ashimema |
https://dashboard.koha-community.org |
10:00 |
|
marcelr |
wish i never saw it lol |
10:00 |
|
ashimema |
lol |
10:00 |
|
ashimema |
it's definitely addictive |
10:00 |
|
astyles |
ahh awesome thanks, makes things a bit clearer. |
10:19 |
|
oleonard |
Hi all |
10:19 |
|
cait |
hi oleonard |
10:19 |
|
cait |
i posted earlier - there are some wierd strings from the wrapper work in po files |
10:20 |
|
cait |
[%%# BOOTSTRAP BREADCRUMBS WRAPPER USAGE [ WRAPPER breadcrumbs ] [ Home breadcrumb automatically included ] [ WRAPPER breadcrumb_item ] |
10:20 |
|
cait |
and some others |
10:21 |
|
oleonard |
:( |
10:21 |
|
|
cait1 joined #koha |
10:21 |
|
ashimema |
Joubu still around? |
10:22 |
|
Joubu |
yes |
10:22 |
|
ashimema |
can I get your opinion on a backport |
10:22 |
|
ashimema |
the es indexing stuff that all just hit master |
10:23 |
|
Joubu |
the new worker? |
10:23 |
|
ashimema |
there's quite a tree and the tip of it is all marked with 22.11 required |
10:23 |
|
ashimema |
yeah |
10:23 |
|
ashimema |
that one |
10:23 |
|
ashimema |
do you agree the whole lot should come in.. or are we a bit late in the cycle for such a change |
10:24 |
|
Joubu |
I don't know |
10:24 |
|
ashimema |
I've fallen a bit behind on know where that area was headed and whether as a full set we're confident |
10:24 |
|
Joubu |
we should backport, but maybe next month? |
10:24 |
|
Joubu |
to let us catch possible problem? |
10:24 |
|
Joubu |
but if you pick bug 32594 you must pick bug 33108 as well |
10:24 |
|
huginn |
Bug https://bugs.koha-community.or[…]_bug.cgi?id=32594 critical, P5 - low, ---, nick, ASSIGNED , Add a dedicated ES indexing background worker |
10:24 |
|
huginn |
Bug https://bugs.koha-community.or[…]_bug.cgi?id=33108 major, P5 - low, ---, tomascohen, Pushed to master , We need a way to launch the ES indexer automatically |
10:25 |
|
ashimema |
definitely |
10:25 |
|
Joubu |
and... it's assigned because we are still working on it |
10:25 |
|
Joubu |
the last patch is not pushed yet |
10:25 |
|
Joubu |
I would push it next month |
10:26 |
|
Joubu |
I mean, I would backport it to stable next month |
10:26 |
|
ashimema |
ta |
10:26 |
|
ashimema |
that's perfect.. all the info I needed.. |
10:26 |
|
ashimema |
it was the current state of affairs I wasn't fully confident in 🙂 |
10:26 |
|
ashimema |
yeah |
10:26 |
|
ashimema |
no point in rushing it out and making things worse with release just around the corner.. it can wait for .05 |
10:26 |
|
ashimema |
in which case.. I'll grab 32923 next 🙂 |
10:27 |
|
ashimema |
and then my queue is set ready for next time. |
10:27 |
|
Joubu |
you need the last patch on 32923, but MatthewBlenkinsop[m] is aware of that |
10:27 |
|
Joubu |
heh, too slow |
10:28 |
|
ashimema |
😄 |
10:28 |
|
Joubu |
ashimema: maybe... bug 33282 |
10:28 |
|
huginn |
Bug https://bugs.koha-community.or[…]_bug.cgi?id=33282 enhancement, P5 - low, ---, pedro.amorim, Pushed to master , Cypress tests are failing |
10:28 |
|
Joubu |
to be honest I don't know if it's for 22.11 |
10:28 |
|
ashimema |
hmm |
10:28 |
|
ashimema |
when did you push that? |
10:28 |
|
Joubu |
but it's easy to test, run cypress tests and see if they are failing, if they are, apply and see if they are fixed |
10:29 |
|
Joubu |
this morning |
10:29 |
|
ashimema |
this morning? |
10:29 |
|
ashimema |
coolios |
10:29 |
|
ashimema |
that's why my queue hasn't caught it yet 🙂 |
10:29 |
|
Joubu |
I wanted jenkins green for the hackfest :) |
10:29 |
|
ashimema |
I'll run my queue management script |
10:29 |
|
ashimema |
agreed |
10:29 |
|
ashimema |
that reminds me.. I was going to boost the sandbox server for hackfest.. thinking I'll tripple it at least |
10:30 |
|
Joubu |
yes, good idea! |
10:31 |
|
Joubu |
and we should freeze ktd, misc4dev, koha-sandbox, etc. :) |
10:31 |
|
ashimema |
yeah.. assuming they're all solid and working right now 😜 |
10:31 |
|
ashimema |
I've not had any reports otherwise this week.. but I've also not really been paying attention this week... soooooooo many meetings |
10:32 |
|
cait1 |
only bug fixes |
10:32 |
|
cait1 |
feature freeze |
10:35 |
|
cait1 |
marcelr: i'll file the POD/unit test bug, but a little later |
10:38 |
|
marcelr |
np |
10:38 |
|
ashimema |
I should probably announce string freeze |
10:38 |
|
ashimema |
what a week |
10:53 |
|
cait1 |
strings have already beenupdated on pootle |
10:53 |
|
cait1 |
was quite a bit of work |
10:54 |
|
ashimema |
okies |
10:54 |
|
pastebot |
"oleonard" at 127.0.0.1 pasted "Is this format for template comments helpful and not annoying?" (13 lines) at https://paste.koha-community.org/2471 |
10:54 |
|
oleonard |
Seems to fix the translation issue cait1 |
10:54 |
|
cait1 |
ah, what was it before? |
10:55 |
|
cait1 |
i think it looks quite readable |
10:55 |
|
cait1 |
and it's only in the one file right? |
10:55 |
|
oleonard |
https://gitlab.com/koha-commun[…]_helpers.inc#L360 |
10:58 |
|
cait1 |
interesting |
10:58 |
|
cait1 |
so it's the multi-line that throws it off? |
11:01 |
|
cait1 |
maybe we shoud put this ina coding guidelines |
11:02 |
|
oleonard |
Apparently so |
11:02 |
|
cait1 |
the comments ending up in translations can get bad if people make mistakes, breaking the templates |
11:02 |
|
oleonard |
https://gitlab.com/koha-commun[…]n-search.inc#L182 |
11:02 |
|
cait1 |
if that's the easiest way to avoid, we shoudl do it |
11:03 |
|
cait1 |
and it looks like there is precedence |
11:03 |
|
cait1 |
bbiab |
12:20 |
|
|
alex_ joined #koha |
12:54 |
|
|
marie-luce joined #koha |
12:59 |
|
|
emlam joined #koha |
12:59 |
|
emlam |
morning everyone! |
13:03 |
|
magnuse |
hiya emlam |
13:06 |
|
|
cait joined #koha |
13:08 |
|
|
Dyrcona joined #koha |
13:48 |
|
|
cait1 joined #koha |
14:29 |
|
magnuse |
have fun #koha, see some of y'all on sunday or monday! |
14:52 |
|
|
cait joined #koha |
15:00 |
|
|
bag joined #koha |
15:17 |
|
|
cait left #koha |
15:18 |
|
|
jzairo joined #koha |
16:03 |
|
ashimema |
did all the cait s disappear already? |
16:52 |
|
|
cait joined #koha |
19:05 |
|
|
jzairo left #koha |
19:48 |
|
|
fridolin joined #koha |
20:30 |
|
fridolin |
Bug 33334 SystemD in the place ); |
20:30 |
|
huginn |
Bug https://bugs.koha-community.or[…]_bug.cgi?id=33334 enhancement, P5 - low, ---, koha-bugs, NEW , koha-worker.service need update for background job queues |
20:30 |
|
fridolin |
;) |