IRC log for #koha, 2019-04-23

All times shown according to UTC.

Time S Nick Message
00:44 dpk_ joined #koha
02:57 wizzyrea joined #koha
05:06 chris joined #koha
05:27 cait joined #koha
05:40 calire joined #koha
05:50 dpk_ joined #koha
05:54 fridolin joined #koha
06:07 paxed hmmm
06:08 paxed i don't think the translation toolchain picks up this:
06:08 paxed koha-tmpl/intranet-tmpl/prog/e​n/modules/tools/                    [% SET location = "OPAC news" %]
06:16 kohaputti joined #koha
06:34 laurence joined #koha
06:37 AndrewIsh joined #koha
06:37 reiveune joined #koha
06:38 reiveune hello
06:42 paul_p joined #koha
06:59 alex_a joined #koha
06:59 alex_a bonjour
07:00 andreashm joined #koha
07:06 cait joined #koha
07:19 cait good morning #koha!
07:40 josef_moravec joined #koha
07:40 josef_moravec morning #koha
07:40 andreashm joined #koha
07:44 sophie_m joined #koha
07:46 cait good morning josef_moravec :)
07:47 sophie_m joined #koha
07:52 ashimema Morning #koha
08:04 cait morning ashimema
08:41 ere joined #koha
08:41 ere cait: yt?
08:53 cait you there? yep
08:54 cait looks like you made more work than I expected :)
08:54 cait will take patience to test :)
08:54 ere cait: hi! Sorry about that... I'm finding it hard to leave it halfway there :P
08:55 cait I can see that
08:55 cait next time you do something like this maybe split it up more :)
08:55 cait like leave the removing of old code to a separate bug at the end of the tree
08:55 cait but it's there... and I started
08:55 cait so better try and finish
08:56 cait ashimema: could you take a look at the blocker bug filed for LDAP? I think it might be a setup issue, but don't know LDAP well enough
08:56 cait bug 22738
08:56 huginn` Bug[…]_bug.cgi?id=22738 blocker, P5 - low, ---, koha-bugs, NEW , Software Error after LDAP Authentication
08:57 cait or anyone else having an idea abot ldap of course :)
08:57 ere cait: I'm trying, but since I initially tried to minimize template changes, I wasn't quite prepared.. Sorry for the burden!
08:57 lukeG1 joined #koha
09:00 ashimema Sorry about that.. been on and offline allot after a set of rebbot
09:00 ashimema reboots.. even
09:00 ashimema right..
09:00 ashimema so bug 22738.. on it
09:03 ere So, is the idea with e.g. staff-global.css that RM will update it whenever needed? I can't find anything about the process in the wiki.
09:05 ashimema correct I believe
09:05 ashimema I certainly do as rmaint
09:05 ashimema it's worth highlighting in the bug that there are scss changes.. they're easy to miss sometimes when they're buried inside a commit/patch containing lots of other changes
09:06 ashimema personally I try to commit the changes to that file separately as a way to highlight it.
09:08 ashimema and.. for testing.. none of the sandboxes yet do a scss -> css rebuild as part of their bug application process.. so to ease SO, it's often worthwhile adding a 'DO NOT PUSH' commit with the SCSS rebuild into the bug.. that way the bug will apply happily on sandboxes for testing but the RM also knows to not push that patch and it acts as a hint to the RM that they need to do a CSS rebuild
09:33 cait ere: yep, RM will generate them
09:35 cait ere: i think the info is missing or in another spot, i usually refer to https://wiki.koha-community.or[…]_and_staff_client
09:36 cait looks like someone wrote a git hook to prevent committing them: https://wiki.koha-community.or[…]i/Tips_and_tricks
09:42 * kidclamp waves
09:44 laurence joined #koha
09:55 ere cait: Well, the hook might be useful, but I think it would be also useful to actually document how you're supposed to do it..
10:01 cait agreed
10:01 cait i'll see about it later
10:01 andreashm joined #koha
10:02 cait lunch :)
10:52 andreashm joined #koha
11:25 Remy_PREBS joined #koha
11:26 gveranis joined #koha
11:30 paul_p joined #koha
11:36 oleonard joined #koha
11:36 oleonard Hi all
11:37 cait hi oleonard :)
11:37 cait did you have a quiet monday?
11:37 oleonard it was unbearably quiet. I almost didn't survive.
11:38 * cait sends cookies
11:45 alex_a joined #koha
11:46 gveranis hi all
11:46 gveranis hi cait
11:51 cait hi gveranis :)
11:51 cait oleonard: ere pointed out earlier that we have it nowhere in writng not to commit the css files
11:51 cait wonderingw here we should put it
11:52 cait is there another page apart from https://wiki.koha-community.or[…]_and_staff_client  you could think of?
11:52 oleonard That's what I think of first
11:52 cait and shoudl it say SASS in the heading?
11:53 cait Compiling Sass files
11:55 cait josef_moravec: i think the YAML patch is already there
11:55 cait bug 22584
11:55 huginn` Bug[…]_bug.cgi?id=22584 enhancement, P5 - low, ---, oleonard, Pushed to Master , Add YAML support for Codemirror
11:56 cait oh i see! never mind
11:56 josef_moravec bug 22757 is followup
11:56 huginn` Bug[…]_bug.cgi?id=22757 enhancement, P5 - low, ---, josef.moravec, Needs Signoff , Use YAML CodeMirror higlighting on YAML preferences
11:56 cait yep, misread, thx! i had a feeling when testing we were missing something... but not that many!
11:56 josef_moravec maybe could be better described, of course
11:57 josef_moravec cait: there will be maybe discussion about that... these are ones used in code by loading using YAML module - but it is often not exactly yaml...
11:58 cait yeah... but it doesn't do syntax check... so will still look nicer i guess
11:59 josef_moravec cait: true
12:00 cait oleonard: reading helps... it says we use sass in scss format at the beginning
12:11 GiuseppeP joined #koha
12:11 cait josef_moravec: will you qa 14576? I started, but woudl not mind :)
12:12 josef_moravec Just wanted to add the follow-up for the syntax highlight ;)
12:12 josef_moravec so you can continue on that, just consider this one small patch from me ;)
12:13 oleonard cait: It's confusing. But I prefer to say "SCSS" because that's the extension of the files
12:14 oleonard cait did you happen to see the latest comment on Bug 22318?
12:14 huginn` Bug[…]_bug.cgi?id=22318 enhancement, P5 - low, ---, oleonard, Pushed to Master , Extend Koha news feature to include other content areas
12:15 cait oleonard: i have kept them, but maybe you want to update? i htink the note on top of the page might not be right either? says we only have one scss file
12:15 cait i just added a paragraph about the committing now
12:15 cait oleonard: i haven't tested it myself, but paxed is usually right
12:15 cait do you know how to test it?
12:16 oleonard I think I do, but it's not immediately obvious to me how to fix it
12:17 cait I'd run a quick test first before you go to the trouble
12:17 cait and we might want to fix others than too...
12:18 cait i guess one coudl do something like
12:18 cait [% IF ( location == 'OPAC news' ) %]Opac News[% END %]
12:18 cait i guess that would work for sure... but not looking really nice
12:31 josef_moravec oleonard++
12:31 oleonard josef_moravec++
12:35 cait oleonard++ josef_moravec++
12:40 andreashm joined #koha
12:43 caroline_catlady joined #koha
12:45 caroline_catlady good morning!
13:31 corilynn love it when I get an email back from LOC and it contains sarcasm :)
13:43 paxed cait, oleonard-away: you can assign a BLOCK into a TT variable, that might be the nicest solution
13:44 koha-jenkins Project Koha_18.05_D8 build #269: SUCCESS in 18 min: https://jenkins.koha-community[…]oha_18.05_D8/269/
13:47 paxed cait, oleonard-away: so something like [% SET location = BLOCK %]OPAC news[% END %] would let the text be translatable while still being assigned into a variable
13:50 koha-jenkins Yippee, build fixed!
13:50 wahanui Congratulations!
13:50 koha-jenkins Project Koha_18.05_D9 build #261: FIXED in 26 min: https://jenkins.koha-community[…]oha_18.05_D9/261/
13:52 wizzyrea joined #koha
13:56 wizzyrea hi
14:11 talljoy joined #koha
14:11 koha-jenkins Project Koha_18.05_U18 build #258: SUCCESS in 44 min: https://jenkins.koha-community[…]ha_18.05_U18/258/
14:18 wizzyrea good morning
14:18 wizzyrea @wunder 66542
14:18 huginn` wizzyrea: Error: I got an error searching 'None'. (keynotfound: this key does not exist)
14:18 wizzyrea @wunder 66542
14:18 huginn` wizzyrea: Error: I got an error searching 'None'. (keynotfound: this key does not exist)
14:18 wizzyrea kek
14:18 wizzyrea @wunder Tecumseh Kansas
14:18 huginn` wizzyrea: Error: I got an error searching 'None'. (keynotfound: this key does not exist)
14:19 wizzyrea @wunder Topeka
14:19 huginn` wizzyrea: Error: I got an error searching 'None'. (keynotfound: this key does not exist)
14:19 wizzyrea hm maybe it's just broken
14:19 wizzyrea @wunder nzwn
14:19 huginn` wizzyrea: Error: I got an error searching 'None'. (keynotfound: this key does not exist)
14:19 wizzyrea yes it is.
14:21 calire left #koha
14:24 cait ere++
14:25 cait corilynn: what was it about? :)
14:26 corilynn I asked them where the data was for a document that is a dead link everywhere it was linked
14:26 cait josef_moravec++ # thx for the rebase!
14:26 corilynn she said it was "approved" but it was still changing, of course!
14:26 cait oh
14:26 corilynn subtle, but still, funny
14:28 caroline_catlady joined #koha
14:31 cait i guess i would have taken that as serious :) not being a native speaker and all :)
15:00 koha-jenkins Project Koha_18.05_D8 build #270: SUCCESS in 18 min: https://jenkins.koha-community[…]oha_18.05_D8/270/
15:09 koha-jenkins Project Koha_18.05_U18 build #259: SUCCESS in 27 min: https://jenkins.koha-community[…]ha_18.05_U18/259/
15:14 oleonard Thanks paxed!
15:15 * oleonard waves to caroline_catlady, corilynn, and wizzyrea
15:15 paxed oleonard: that was off the top of my head, so i haven't tested it
15:16 caroline_catlady hi oleonard!
15:16 wahanui hi oleopard
15:17 wizzyrea whassup oleonard
15:19 oleonard Just briefed the branch managers about starting online patron registration
15:21 cait nice :)
15:21 wizzyrea ooooo
15:21 wizzyrea so exciting
15:23 oleonard If anyone has experienced any problems with that feature please let me know :)
15:23 cait when it works it works for us
15:24 cait we sometimes had issues with validation - submitting the form when mandatory and hidden fields didn't match up
15:24 cait but that you figure out pretty fast, you just can't use it
15:24 wizzyrea there were bugs when it was new there are a lot fewer now
15:25 caroline_catlady I haven't been able to work out the email validation either, but I didn't try very hard
15:25 cait we use it with, seems to work well
15:25 oleonard I think the only open question right now is whether we can get our message queue processed on a much faster schedule
15:25 cait oleonard: get the bug fixed instead .)
15:26 cait we figured out a way to send the password reset immediately
15:26 wizzyrea the answer is: yes.
15:26 wizzyrea lol
15:26 cait bug 21390
15:26 huginn` Bug[…]_bug.cgi?id=21390 enhancement, P5 - low, ---, koha-bugs, NEW , Send self registration verification emails immediately
15:26 wizzyrea or that
15:26 koha-jenkins Project Koha_18.05_D9 build #262: SUCCESS in 42 min: https://jenkins.koha-community[…]oha_18.05_D9/262/
15:26 cait we fixed bug 18570 - i see no reason why the solution wouldn't work for selfreg
15:26 huginn` Bug[…]_bug.cgi?id=18570 normal, P5 - low, ---, mtompset, RESOLVED FIXED, Password recovery e-mail only sent after message queue is processed
15:27 cait it still goes via the queue, but is sent immediately, best of both worlds
15:28 oleonard Yeah it has been on our list to get work done on Bug 21390 but we don't want to wait
15:29 cait understandable
15:29 edveal joined #koha
15:29 cait bye all, cya later :)
15:30 cait left #koha
15:33 ashimema wizzyrea still around?
15:33 wizzyrea yep sup
15:33 ashimema just revisiting bug 11573 and contemplating how to do it
15:34 huginn` Bug[…]_bug.cgi?id=11573 normal, P5 - low, ---, wizzyrea, REOPENED , Fine descriptions related to Rentals are untranslatable
15:34 ashimema it's about making the rental charge descriptions translatable
15:34 wizzyrea hmm
15:35 ashimema there are currently four distinct strings in there.. 'Rental', 'Daily Rental', '*Rental Renewal*' and '*Daily Rental Renewal*' (the one's with * actually add a bunch of extra details into the description too)
15:35 ashimema just wanted some pointer as to where we should go with it
15:36 wizzyrea oh gross so the question is how to translate those stinky strings into something translatable?
15:36 ashimema I already sort of don't like that the 'Rent' accounttype is used for both a fixed rental charge and an accrued daily rental charge so I was thinking of creating a second rental chage type for that
15:36 oleonard ("stinky strings" is what I call shoelaces now)
15:36 wizzyrea keke
15:37 ashimema but.. the renewal case has stumped me.. do we need the fact that it's a renewal distinctly?
15:37 ashimema are librarians likely to miss that distinction?
15:37 * wizzyrea ponders
15:37 wizzyrea what is the use case - someone renews an item that is a rental, and they get a whole new fee?
15:37 wizzyrea well ok
15:37 wizzyrea now that I've said it out loud
15:38 wizzyrea the librarians aren't the ones who would care about the distinction
15:38 ashimema hehe
15:38 wizzyrea the borrowers are
15:38 ashimema indeed
15:38 ashimema fair point
15:38 wizzyrea the people who pay, not the ones levying the fee care
15:38 fridolin left #koha
15:38 wizzyrea librarian won't care, a fee's a fee.
15:38 wizzyrea the patron tho
15:38 wizzyrea they'll want to know what exactly they got charged for
15:39 ashimema lol
15:39 wizzyrea so yes sorry I think you need to keep it
15:39 ashimema I'm rapidly getting to turning one accounttype into four!
15:39 ashimema feels somewhat over the top.. but I can't think of a better way at present
15:39 wizzyrea (the counter argument is, if you were going to say, a video rental store, and you checked out Top Gun, then went back and re-rented it, do you care that you renewed it?)
15:40 wizzyrea top gun because that's from the era of video rental stores.
15:40 ashimema hehe
15:40 ashimema I like the accuracy in your argument there ;)_
15:40 oleonard I would have said Showgirls, but that works.
15:40 wizzyrea so I guess, do we take the (LOOOL) "this turns the library into a rental shop" route,
15:40 wizzyrea or the "this is a library and library patrons want to know what they are being charged for and why"
15:41 wizzyrea route
15:41 corilynn the second one, wizzyrea
15:41 ashimema okies.. four accounttypes it is
15:41 ashimema will that piss off librarians doing reports though would be the next logical question ;)
15:41 wizzyrea (that's my instinct too but didn't want to commit without thinking about the other side!)
15:41 corilynn as a person who mans the ref desk and has to answer patron questions about their "mysterious" charges when the circ desk is scared
15:42 wizzyrea ^ this is 100% the best take to give weight to.
15:42 wizzyrea what is it like for the poor librarian who has to answer the questions about it
15:42 ashimema though I suppose it's not especially hard to update a report from `WHERE accountype = 'Rent'` to `WHERE accounttype LIKE 'Rent_%'`
15:42 ashimema or something like that
15:43 wizzyrea well
15:43 oleonard Librarians will usually prefer more granular statistics anyway
15:43 wizzyrea the reports are moving targets anyway
15:43 corilynn "says you have a .25 charge for being a week late on BLAH BLAH by WHOMEVER and a $23 charge on damaging WHOIST by YACK"
15:43 ashimema awesome
15:44 ashimema thanks guys, pleasure using your as my rubber duckies ;)
15:44 ashimema hehe
15:44 wizzyrea here to help
15:44 wizzyrea corilynn++ for excellent front line opinions
15:44 corilynn anytime, seriously
15:44 ashimema accounts is gonna just be waaay nicer in 19.05 me thinks.. so long as these keep moving quickly enough to get in
15:44 wizzyrea :D
15:47 oleonard wizzyrea: Next time you are up for doing a signoff could you look at Bug 21784?
15:47 huginn` Bug[…]_bug.cgi?id=21784 enhancement, P5 - low, ---, oleonard, Needs Signoff , Clean up
15:48 wizzyrea i am up now, because that might help me get back on proper task
15:49 wizzyrea lol that one is rather huge teehee
15:49 wizzyrea deceptive "apply the patch and look at various pages in the staff client
15:49 wizzyrea to confirm that no JavaScript-driven interactions are broken."
15:49 wizzyrea Is this a challenge, I think it is.
15:49 oleonard The changes are teeny though
15:49 wizzyrea mua ha ha ha ha
15:50 wizzyrea (I'm teasing you)
15:50 oleonard I mean... You definitely could check every page in the staff client
15:50 wizzyrea i know it's wee.
15:50 wizzyrea might find other bugs tho lol
15:54 wizzyrea j/k I won't
15:54 wizzyrea do we like what we did to the circ home page
15:55 wizzyrea I do, was thinking of doing that to other areas.
15:55 oleonard Yes. Like where?
15:55 wizzyrea like maybe um
15:56 wizzyrea instead of the paragraph intro we had before, and the vacuum we've got now
15:56 wizzyrea on the label creator
15:56 wizzyrea and maybe on serials?
15:56 wizzyrea so buttons for "claims" "check expiration"
15:56 wizzyrea "manage frequencies" etc
15:57 wizzyrea and on labels maybe have buttons to the manage ones
15:57 wizzyrea instead of a drop down
15:57 laurence left #koha
15:57 wizzyrea like that
15:57 ashimema yup.. this could lead to me having to increase the accounttype column again
15:58 wizzyrea >.<
15:58 wizzyrea there are a lot of tools, but I'd like to do the tools
15:58 ashimema RENTAL_DAILY_RENEWAL won't fit inside a varchar16
15:58 wizzyrea oops.
15:58 wizzyrea would that db update be brutal for libraries with a lot of accountlines
15:59 ashimema grr
15:59 ashimema erm.. pass
15:59 wizzyrea brutal meaning "take a very long time"
15:59 ashimema we've recently updated it to 16 within the 19.05 cycle too
15:59 wizzyrea (at upgrade)
15:59 ashimema I don't think increasing is too honerous.. but I didn't benchmark it
15:59 wizzyrea i'm talking places with millions of lines (like some of the NZ publics)
16:00 wizzyrea and probably places like NEKLS and other consortia
16:00 ashimema indeed
16:00 ashimema `RENTAL_DAILY_REN` could suffice
16:01 ashimema less descriptive a code.. but it is a code so should never be in display anyway
16:01 wizzyrea my feeling is "make it fit" lol.
16:02 wizzyrea oleonard, done
16:02 oleonard Thank you!
16:02 oleonard wizzyrea++
16:02 ashimema RENT_DAILY_RENEW
16:02 ashimema that'll fit
16:03 wizzyrea yes and readable as well
16:03 wizzyrea I wanted to do buttons to reports, and rearrange that page
16:03 wizzyrea but oleonard beat me to it
16:03 wizzyrea was sad bc I had it almost half done
16:03 oleonard :(
16:03 wizzyrea naw sok
16:04 cait joined #koha
16:04 wizzyrea I was alphabetising things and promoting some things that work well and demoting things that don't
16:05 * cait waves
16:05 wizzyrea but it wasn't done and you beat me so I kinda said yeah sok. :)
16:11 wizzyrea bug 19014 - corilynn, do you have an opinion?
16:11 huginn` Bug[…]_bug.cgi?id=19014 major, P5 - low, ---, jonathan.druart, Needs Signoff , Patrons should not get a on_reserve notification if the due date is far into the future
16:12 corilynn sorry, on what?
16:12 corilynn I'm reading AWS documentation, it's like the twilight zone
16:12 wizzyrea on bug 19014, what libraries might generally expect
16:12 wizzyrea in terms of what happens when things auto renew
16:13 wizzyrea (it's ok to say "no idea")
16:13 wizzyrea and when they'd expect a notice
16:13 corilynn looking
16:13 wizzyrea it's probably the last mmm 4 or 5 comments
16:13 wizzyrea where the argument is happening
16:14 wizzyrea the patch is teasing me for a sign off but I don't want to wade into that drama
16:14 corilynn lol
16:14 wizzyrea without advisement
16:14 wizzyrea i really feel we should make enhanced messaging default on
16:14 wizzyrea non sequitur
16:15 corilynn ok, on the face of it, YEAH seems dumb to notify peeps of renewals that are in the far future
16:16 cait it's been a while, but far future or the day of renewal both seem not right to me
16:16 cait it is the message when the item can't be renwed right?
16:17 cait so i would have no time to return it
16:17 wizzyrea that's right
16:17 cait without getting fines (not all libraries... but still a lot)
16:17 corilynn ok, so what my library does, (which makes sense to me), is to say three days before the item is due: a. we auto-renewed the item OR b. we cannot auto-renew the item
16:17 cait yeah
16:17 wizzyrea I like 3 days
16:17 cait that was how we thoughtout the renewal feature at the beginning, using the normal reminder emails
16:17 wizzyrea what would you say though
16:17 cait i titally missed out on the discussion of when the specific auto-renew mail was added
16:18 wizzyrea "we may not be able to renew this item"
16:18 wizzyrea "to avoid fines you should return it"
16:18 cait we made it so the feature would start renewing 3 days before too
16:18 cait so it would be 'couldn't renew'
16:19 corilynn "The following books are due on XX-XX-XX and were unable to be auto-renewed"
16:19 cait i am not sure why we can't make this one work similarly?
16:19 corilynn where XX-XX-XX is three days from now
16:19 cait there is the 'don't renew before' setting in circ
16:19 corilynn do you want me to wade in on the bug?
16:20 cait comments are always good i tihnk
16:20 wizzyrea comments are really good
16:20 wizzyrea as it's devolved into a bit of ego
16:20 corilynn it's not my particular bally-wack, but okay
16:20 wizzyrea my way! no MY way
16:20 cait heh
16:20 wizzyrea "well, here's what we do..."
16:21 cait I guess i was a little afraid of it too
16:21 wizzyrea dont feel like you have to get involved any more than you want to
16:21 wizzyrea perspectives are good tho
16:21 corilynn you asked, i answered, might as well chip it in stone
16:21 cait i know Holger was participating - we worked out the initial feature together and he did the programming, but that was before the new notice was added and some changes were made
16:21 corilynn (plus my boss monitors my activity on bugzilla for some reason)
16:22 wizzyrea like, for performance review?
16:22 cait ugh?
16:22 corilynn no, he's just curious
16:22 wizzyrea that could be good or bad. :)
16:22 corilynn exactly
16:22 corilynn no one knows what i do
16:22 wizzyrea well you send him to me and I'll say "librarian viewpoints are really valuable"
16:23 wizzyrea >.>
16:23 wizzyrea "you let them do what they want"
16:23 corilynn It's like I make pizza in a ramen noodle shop
16:23 wizzyrea haha
16:23 wizzyrea ramen pizza!
16:23 corilynn lol
16:23 wizzyrea this idea has merit.
16:23 wizzyrea pork belly and miso pizza? yes.
16:23 corilynn my *other* boss goes glassy-eyed if I say software development words
16:26 wizzyrea hehe
16:26 corilynn yet, i was hired to "make things work using software"
16:27 wizzyrea hehe
16:27 wizzyrea that's why I call myself "doer of things"
16:27 oleonard[…]oodles-Pizza-Bake
16:27 corilynn i used to call myself SQL-Bitch
16:28 wizzyrea omfg oleonard
16:28 corilynn yeah, that looks kinda tasty oleonard
16:28 wizzyrea ok
16:28 wizzyrea in nz
16:28 wizzyrea spaghetti pizza is 100% a thing
16:29 oleonard If you're looking to be dissuaded from doing ramen pizza look no further than this:
16:29 wizzyrea but it uses tinned spaghetti
16:29 wizzyrea see I was thinking of a ramen *flavoured* pizza
16:29 wizzyrea with a regular bready crust
16:29 wizzyrea and a miso kind of sauce with bok choy and pork belly and sesame oil
16:29 corilynn Maybe I should have said making Pizza in a window factory?
16:30 corilynn added my $1.02 to bug 19014
16:30 huginn` Bug[…]_bug.cgi?id=19014 major, P5 - low, ---, jonathan.druart, Needs Signoff , Patrons should not get a on_reserve notification if the due date is far into the future
16:30 oleonard corilynn: Only if you don't want the conversation to be completely derailed
16:30 wizzyrea corilynn++ yay
16:38 corilynn cait, I failed bug 18823, but when you have a moment can you go through Alex's comments?
16:38 huginn` Bug[…]_bug.cgi?id=18823 enhancement, P5 - low, ---, alex.arnaud, Failed QA , Advanced editor - Rancor - add ability to edit records in import batches
16:39 corilynn comment 106
16:44 tcohen hola  #koha
16:45 oleonard Hi tcohen
16:49 cait corilynn: on house chores now, but will try when i get back on the laptop properly
16:50 wizzyrea cait (or ashimema) there's another one that needs looking at reasonably urgently - 22759
16:50 wizzyrea bug 22759
16:50 huginn` Bug[…]_bug.cgi?id=22759 blocker, P1 - high, ---, nick, Signed Off , Circulation rules for maxissueqty are applied per branch even for defaults
16:55 cait there is tons... queue exploding
16:55 cait qa_team help!
16:55 cait qa_team?
16:55 wahanui qa_team is cait alex_a jajm khall josef_moravec ashimema Joubu marcelr tcohen rangi
16:56 wizzyrea qa_team help! is <reply> cait alex_a jajm khall josef_moravec ashimema Joubu marcelr tcohen rangi
16:56 wizzyrea qa_team help!
16:56 wahanui cait alex_a jajm khall josef_moravec ashimema Joubu marcelr tcohen rangi
16:56 wizzyrea ha
16:56 oleonard qa_team ASSEMBLE!
16:56 ashimema lol
16:57 wizzyrea i like that too
16:57 * ashimema is about to have dinner
16:57 ashimema I can have a punt at a bunch of them tomorrow unless you beat me to it cait
16:57 * ashimema is struggling to keep up with QA demand at the moment
16:57 wizzyrea that one I linked is rather bad
16:57 ashimema and is waaay behind on his RMaint queue :(
16:58 edveal joined #koha
16:59 ashimema cait
17:00 ashimema bug 11573..
17:00 huginn` Bug[…]_bug.cgi?id=11573 normal, P5 - low, ---, wizzyrea, REOPENED , Fine descriptions related to Rentals are untranslatable
17:00 ashimema is it important to keep the inconsistency in the untranslatable strings
17:00 ashimema i.e.
17:01 ashimema 'Rental' vs 'Rental fee' vs 'Renewal of Rental Item TITLE BARCODE'
17:02 ashimema and the same for with 'Daily' prepended
17:02 ashimema i.e. do we need to keep the display of TITLE and BARCODE from the item in the fee description even though we store both the issue and item id in the table and I think display that already
17:03 wizzyrea i'd rather reduce display redundancy
17:03 ashimema me too :)
17:04 wizzyrea if it's doubled up, undouble it lol
17:07 oleonard If it's quadrupled let's halve it!
17:12 wizzyrea what you said.
17:13 ashimema ok.. it's ready for a first round of testing now :)
17:13 wizzyrea oo
17:14 ashimema it's at the top of my accounts tree at the moment.. can probaly be pushed a but futher back if we want to but I'm sorta hoping it might encourage some eyes on the stuff further up the tree too ;)
17:14 ashimema it will fail QA as it stands (I've detailed why in a comment.. just needs a DB update really which I'll work on after tea)
17:14 wizzyrea oleonard, has you a minute
17:14 ashimema have a good evening #koha
17:15 * oleonard do
17:15 wizzyrea i'm gonna sign off on 21946
17:15 wizzyrea and you had some display ideas about the hierarchy
17:16 wizzyrea (that one depends, functionally, on 22759, but there's nothing to do with the interface wrong except it needs UX love)
17:18 wizzyrea the tests will conflict, I tested the function without the unit tests and it works well with 22759 in place. if you try you'll just want to skip the unit tests patch on 22759 for now, til nick gets them resolved
17:19 cait oh,... now i missed ashimema
17:19 cait ashimema: what's for tea? :)
17:22 wizzyrea golly I keep trying to test bug 22225 but... i'm never sure if what I'm seeing is the right behaviour
17:22 huginn` Bug[…]_bug.cgi?id=22225 normal, P5 - low, ---, martin.renvoize, Needs Signoff , Tax hints and prices on may not match
17:23 wizzyrea I have testing waffle sickness - I'm waffling about whether what it's doing is right!
17:23 cait taxes do that to you
17:23 cait I've had this staring at 880 with hebrew today..
17:24 wizzyrea >.<
17:27 lukeG joined #koha
17:28 ashimema Thai yellow curry.. the girls are loving it.. nice the find a Thai curry that's kiddy friendly
17:28 ashimema Our green and red ones are rather spicy 😉
17:29 ashimema As for bug 22225, I kinda took nicks word for it that it was wrong in the first place
17:29 cait i love yellow too :) always a good staple to have at home (curry paste)
17:29 huginn` Bug[…]_bug.cgi?id=22225 normal, P5 - low, ---, martin.renvoize, Needs Signoff , Tax hints and prices on may not match
17:29 corilynn no rush, cait, just wanted to make sure he's tackled all that you asked
17:29 ashimema Code wise it reads like it needs the change I made, but test plan wise I agree it's a pain
17:30 ashimema Especially as the actual numbers dont change retrospectively if I remember rightly.. just the descriptions
17:30 corilynn Maybe I shouldn't try and write instructions while doing the thing I'm writing the instructions for?
17:30 wizzyrea OK GREAT that is where I got hung up I think
17:31 wizzyrea that the numbers weren't changing
17:31 wizzyrea i ws like "should this number not be changing I'm so confused!"
17:32 cait corilynn: we are on a tight deadline currently with feature slush soon :( lots of things to do right now
17:33 cait corilynn: instructions or even better a patch for the manual would be great
17:33 wizzyrea afk a little bit
17:33 ashimema My bad, I should have made that clear in the test plan.  We're cheating in the test plan just to prove the change.. really once set those Prefs should not be changed ever as we don't update the numbers behind the scenes and if we did we'd be breaking all sorts of calculations down the line
17:33 cait i have another feature i'd love to see that i just don't understand how to set up (save fields/subfields from being overwritten by import/z3950)
17:33 ashimema Ta
17:33 cait not touching that one without instructions again :(
17:34 corilynn cait, I mean for a seperate thing I'm doing
17:35 cait ah
17:35 cait sorry :)
17:35 corilynn I'm writing the instructions *while* writing the software, it's not efficent, but when I go back to do the thing it's confusing to recreate
17:35 corilynn because Amazon Web Services instructions are circular
17:36 corilynn i wrote the instructions for bug 18823
17:36 huginn` Bug[…]_bug.cgi?id=18823 enhancement, P5 - low, ---, alex.arnaud, Failed QA , Advanced editor - Rancor - add ability to edit records in import batches
17:41 cait yep true, there is a good attachement there
18:16 oleonard wizzyrea: When I linked to this before you said you thought it didn't need the parent type repeated:[…]?p=15550021303140
18:17 oleonard ...but they're not naturally grouped ,as it is right now
18:17 wizzyrea I still don't but realise that maybe it's an accessibility thing
18:17 oleonard Adding the parent item type to the column would create de facto grouping because of the sorting
18:17 wizzyrea which I think is why it is the way i tis
18:17 wizzyrea it is*
18:18 oleonard Oh sorry it is currently repeating the item type, I forgot
18:23 oleonard If the item type details do not cascade like circulation rules do then the hierarchy isn't as important to the display in item types administration
18:23 cait wizzyrea: if you got a moment - bug 22753
18:23 huginn` Bug[…]_bug.cgi?id=22753 major, P5 - low, ---, koha-bugs, Needs Signoff , Move hold to top button doesn't work if waiting holds exist.
18:23 wizzyrea yep i'm looking at that one :)
18:24 cait cool thx!
18:29 cait 75 *sighs*
18:40 wizzyrea what does "split the hold queue" mean
18:44 wizzyrea anybody?
18:45 wizzyrea nvm I found it
18:45 wizzyrea not the hold queue. The hold queue.
18:45 wizzyrea get it
18:45 cait now you lost me :)
18:45 wizzyrea right?
18:46 cait there is no spelling difference,.. right?
18:46 wizzyrea (there's a syspref, split the holds pending list for a single item by one of several things, and then there's the hold queue, which is the thing you get from the circ home page)
18:46 andreashm joined #koha
18:46 wizzyrea in this case they were referring to the holds pending list for a single item.
18:47 wizzyrea a single biblio
18:47 cait oh
18:47 cait maybe should be rephrased?
18:47 khall joined #koha
18:47 wizzyrea but I don't call that the hold queue :P anyway it's alright
18:47 wizzyrea I figured it out
18:47 * cait waves at khall
18:47 wizzyrea hi khall
18:47 oleonard I've been trying to get this working:[…]?p=15560451934595
18:47 * khall waves
18:47 cait khall: did you see my comment on bug 22761?
18:47 huginn` Bug[…]_bug.cgi?id=22761 enhancement, P5 - low, ---, kyle, Needs Signoff , Move "Fee receipt" from template to a slip
18:48 oleonard ...and it works well except when there's *not* a filter on the DataTable
18:48 khall cait: I can do that
18:48 cait if it makes sense to you
18:48 cait wizzyrea: oleonard maybe you want to chip in?
18:48 cait btw - love seeing this
18:49 cait khall++
18:49 khall thanks! I imagine I'll do the "fee invoice" one too, because it'll be lopsided otherwise :)
18:50 cait I would love that even more :)
18:50 cait my thinking was that we have always slip and email in different letter codes
18:50 cait so far
18:50 khall cait: how about ACCOUNT_CREDIT? Then we can do ACCOUNT_DEBIT for the fee/fine one
18:50 cait so credit is when you pay
18:50 cait payment receipt?
18:51 cait and debit is list of existing fines?
18:51 cait sorry, i still get confused with the newer terminology
18:51 khall cait: yeah, a credit is anything like a payment, writeoff, forgiveness, etc
18:52 wizzyrea oh holy geez
18:52 wizzyrea so when you split the biblio hold priority list
18:52 wizzyrea and you have the numbers set to priority within each table
18:52 wizzyrea it LOOKS like you can change the ordering with the dropdown
18:52 wizzyrea but you can't
18:52 wizzyrea it is vaguely greyed out
18:52 wizzyrea but not enough to make it clear that you can't do that.
18:53 * wizzyrea grumbles
18:53 cait heh
18:53 cait come... look at the slip bu
18:53 cait g
18:53 cait it will cheer you up
18:53 wizzyrea hehe
18:53 wizzyrea just after this
18:53 wizzyrea the thing works it's just been a lil frustrating testing it :)
18:57 oleonard See y'all tomorrow
18:57 wizzyrea by
18:57 wizzyrea w
18:57 wizzyrea lolgdi
18:59 wizzyrea I think that will be lovely
18:59 wizzyrea the work around the slip
18:59 wizzyrea everything that a library could hand to a patron ought to be a notice
18:59 wizzyrea that can be edited
19:05 lukeG joined #koha
19:08 cait wizzyrea: bug 22225 works completely different to what i thought
19:08 huginn` Bug[…]_bug.cgi?id=22225 normal, P5 - low, ---, martin.renvoize, Signed Off , Tax hints and prices on may not match
19:09 wizzyrea how so
19:09 cait I commented
19:09 wizzyrea see I was very confused by that
19:09 cait maybe i am super wrong... but isn't it confusing to display the price differently? especially budgeted cost? if it's budgeted... it hsould not go up or down on receive
19:09 wizzyrea it doesn't change retroactively, I'm told
19:09 cait for me... i think the formula is wrong
19:10 cait not the template
19:10 wizzyrea that could be?
19:10 wizzyrea I really wasn't sure
19:10 wizzyrea but the labels swapped around correctly as far as I could tell even though the prices themselves didn't update as I expected
19:10 cait where are the acq librarians when you need them?
19:10 wizzyrea THAT'S WHAT I'M SAYING
19:11 wizzyrea :)
19:11 wizzyrea anyway i'm going away now
19:11 wizzyrea to get my dog from the groomer
19:15 khall joined #koha
19:18 * ashimema loves the print slip bug too
19:19 cait actually... i know some acq librarians, sending them an email
19:19 cait ashimema: what do you think - new letter code or use old?
19:19 cait well use the letter for the slip too or have separate
19:20 ashimema Not entirely sure yet
19:20 ashimema Still wrapping my head around it
19:21 ashimema As for bug 22225
19:21 huginn` Bug[…]_bug.cgi?id=22225 normal, P5 - low, ---, martin.renvoize, Signed Off , Tax hints and prices on may not match
19:22 ashimema The bug just corrects a mistake of mine regarding when I added the labels in the first place to make it adamantly clear what is being displayed
19:22 ashimema It was my confusion as to what numbers were being displayed in the first place that lead me to add the labels
19:22 ashimema So I totally understand the wtf moment your all having there
19:22 cait i understand
19:22 cait it might only highlight an issue
19:23 ashimema It's also one of the big reasons I want acq to be a focus area during the next couple of cycles
19:23 cait let's see if i can get some comments on the question
19:23 ashimema Exactly.. I think it highlights and issue.. I bet people have been misunderstanding it for years
19:25 ashimema I reckon it's a Pandora's box of subtle but important bugs
19:25 cait probably
19:25 wahanui probably is too hard.
19:26 ashimema We have customers that want allot of enhancements in acq but the foundations there really scare me... I've not found anyone that fully understands it end to bed yet
19:37 cait I though I did... until an hour ago
19:39 * cait creates qa branch 100 for this month :D
19:41 ashimema Yowsers
19:42 cait which is a problem... not they no longer sort right
19:42 cait i started with 01
19:44 ashimema Lol
19:44 * ashimema wonders how you qa
19:46 ashimema Be interesting to go back to basics and retrain on QA some time.. it's been a while since I've reassessed my own routines.. they get minor adaptations and additions frequently..
19:46 cait we can compare
19:46 cait i have a naming scheme for branches
19:46 cait q01-xxxxx-keywords
19:46 cait s01.... for sign offs
19:46 cait m01... for my own patches
19:47 cait i clean them up once a month
19:48 cait apply... run the qa script while i read the bug and test plan
19:48 ashimema Interesting
19:49 ashimema I nearly always start by reading the code and test plan.. see if I spot any immediate fails
19:49 cait then read the code... see where i might want to run off the test plan
19:49 cait test
19:49 cait ... comment... etc :)
19:49 ashimema The I'll checkout a bug_xxxx branch and apply the patch to that
19:50 ashimema I test, run QA scripts, write minor followups and add signoff lines, attach, comment then throw the branch away
19:51 ashimema If it fails I comment why and similarly throw away the branch
19:51 cait i keep the branch arund a bit, so i can get back to where i was easier
19:51 cait also helps me to remember where i have been and need to finish
19:53 ashimema I also more recently have got into the habit of running the whole test suit against any of the more complicated ones.. I used to run just the obvious affected tests
19:53 ashimema But I've been bitten enough now that I try to run the whole lot
19:54 cait it's good
19:54 cait but it takes a while
19:54 ashimema I try to use Bugzilla assignee, QA etc fields to keep track . I do however leave some branches around.. but I find I often forget what I was doing anyway to end up nuking them and reapplying anyway
19:54 cait so that's why apply first and read while they the tests run :)
19:55 cait maybe i just like to see the list grow
19:55 cait gamification etc
19:55 ashimema Yeah.. I also use git worktree
19:55 cait what is that?
19:55 wahanui that is wrong - it's about 1/2 that (about 8C) + raining!
19:55 cait wahanui: forget that
19:55 wahanui cait: I forgot that
19:55 cait what is that?
19:57 ashimema So I always have two different branches checked out at a time.. one directory pointed to but koha-testing-docker so I can run the test suit against any branch whilst I'm working in another branch doing other cosing
19:57 ashimema Helps me multitask
19:57 ashimema I love being able to have two branches checked out
19:57 cait sounds nice :)
19:57 cait i still have to give koha-testing-docker a go
19:59 ashimema It needs a fair chunk of space.. but it's so nice to be able to run tests consistently
19:59 ashimema And it's actually really simple to setup and use
20:15 cait maybe show me in dublin? :)
20:17 ashimema Sure
20:18 * ashimema wishes Dublin was longer.. feels like I have so much I want to fit in
20:20 cait sam
20:20 cait e
20:20 cait maybe we can do a thing sometime later if we don't manage
20:22 andreashm joined #koha
20:39 khall joined #koha
20:43 cait oh no
20:43 cait just missed khall
20:43 cait wizzyrea: if i enter a max total checkouts on all libraries... sohuld that show up for all libraries now?
20:51 andreashm joined #koha
21:08 reiveune bye
21:08 reiveune left #koha
21:25 andreashm joined #koha
21:34 rangi ashimema: are you about?
21:35 cait bit late :)
21:35 cait hm a little earlier than here
23:39 papa joined #koha

| Channels | #koha index | Today | | Search | Google Search | Plain-Text | plain, newest first | summary