Time Nick Message 23:25 oleonard I do know that my Bootstrap 4 work included a fair number of grid markup differences for different pages' particular needs 23:24 oleonard Off the top of my head I think we aren't as consistent as we'd need to be to use WRAPPER, and I don't know if that's out of necessity or not 23:21 oleonard No I hadn't noticed it before 23:20 dcook Are you familiar with the WRAPPER directive in Template::Toolkit? 23:19 dcook I just sent out an email that might be of interest to you 23:18 dcook Hola, oleonard! 23:18 oleonard Hi dcook 23:05 * dcook waves 22:26 cait1 kind of :) 22:20 caroline cait1: still around? 20:00 cait1 ? 20:00 cait1 that doesn't sound right 19:59 * oleonard wonders why we have empty <label> tags on the OPAC search results page 18:55 oleonard Cool. 18:54 ashimema More than happy if we can get it closer to that 18:53 ashimema I pretty much aimed to steal that.. just using some of bootstraps existing classes.. hence the sr- tags 18:53 ashimema They're nice 18:52 oleonard What would you think about stealing that? 18:52 oleonard ashimema: I like the "skip" link on this page: https://webaim.org 18:51 ashimema Lol 18:51 oleonard Yes wahanui that is perpetually true isn't it? 18:51 wahanui oleonard is seeing some buggy stuff though 18:51 ashimema oleonard 18:50 ashimema Hi 18:12 oleonard ashimema still around? 17:58 caroline juste sent them an email... we'll see... *shrugs* 17:50 oleonard Didier Gautheron at biblibre 17:48 caroline who is did? 17:36 kohaputti wondering if it could be related to those issues Joubu described where the tests stop working after ~30 runs 17:35 huginn Bug 11175: enhancement, P5 - low, ---, joonas.kylmala, In Discussion , Show the parent record's component parts in the detailed views 17:35 kohaputti anyone else come across such issue with the test only working on the first time after running reset_all: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11175#c64 ? 17:27 reiveune bye 17:24 oleonard Someone please check on did 16:55 ashimema Is a cait here? 16:29 oleonard kidclamp++ 16:01 oleonard Cool 16:00 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22807 enhancement, P5 - low, ---, martin.renvoize, Needs Signoff , Accessibility: Add 'Skip to Main Content' link 16:00 ashimema bug 22807 is now ready to test.. wouldn't mind a bit of your expertise in making the actual 'skip to main content' button look a bit nicer oleonard.. for some reason the styling of the alluded me 15:58 ashimema ? 15:58 caroline oh? did it work? 15:58 wizzycray that was a good thought and something I can check on 15:58 * ashimema hasn't actually tried it on mobile for ages.. 15:58 wizzycray caroline++ 15:55 oleonard Obviously that ended up being a little half-baked 15:55 oleonard Haha... yeah... That was intended to make the experience on mobile better by doing the same kind of thing, jumping past all the nav stuff... 15:55 eythian hi oleopard 15:54 wahanui hi oleopard 15:54 wizzycray hi oleonard 15:54 * ashimema wonders two things.. whether he should move his code to script.js instead of global.js and whether he should be closer matching 15:54 wizzycray oh bugger i've gone slack >.< 15:54 wizzycray hi @oleonard 15:54 ashimema they're rather similar to what I'm adding 15:54 ashimema just found lines 2-13 in bootstrap/js/script.js 15:53 ashimema interesting... 15:52 oleonard ashimema I have never tried 15:52 ashimema oleonard.. did you already try adding a 'skip to main content' functionality? 15:52 wahanui er is as you said c 15:52 wizzycray er? 15:52 wizzycray but it doesn't look like it passes a -r (though I could be wrong) 15:52 ashimema er... 15:52 wizzycray and i may be wrong in the 2 minutes I looked at the sliced rebuild script 15:52 * oleonard waves to wizzycray 15:51 wizzycray that's an interesting observation because this one was done sliced 15:47 wizzycray hrm 15:46 caroline With zebra I've seen it a lot if I reindex without -r 15:46 wizzycray zebra 15:46 caroline zebra or elastic? 15:44 wizzycray i... i am stumped. 15:44 wizzycray hi, anybody seen something like a double up on a result page, but one of the biblionumbers doesn't exist anywhere in the database? This happens even after a reindex? 15:41 ashimema ooh.. what is idref? 15:35 ashimema indeed 15:35 oleonard I would say so. I think the only gotcha with regard to adding IDs or classes to Bootstrap grid <div>s is that we don't want to be introducing additional style to that <div> which could break the grid layout 15:34 ashimema probably in alongside the span9 of that div then.. rather than introducing another dib 15:33 * ashimema tries to work out where maincontent should sit 15:33 ashimema n 15:33 ashimema funky by design the 15:32 ashimema oh, ok. :) 15:32 oleonard That is, it shouldn't extend over the right-hand column with the thumbnails (just a design decision) 15:31 oleonard I see that the page heading is in a .span9 div with no corresponding .span3. That's intentional to keep the heading contained within the same space as the large image in cases where the title is especially long 15:26 ashimema I thought that was a requirement in bootstrap 15:26 ashimema they don't seem to add up to 12 for all rows to me 15:25 ashimema is it just me or does the imageviewer template markup look a little funky? 15:21 oleonard ashimema: Yes, I understand your position with regard to your customer, that's why I want to be very proactive in adapting my work to yours rather vice versa 15:20 ashimema I'm very keen to jump on 20168 btw.. it'll be great to have that. 15:19 ashimema I think this is going to end up as a huge number fo tiny patches spread across lots of bugs.. I'm ready and prepared for the rebase hell.. honestly as I need to backport these to a 19.11 for the customer in question it'll be good to do the initial work pre your bug and then adapt it to your bug as part of rebasing 15:18 ashimema no worries 15:18 ashimema these are sooo going to conflict like mad 15:18 oleonard ashimema: I can work to incorporate your changes into my branch of course 15:15 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20168 enhancement, P5 - low, ---, oleonard, Needs Signoff , Update of the OPAC bootstrap template to bootstrap v4 15:15 oleonard ashimema I wish we could combine our efforts somehow but I don't think it would be standard procedures... I had to touch every OPAC template for Bug 20168 15:15 ashimema I'm dreading the header level issues I've got to work through 15:14 ashimema as part of a set of accessibility tasks I'm working through I'm basically having to look at every template in the OPAC.. it's tedious 15:14 ashimema and I've never really thought about how inconsistent we are until now.. 15:13 ashimema me too 15:13 jajm ashimema, a model template would be great, i always pick a random template as a starting point when creating a new page 15:12 ashimema or perhaps have a 'model template' we recommended as the starting point for all pages 15:11 ashimema I'd love to work through and clean them all up... 15:11 ashimema indeed. 15:11 ashimema in other cases we don't both with the additional div and just stick the content directly inside the 'row-fluid' 15:11 Joubu "413 developers" 15:10 ashimema other interesting inconsistency is how we nest divs.. in lots of cases what I think of as the main content is nested within a div with a suitable id matching the pages use.. 15:10 ashimema I'm working through these 'maincontent' issues.. that's what I'm looking at lots of templates 15:09 ashimema the clubs tab is another interesting case.. it's uses ajax to embed a whole template into a page.. rather than using it as an include like everywhere else.. 15:08 ashimema I would have imagined we would either use one template for all three stages of the form.. or a distinct template for each 15:08 ashimema memberentry.. why do we use two templates for it.. one for 'edit' or 'add' and an additional one for 'submitted' 15:07 ashimema you want examples.. lol 15:07 oleonard ashimema: ? 15:06 ashimema lol 15:06 jajm tcohen, and also verify that the patches do not break anything when you use standard apache+starman 15:06 tcohen s/in our.*// 15:05 ashimema we are sooooo inconsistent in our use of templates 15:05 jajm tcohen, my advise would be to start by checking that basic features work (cataloguing, search, checkout/checkin), read the code, understand how the whole thing work, and then test a few specific things like uploading a file, exporting a MARC file and verifying its encoding 14:58 tcohen it is my next assigned thing, make it happen 14:58 tcohen jajm any advise on Turn Koha into a Mojo app testing? 14:47 ashimema lol 14:43 oleonard ashimema: Let me know if you find the answer :D 14:38 kohaputti that would test the code I want to test Koha::Biblio::components, although misses then testing whether the zebra index has the new search field required for it to work 14:36 kohaputti maybe I could just mock the whole search function before it goes to any of the zebra stuff 14:36 kohaputti tcohen, ah, the zebra index is stored in Koha source code for unit tests at t/db_dependent/data/marc21/zebraexport/biblio/exported_records that's why it gave the error 14:27 ashimema did we ever decide is results-grouped is actually used still? 13:43 jajm oleonard-away, yes, i think it would be better if all js translations were using Gettext.js 13:34 oleonard jajm is it a goal to eliminate the use of the _() function altogether? (single underscore) 13:28 koha-jenkins Project Koha_19.11_D10 build #13: STILL FAILING in 4 min 49 sec: https://jenkins.koha-community.org/job/Koha_19.11_D10/13/ 13:27 oleonard Okay jajm I wanted to be sure I wasn't missing something 13:26 jajm oleonard, because it's not used anywhere in opac ? (i don't know, i'm just making a guess) 13:24 oleonard Anyone know why we're not already including Gettext.js in the OPAC? jajm? 13:21 kohaputti there is first a cron which puts the message in the queue for sending and then process_message_queue.pl send those when its run 13:19 kohaputti TimothyAlexis, the cron is which one? 13:18 TimothyAlexis If the cronjob runs every 15 minutes then the message will be sent at latest 00:15, while it's set at a different time (same as patron creation I'd suppose). Is this how it's happening? 13:02 koha-jenkins Project Koha_Master_U18 build #817: FIXED in 1 hr 17 min: https://jenkins.koha-community.org/job/Koha_Master_U18/817/ 13:02 wahanui Congratulations! 13:02 koha-jenkins Yippee, build fixed! 12:16 TimothyAlexis We have added a button where patrons can renew their membership when logging in. But it happens that the e-mail is sent before it's actually triggered in the system. 12:14 Joubu TimothyAlexis: the cronjob script enqueues it in the message_queue table 12:14 TimothyAlexis If the e-mail is sent at a scheduled time and the actual expiration is triggered at a specific time during that date, then there could be a difference. 12:13 TimothyAlexis There is also a setting MembershipExpiryDaysNotice 12:12 huginn Bug 20573: minor, P5 - low, ---, koha-bugs, NEW , Patron receives membership expiry notice but 'will expire soon' alert doesn't show for staff at checkout 12:12 TimothyAlexis Looking at this: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20573 I'm thinking, is the e-mail sent with misc/cronjobs/membership_expiry.pl ? 12:01 ashimema Wonderful to hear TimothyAlexis :) 11:56 oleonard ðŸ‘ï¸ 11:56 TimothyAlexis Thank you magnuse, oleonard, joubu, ashimema, tuxayo, lukeG1, dcook and cait for your humble guidance on every occasion. 11:54 TimothyAlexis I want to tell everyone that I apologize for the times where I have been affected by stress and if it has extended to anyone else. I also want to thank everyone here for the helpfulness and compassion. Thank you. :) 11:52 TimothyAlexis Hello :) 11:51 Joubu I think we should restore the behaviour 11:51 Joubu before it was possible to `sudo koha-shell kohadev`, now it's broken 11:51 Joubu in the alias you mean? 11:50 tcohen shouldn't we fix the problem with koha-shell on the call in KTD? 11:44 koha-jenkins Project Koha_Master_U18 build #816: ABORTED in 44 min: https://jenkins.koha-community.org/job/Koha_Master_U18/816/ 11:41 ashimema pleasure oleonard.. I'm slowly working my way through them 11:41 Joubu you may need to cherry-pick the commit I am talking about in the last comment 11:41 Joubu checkout, prove and enjoy the failures 11:40 Joubu yes 11:40 tcohen I mean, should I check the 19.11.x branch? 11:40 tcohen is that already pushed to 19.11.x? 11:39 tcohen Joubu trying to read it 11:30 oleonard ashimema I'm glad to see you raise the issue of the "maincontent" class in the OPAC, it was something I noticed was inconsistent in my Bootstrap 4 work. 11:30 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22522 critical, P1 - high, ---, ere.maijala, Pushed to master , API authentication breaks with updated Mojolicious version 11:30 Joubu tcohen: any ideas about what's missing on bug 22522? 11:22 tcohen ah, we changed it into using sudo 11:21 Joubu nope, good catch! 11:21 ashimema hope I wasn't sounding too picky ;) 11:20 ashimema thanks Joubu 11:20 ashimema or.. where ever it is we're currently calling it prior to running all the tests at the moment we should just add the -l option there and not change koha-shell itself ? 11:20 Joubu ashimema: I am attaching a follow-up 11:19 ashimema aha.. piping.. that makes allot of sense :) 11:19 eythian I think it was do align with 'su' perhaps. 11:19 Joubu yep 11:18 oleonard Hi all 11:18 eythian Joubu: that's reasonable, so long as you also then have an option to turn it off (for example if you're piping commands into koha-shell.) 11:17 eythian I seem to recall the the default was chosen to align with something else, but I can't remember what. 11:17 Joubu it should default to login if no cmd passed 11:17 ashimema which I'm OK with.. but I'm not close enough as to why you wouldn't want to in the first place 11:16 eythian My guess at the moment would be that if you use it in scripts to run commands, which is common, a login shell is less useful in some fashion. 11:16 ashimema Joubu wants to default to -i (to help some tests pass)... 11:16 ashimema yeah.. brain dredging.. sorry ;) 11:16 ashimema haha.. 11:16 eythian um. 11:16 eythian cripes. 11:16 ashimema and instead allowed it via an option 11:16 ashimema harking back to when you wrote koha-shell.. any hints as to why you didn't set --login/-i on the call to sudo by default? 11:15 eythian hi 11:14 ashimema had a mental block 11:14 ashimema that's the one.. 11:14 ashimema eythian ^ 11:14 marcelr eythian 11:14 * ashimema tries to remember robins irc nick 11:14 ashimema i.e without --login or -i ;) 11:14 Joubu no* 11:13 Joubu what is supposed to to koha-shell without -l? To crash if not cmd passed? 11:13 Joubu indeed 11:13 tcohen can you define non-login shell? 11:12 ashimema tcohen.. any thoughts on why kshell gives a non-login shell by default? 11:12 ashimema thus I would expect the option to be removed or reversed so you can request a non-login shell if you really want to. 11:11 ashimema that was more a comment on the patch as is.. I would FQA it as it stands because it breaks the --login/-l option of the script 11:11 marcelr looking for gold ? 11:10 ashimema still digging.. 11:10 ashimema I've not delved deep enough yet ;) 11:10 marcelr hi #koha 11:10 ashimema pass 11:10 Joubu Should not the default behaviour of kshell be to give you a login shell? 11:10 ashimema what I'm saying is your patch forces it.. so either you should remove the option as part of that (as it's no longer an option.. it's enabled by default) 11:09 ashimema as in Joubu, you can run kshell as `kshell -l` to get a login shell 11:00 tcohen morning 10:59 Joubu and only for onloan items... 10:59 Joubu but there is code to display "X Lost" 10:59 Joubu they are not "grouped" if on loan 10:59 kidclamp I am not defending how we do thing there, it is confusing 10:58 kidclamp you are looking at unavailable items, not the same code :-) 10:58 Joubu :D 10:58 Joubu kidclamp: ok but... why not me?? 10:58 Joubu ashimema: I don't understand. If you remove the -i you cannot `kshell` correctly 10:57 ashimema --login == -i in my reading 10:57 ashimema literally.. the line above your change 10:57 kidclamp Joubu: see my link above 10:56 ashimema as opposed to hard coding -i 10:56 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25538 major, P5 - low, ---, jonathan.druart, Needs Signoff , www/search_utf8.t is failing randomly 10:56 ashimema regarding bug 25538 we actually already optionally allow setting a login shell.. should be instead be using that for running the tests? 10:56 kidclamp https://snipboard.io/5dham4.jpg 10:54 Joubu kidclamp: https://snipboard.io/Cslw8V.jpg - there are 2 CN_1 10:54 davidnind On master for me it shows as one line with 4 in brackets - doesn't show the [CN_XX], so maybe a system preference? 10:52 kidclamp same availability. same home/holding, same location, same collection, same callnumber 10:52 kidclamp make another with same callnumber 10:50 ashimema mm.. seems a little odd to me 10:46 Joubu how can you have more than 1 there? 10:46 Joubu UI question - https://snipboard.io/TNvaDf.jpg => What's the "(1)" in the item list? 10:40 ashimema -i seems sane to me 10:32 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25538 major, P5 - low, ---, jonathan.druart, Needs Signoff , www/search_utf8.t is failing randomly 10:32 ashimema looking at bug 25538 now 10:32 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25701 normal, P5 - low, ---, jonathan.druart, Signed Off , Facets display in random order 10:32 ashimema Thanks for picking off bug 25701 Joubu.. that was likely to end up on my list but after the accessibility work I'm in the middle of.. really happy to see that one done 10:27 Joubu thx 10:26 davidnind done! 10:24 Joubu davidnind: yes please :) 10:23 davidnind Joubu: your patch for 25701 works for me, let me know if you want it signed off. 10:10 kidclamp looking 10:07 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25701 normal, P5 - low, ---, jonathan.druart, Needs Signoff , Facets display in random order 10:07 Joubu kidclamp: can you have a look at bug 25701 please? 10:05 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16248 normal, P5 - low, ---, tomascohen, CLOSED FIXED, ModZebra doesn't update zebraqueue if ES is enabled 10:05 did kohaputti: thanks, yes with ES but without zebra, seems to me at least zebraqueue is never purged but it's populated by bug 16248 10:03 kohaputti it probably works the same way if you have selected zebra as the searchengine but don't have it running, not totally sure. 10:02 kohaputti did, yup, with elasticsearch, and I guess it works even if you don't have elasticsearch but lots of things are then non-functional 10:01 did can Koha run WITHOUT zebra installed at all? 10:00 did hi, 09:27 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25538 major, P5 - low, ---, jonathan.druart, Needs Signoff , www/search_utf8.t is failing randomly 09:27 Joubu ashimema, tcohen: please have a look at bug 25538 06:45 wahanui niihau, alex_a 06:45 alex_a Bonjour 06:43 reiveune hello 03:56 Gary Hello @inlibro 03:44 Gary I have a fresh install on debian, currenlty pushing through the onboarding and keep getting stuck at the 'Create a patron category' page when it asks for 'please only choose one enrollment period' 03:43 Gary ANyone active right now?