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/en/modules/tools/koha-news.tt: [% 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 http://bugs.koha-community.org[…]_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 http://bugs.koha-community.org[…]_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 http://bugs.koha-community.org[…]_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 http://bugs.koha-community.org[…]_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 http://bugs.koha-community.org[…]_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 http://bugs.koha-community.org[…]_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 http://bugs.koha-community.org[…]_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 http://bugs.koha-community.org[…]_bug.cgi?id=21784 enhancement, P5 - low, ---, oleonard, Needs Signoff , Clean up js_includes.inc |
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 http://bugs.koha-community.org[…]_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 |
https://www.mrfood.com/Pizza/R[…]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: https://www.wikihow.com/Make-a-Ramen-Pizza |
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 http://bugs.koha-community.org[…]_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 http://bugs.koha-community.org[…]_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 http://bugs.koha-community.org[…]_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 http://bugs.koha-community.org[…]_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 http://bugs.koha-community.org[…]_bug.cgi?id=22225 normal, P5 - low, ---, martin.renvoize, Needs Signoff , Tax hints and prices on orderreceive.pl 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 http://bugs.koha-community.org[…]_bug.cgi?id=22225 normal, P5 - low, ---, martin.renvoize, Needs Signoff , Tax hints and prices on orderreceive.pl 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 http://bugs.koha-community.org[…]_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: https://zivotdesign.com/p/view[…]?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 http://bugs.koha-community.org[…]_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: https://zivotdesign.com/p/view[…]?p=15560451934595 |
18:47 |
|
* khall |
waves |
18:47 |
|
cait |
khall: did you see my comment on bug 22761? |
18:47 |
|
huginn` |
Bug http://bugs.koha-community.org[…]_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 http://bugs.koha-community.org[…]_bug.cgi?id=22225 normal, P5 - low, ---, martin.renvoize, Signed Off , Tax hints and prices on orderreceive.pl 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 http://bugs.koha-community.org[…]_bug.cgi?id=22225 normal, P5 - low, ---, martin.renvoize, Signed Off , Tax hints and prices on orderreceive.pl 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 |