| 04:50 |
pinesol_green |
Incoming from qatests: Test Success - http://testing.evergreen-ils.org/~live/test.html <http://testing.evergreen-ils.org/~live/test.html> |
| 07:14 |
|
graced joined #evergreen |
| 07:35 |
|
rjackson_isl joined #evergreen |
| 07:37 |
|
Arlene joined #evergreen |
| 11:00 |
|
sciani joined #evergreen |
| 11:04 |
|
mllewellyn joined #evergreen |
| 11:17 |
dbwells |
kmlussier: In my opinion, cases like this one depend on the level of changes to the code. If the changes are relatively simple, I'd backport it, but if they are more fundamental, it's probably really a new feature. I know that just kicks the can a bit, but it's the best I've got. |
| 11:19 |
kmlussier |
dbwells: I have a branch at working/user/kmlussier/lp1373203-let-users-escape-metarecord-holds. I just haven't had a chance to do anything beyond light testing, but should be able to post it to LP later today. |
| 11:30 |
|
dreuther_ joined #evergreen |
| 11:34 |
* dbs |
always likes getting "System was down yesterday so we recorded circulations in Word" messages |
| 11:35 |
dbs |
(first, local network was down, the library system was fine--but I get what they mean; second, we have an offline client! Jeez.) |
| 11:53 |
tsbere |
I can at least understand "our local network was down so we couldn't even log into the domain, so we recorded circs on the laptop that doesn't have Evergreen on it in Word" |
| 11:58 |
|
BigRig_ joined #evergreen |
| 12:03 |
dbs |
but you don't even need to log into the domain for standalone, at least not if you've connected in the past |
| 12:03 |
berick |
phasefx: curious if you tested commit e45a691 in google chrome ("add a Retrieve All These Patrons button") |
| 12:04 |
berick |
w/ more than one user selected |
| 12:05 |
berick |
phasefx: i ask becuase .. search down for "opening multiple tabs" in http://wiki.evergreen-ils.org/doku.php?id=dev:browser_staff:dev_notes&s[]=browser&s[]=client |
| 12:06 |
phasefx |
berick: I did test it. Chrome offered an option for allowing the web page to open multiple tabs.. it showed up as an icon on the location bar |
| 12:07 |
goood |
berick: yeah, it's the "enable pop-ups for this site" feature that allows opening multiple tabs |
| 12:07 |
berick |
crazy, is that new? |
| 12:07 |
berick |
either way.. cool! |
| 15:32 |
kmlussier |
There is this: http://wiki.evergreen-ils.org/doku.php?id=dev:browser_staff:dev_sprints |
| 15:32 |
* kmlussier |
was just looking for that info |
| 15:33 |
|
gdunbar joined #evergreen |
| 15:33 |
bshum |
gmcharlt: I tested the warning but I think it gave me an uncaught exception error in my console rather than displaying me anything |
| 15:33 |
bshum |
I'm double checking that. |
| 15:33 |
bshum |
Ah nevermind there it is |
| 15:34 |
bshum |
Testing by trying to delete "Stacks" for the demo concerto data just takes awhile to figure out that hey, maybe you can't do that. |
| 15:34 |
bshum |
I got the exception popup finally after a nonresponsive JS waiting |
| 15:34 |
Dyrcona |
kmlussier: Thanks muchly. |
| 15:35 |
Dyrcona |
I was getting documentation sprints and list archives. |
| 15:35 |
jboyer-isl |
bshum: That check is pretty basic and could probably be improved by studying some SQL. (Or even throwing a LIMIT 1 on the end to get the Q planner to stop being so picky) |
| 16:04 |
gmcharlt |
bshum: and bug 1424827 now exists, but just as a wishlist item |
| 16:04 |
pinesol_green |
Launchpad bug 1424827 in Evergreen "logically deleted copy locations should be accessible in copy location editor" (affected: 1, heat: 6) [Wishlist,New] https://launchpad.net/bugs/1424827 |
| 16:06 |
|
ericar_ joined #evergreen |
| 16:08 |
bshum |
gmcharlt++ # I'll go ahead and finish testing and push those working fixes to master in a bit. |
| 16:21 |
|
dreuther_ joined #evergreen |
| 16:24 |
|
akilsdonk joined #evergreen |
| 16:53 |
berick |
looking at working/collab/miker/web-client-28-rebase, which is browser client bug fixes, w/ a dash of features. |
| 04:22 |
|
Stompro joined #evergreen |
| 04:43 |
|
StomproJ joined #evergreen |
| 04:55 |
pinesol_green |
Incoming from qatests: Test Success - http://testing.evergreen-ils.org/~live/test.html <http://testing.evergreen-ils.org/~live/test.html> |
| 06:00 |
|
StomproJosh joined #evergreen |
| 07:21 |
|
graced joined #evergreen |
| 07:40 |
|
rjackson_isl joined #evergreen |
| 13:30 |
kmlussier |
eeevil++ gmcharlt++ |
| 13:30 |
kmlussier |
The action/trigger based messages seem to be working now. |
| 13:31 |
kmlussier |
Of course, I need to remind myself to wait for the delay. I almost filed a bug report saying they still weren't working. ;) |
| 13:32 |
berick |
kmlussier: let me know when you wrap up your testing. |
| 13:32 |
berick |
i'll do a final sweep and merge |
| 13:32 |
kmlussier |
berick: Will do |
| 13:33 |
* berick |
thinks this is a cool feature |
| 13:33 |
kmlussier |
Me too! :) |
| 13:47 |
gmcharlt |
kmlussier: yeah, I dropped the A/T delay down to 10 seconds in the test database :) |
| 13:47 |
kmlussier |
Apparently I was a bit impatient. :) |
| 13:55 |
jonadab |
Hmm... for current git Evergreen, is it recommended to upgrade to Apache 2.4, if the OS comes with 2.2 (e.g., wheezy)? |
| 13:56 |
csharp |
jonadab: nope |
| 15:53 |
dbwells |
berick: I can confirm both the bug and fix. berick++ |
| 15:53 |
jeff |
does this branch still try to "pay" bills out of order, or has it moved away from that? |
| 15:53 |
jeff |
an early version tried to "pay" (when simulating/checking) "larger" bills first |
| 15:54 |
dbwells |
The branch being tested right now doesn't try to do anything at all, it is strictly refactoring. |
| 15:54 |
jeff |
ah! i had lost context. thanks! |
| 15:55 |
jeff |
(checking in from a service desk with numerous distractions, i still have my coat on -- thanks for indulging a silly question) |
| 15:55 |
dbwells |
No problem, it is confusing, and who has time to read the 77 comments therein. |
| 15:58 |
kmlussier |
berick: I've finished up testing the message center for today. Sign-offs are forthcoming. |
| 15:58 |
berick |
kmlussier++ |
| 16:01 |
berick |
dbwells: Dyrcona: what's the situation w/ bug 1198465 -- if the fine-gen branch is merged, is the rest basically ready to go? or does that all need more testing, sign-off's, etc.? |
| 16:01 |
pinesol_green |
Launchpad bug 1198465 in Evergreen "Support for Conditional Negative Balances" (affected: 15, heat: 70) [Wishlist,Confirmed] https://launchpad.net/bugs/1198465 |
| 16:02 |
Dyrcona |
I'll let dbwells answer that. It's bascially all his now. |
| 16:02 |
berick |
*wistfully* it's more machine than man now |
| 16:03 |
kmlussier |
berick: There was still a bug with that branch the last time I tested that was supposed to be fixed by the fine-gen branch. I'm guessing it should be checked to see if it really is fixed. |
| 16:04 |
dbwells |
berick: I would need to rebase again, then testing and sign-offs still needed, too. And yes, the branch as it now exists is twisted, and evil. |
| 16:04 |
berick |
heh |
| 16:04 |
|
artunit joined #evergreen |
| 16:05 |
kmlussier |
The bug that needed to be fixed was related to generating new overdue fines on a lost checkin. The way I do my testing is very manual and requires that I do it over a few days to check that scenario. Not sure if you all have fancier ways of testing it. |
| 16:05 |
berick |
so, i really like the fine-gen changes. i think that will help a lot. i'm ready to sign off on those (and maybe do some squashing) unless there are any objections. |
| 16:06 |
dbwells |
berick: the goal in separating out the cstore part was to make it easier to test, so hopefully that will be worth it. I'll rebase the other stuff to that latest branch and see how far off we are from at least having working code again. |
| 16:06 |
berick |
dbwells: let me add sign-off's and squash and push another branch before you rebase. cool? |
| 16:07 |
dbwells |
sure, thanks |
| 16:11 |
Dyrcona |
Billing is twisted and evil, but it's what we got for now. |
| 05:54 |
* kmlussier |
yawns |
| 06:16 |
kmlussier |
Looks like bug 1350371 has had a signoff for a while if anyone feels inclined to merge it. :) |
| 06:16 |
pinesol_green |
Launchpad bug 1350371 in Evergreen "ACQ improved duplicate order detection" (affected: 1, heat: 8) [Wishlist,New] https://launchpad.net/bugs/1350371 |
| 06:20 |
kmlussier |
Ooh! I missed bug 1379815 when it was first added. I'll have to add it to my testing list for today. |
| 06:20 |
pinesol_green |
Launchpad bug 1379815 in Evergreen "Assign stat cats during Vandelay import/overlay of items" (affected: 1, heat: 8) [Wishlist,New] https://launchpad.net/bugs/1379815 |
| 06:48 |
|
berick joined #evergreen |
| 07:31 |
|
graced joined #evergreen |
| 08:53 |
|
maryj joined #evergreen |
| 08:55 |
eeevil |
kmlussier / Dyrcona: working/user/miker/lp1251394-metabib-display-fields updated |
| 08:55 |
kmlussier |
eeevil++ |
| 08:56 |
eeevil |
either rebasing shifted the code around and I missed it ... or, the baseline schema was never tested -- only the upgrade version was |
| 09:02 |
|
maryj joined #evergreen |
| 09:23 |
|
julialima_ joined #evergreen |
| 09:25 |
kmlussier |
eeevil: Still getting the schema does not exist messages, but the initial error is a bit different this time: |
| 10:54 |
Dyrcona |
dbwells: Good suggestion. I'll do that for now until we decide what to do about point release notes. |
| 10:54 |
kmlussier |
It could just be a matter of prettying up the change logs for each point release. |
| 10:54 |
Dyrcona |
Well, I think security fixes should get extra attention. |
| 10:57 |
kmlussier |
eeevil: Still no luck on the metabib display fields branch. Also, I don't know if the person who was planning to test it will be able to do it for tomorrow. Just wanted to let you know in case there are other things you need to do. |
| 10:57 |
kmlussier |
Latest error is: psql:002.schema.config.sql:188: ERROR: function config.metabib_representative_field_is_valid(integer, text) does not exist |
| 10:57 |
kmlussier |
HINT: No function matches the given name and argument types. You might need to add explicit type casts. |
| 10:58 |
|
dreuther_ joined #evergreen |
| 10:59 |
eeevil |
kmlussier: ah... yeah, there's more to move around. Thanks for looking at it. I won't have tuits to get back to it today, sorry. if there's a way to use the upgrade script on an existing master db, that should get you there, but I don't know if that's easy in your setup |
| 10:59 |
|
sandbergja joined #evergreen |
| 11:00 |
kmlussier |
eeevil: OK, I'll just add that error to the LP bug then so it doesn't get lost. |
| 11:00 |
eeevil |
thanks much! |
| 11:00 |
dbwells |
I am also going to make time to test the metabib display fields before the cutoff, since I'm at least partially responsible that it's been delayed this long. Must make amends! |
| 11:03 |
pinesol_green |
[evergreen|Steven Chan] LP#1418772: Avoid internal server error on viewing full record when copy create_date is null - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=55aaea8> |
| 11:03 |
pinesol_green |
[evergreen|Galen Charlton] LP#1418772: (follow-up) tweak undef-edness check - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=fcaf1cc> |
| 11:12 |
|
jwoodard joined #evergreen |
| 13:30 |
krvmga |
in the hen story, the animals who come later don't get to eat the cake; only the hen and her chicks :) |
| 13:30 |
yboston |
BTW, at some point we shuold ask opinions fromt he whole community |
| 13:31 |
yboston |
but for example that can be done after we have had a secodn meeting to talk about re-orgs |
| 13:31 |
krvmga |
yboston: in that same line, i think, at some point, we should "play test" the docs with our expected end users. |
| 13:31 |
remingtron |
yboston: might be good to have a proposed outline to get feedback on |
| 13:31 |
sandbergja |
yboston: I like the sound of that -- I think we would get more feedback if we send out something concrete |
| 13:32 |
yboston |
I prefer to first have a clear list of things that we want to adress in the re-org and the minimun requirements for the re-org. Though both can be revised with time |
| 13:34 |
remingtron |
jihpringle: not that soon |
| 13:34 |
yboston |
yboston: I am concerned that we might not be ready to support seperated docs yet |
| 13:34 |
remingtron |
this is just a brainstorming meeting, so it may take a while to slowly change |
| 13:35 |
yboston |
I would like to run some tests to make sure we have the correct automated worlflow first |
| 13:35 |
krvmga |
yboston: that sounds good |
| 13:35 |
remingtron |
yboston: right, it would require changes to the docs building process on the server |
| 13:36 |
|
Arlene joined #evergreen |
| 13:55 |
yboston |
krvmga++ |
| 13:57 |
jihpringle |
afterl: I've created the comments/complaints wiki page (nothing on it yet) http://evergreen-ils.org/dokuwiki/doku.php?id=evergreen-docs:reorg_2014:comments |
| 13:59 |
bbqben |
Hi all - EOB mtg starting here in a couple of minutes. cheers! |
| 13:59 |
kmlussier |
remingtron: I quick question before the next meeting starts. |
| 14:00 |
kmlussier |
Or maybe dbwells is the best person to answer it. |
| 14:00 |
kmlussier |
I'm going to test bug 1379815 |
| 14:00 |
pinesol_green |
Launchpad bug 1379815 in Evergreen "Assign stat cats during Vandelay import/overlay of items" (affected: 1, heat: 8) [Wishlist,New] https://launchpad.net/bugs/1379815 |
| 14:00 |
dbwells |
yay :) |
| 14:00 |
|
buzzy joined #evergreen |
| 14:06 |
bbqben |
#topic OPW project update |
| 14:07 |
yboston |
#info yboston is Yamil Suarez @ Berklee College of Music |
| 14:07 |
dbwells |
Good afternoon everyone. The OPW group gave a brief progress report at the dev meeting a couple days ago, and we were invited to do the same for the EOB, and perhaps answer any questions you might have. Our intern julialima_ has joined us today to give the update. |
| 14:07 |
julialima_ |
Good afternoon! We have been working very hard and we have made a lot of progress. |
| 14:07 |
julialima_ |
We are focused, mainly, in ensuring consistency and providing the best user experience we can. Of course it is a working progress, we are still testing some ideas and thinking new solutions for different situations. We still have 3 weeks until my internship is finished so we have a lot of time to try new things. |
| 14:07 |
julialima_ |
You can find the UI style guide in https://github.com/JuliaLima/Evergreen/tree/patch-1/docs/style_guide, remember that we are working on it, so nothing is definitive; and also you can check my blog for updates about our progress: http://lima-julia.tumblr.com/EG-style-guide. |
| 14:08 |
julialima_ |
Feel free to contact us and give your opinion and ideas, we need your feedback in order to improve our work, it is very important for us. |
| 14:08 |
dbwells |
julialima_ also posted a similar update to the mailing list a few days ago, so that might be the best overall venue for feedback at this point. |
| 14:09 |
kmlussier |
julialima++ dbwells++ |
| 14:09 |
bbqben |
julialima++ dbwells++ |
| 16:24 |
pinesol_green |
[evergreen|Dan Scott] lp1422555: Link permalink icon on record summary - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=4b30fb8> |
| 16:25 |
bshum |
jboyer-isl: That sounds... "special" |
| 16:25 |
bshum |
dbs++ kmlussier++ |
| 16:27 |
kmlussier |
Hmmm...is bug 1287370 considered new feature or bug fix? I just want to know if I should worry about testing it by tomorrow or not. |
| 16:27 |
pinesol_green |
Launchpad bug 1287370 in Evergreen "Funds Go Back Option Takes User Back to the First Fund Screen" (affected: 5, heat: 26) [Low,Confirmed] https://launchpad.net/bugs/1287370 |
| 16:28 |
dbs |
bug 1423585 is a nice, lightweight feature :) |
| 16:28 |
pinesol_green |
Launchpad bug 1423585 in Evergreen "TPAC discoverability: include Open Graph Protocol markup for bonus points" (affected: 1, heat: 6) [Wishlist,New] https://launchpad.net/bugs/1423585 |
| 16:31 |
bshum |
dbs: That sounds fancy. |
| 16:31 |
dbs |
I say it's a bug fix |
| 16:31 |
bshum |
So let it be done. |
| 16:32 |
kmlussier |
OK, well if I have time I'll test it anyway because our people would like to see it. And who knows when I'll get back to testing. |
| 16:32 |
kmlussier |
But there is always bug squashing day. :) |
| 16:54 |
|
dreuther joined #evergreen |
| 16:54 |
jonadab |
dbs: That _looks_ like it could just be a static background image with text added. Probably could be done with imagemagick. |
| 16:55 |
jonadab |
That's a guess, though. |
| 16:58 |
|
mdriscoll left #evergreen |
| 16:58 |
|
yboston left #evergreen |
| 17:02 |
|
mmorgan left #evergreen |
| 17:12 |
pinesol_green |
Incoming from qatests: Test Success - http://testing.evergreen-ils.org/~live/test.html <http://testing.evergreen-ils.org/~live/test.html> |
| 17:28 |
pinesol_green |
[evergreen|Galen Charlton] LP#1378169: make context OU selector on funds page sticky - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=8e8c2a8> |
| 17:31 |
|
dreuther_ joined #evergreen |
| 17:34 |
|
dreuther__ joined #evergreen |
| 18:27 |
|
Arlene joined #evergreen |
| 05:14 |
pinesol_green |
Incoming from qatests: Test Success - http://testing.evergreen-ils.org/~live/test.html <http://testing.evergreen-ils.org/~live/test.html> |
| 07:31 |
|
graced joined #evergreen |
| 07:39 |
|
rjackson_isl joined #evergreen |
| 07:55 |
|
graced joined #evergreen |
| 09:13 |
bshum |
They're just deprecation warnings. Not errors. |
| 09:14 |
bshum |
krvmga: With autoreconf -i you only do that on git installs, from a tarball OpenSRF, it's not a necessary step. |
| 09:15 |
bshum |
Might be why you never noticed before? Cause it's been doing those little warnings for years now. |
| 09:18 |
StomproJ |
Has anyone tested/used one of the new intel DDR4 memory based servers for postgresql? Does the increased memory bandwidth give a big boost in performance? |
| 09:18 |
bshum |
There's DDR4? :\ |
| 09:18 |
* bshum |
goes back to living under his rocks |
| 09:19 |
bshum |
StomproJ: Might be an interesting question to ask over in the #postgresql channel too :) |
| 10:17 |
pinesol_green |
[evergreen|Remington Steed] LP#957466: Update editor/edit_date/source on overlay - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=62510da> |
| 10:18 |
pinesol_green |
[evergreen|Jason Stephenson] LP#957466 Vandelay set the 905$u on imported bib records to current user. - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=522b6ff> |
| 10:18 |
pinesol_green |
[evergreen|Jason Stephenson] LP#957466: Stamping Upgrade Script - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=fe07338> |
| 10:43 |
jeff |
I've noticed an odd (to me) difference between xulrunner and webstaff, and I think it comes down to pcrud permissions vs open-ils.circ api permissions: |
| 10:47 |
jeff |
given a patron with open circulations at the CONS level (i can't recall offhand if those were generated by concerto or if that was me manually checking things out without having a workstation set), users without the EVERYTHING permission can't see the circs in the web client. |
| 10:47 |
jeff |
but they can in the xulrunner client. |
| 10:47 |
jeff |
I haven't yet given it a more realistic test (usually you wouldn't have circs at CONS, etc) |
| 10:48 |
berick |
jeff: does your login have global VIEW_CIRCULATIONS permissions? |
| 10:48 |
jeff |
And I'm going to take the opportunity to refresh/enhance my understanding of permissions in general, but I wasn't able to make the circs show in the web client even when making someone global admin and assigning them working locations across the org tree. |
| 10:49 |
jeff |
berick: i'll verify that specifically. one set. |
| 13:32 |
kmlussier |
yboston: I'm here now |
| 13:32 |
collum |
Just got it buzzy |
| 13:32 |
csharp |
buzzy: I got it at 1:29 p.m. |
| 13:34 |
yboston |
kmlussier: I sent you an email a little earlier. just letting you know I am building a Vm to test auth overlay. Also that, I if you had time/VM, that I have some auth code needing a sign-off bug 1403967 |
| 13:34 |
pinesol_green |
Launchpad bug 1403967 in Evergreen "Display 'subject heading thesaurus' value in Manage Authorities results" (affected: 1, heat: 6) [Undecided,New] https://launchpad.net/bugs/1403967 |
| 13:35 |
buzzy |
great, thanks. i just did, too. for some reason, it didn't go through the first time |
| 13:37 |
|
mmorgan joined #evergreen |
| 13:41 |
|
maryj_ joined #evergreen |
| 13:42 |
kmlussier |
yboston: OK, good. I'm a little behind in my testing this week. |
| 13:43 |
kmlussier |
yboston: I can take a look at your code. |
| 13:44 |
|
maryj joined #evergreen |
| 13:53 |
jeffdavis |
jonadab: you are correct - passwords stored in the db are hashed but not salted |
| 13:55 |
Dyrcona |
Salt is applied when a user authenticates, but if someone grabs your database, you may have bigger problems than worrying about the user's password hashes. |
| 15:15 |
* phasefx |
found a wish for this earlier when trying to use Consortium as a pickup lib.. wasn't obvious to me at first what was happening |
| 15:16 |
jonadab |
Having the page return a partly-filled form with an informative error about the missing required field(s) is what is what I would probably do, _possibly_ with optional Javascript designed to pre-empt the submit if Javascript is working ok, letting the server-side stuff catch it if not. |
| 15:18 |
kmlussier |
phasefx: You can select the consortium as a pickup library? |
| 15:19 |
phasefx |
kmlussier: I'm sure it was an admin user on a test system, maybe with the consortium as a the workstation lib. Once deselected, you can't reselect it |
| 15:19 |
kmlussier |
jonadab: The problem with the optional Javascript is that we've tried to keep it out of the catalog unless it's aboslutely necessary. |
| 15:19 |
phasefx |
but the user had no sane default for whatever reason |
| 15:19 |
kmlussier |
phasefx: Ah, ok. |
| 15:39 |
pinesol_green |
Launchpad bug 1366026 in Evergreen "Add Copy Active Date to Staff Client Display" (affected: 4, heat: 24) [Wishlist,Incomplete] https://launchpad.net/bugs/1366026 |
| 15:39 |
jboyer-isl |
lp 1210541 |
| 15:40 |
jboyer-isl |
Drop the comma or add the space, who knows where things went off. |
| 15:47 |
yboston |
berick: am I wrong, or did your authority overlay code get signed off and merged to master? Should I still test off of master or should I test from your rebased branch? |
| 15:48 |
kmlussier |
jboyer-isl: Hmmm...I was just discussing it with mmorgan. I think there's a fine line between bug fix and enhancement, but if it isn't mentioned in the release notes, then all the people who gave up on deleting copy locations won't know they can delete them now. |
| 15:49 |
dbs |
Bug fixes should have release notes entries too! |
| 15:49 |
goood |
yboston: it got pushed today, you are not wrong. testing on master is best now |
| 15:49 |
jboyer-isl |
Sigh. I suppose that is true. (dbs's point includeD) |
| 15:49 |
dbs |
lots of bug fixes listed in http://www.postgresql.org/docs/9.4/static/release-9-4-1.html :) |
| 15:50 |
* dbs |
promises to start writing now overdue release notes for TPAC features |
| 16:45 |
hopkinsju |
Right. One favors readability, the other writeablilty I suppose |
| 16:56 |
dbs |
bshum: aw yeah, that's how I roll. I'll fix it |
| 16:56 |
bshum |
dbs++ |
| 16:57 |
pinesol_green |
Incoming from qatests: Test Success - http://testing.evergreen-ils.org/~live/test.html <http://testing.evergreen-ils.org/~live/test.html> |
| 17:04 |
|
vlewis joined #evergreen |
| 17:05 |
|
vlewis joined #evergreen |
| 17:09 |
|
mdriscoll left #evergreen |
| 17:11 |
|
mmorgan left #evergreen |
| 17:13 |
pinesol_green |
[evergreen|Dan Scott] Fix typo in release notes for TPAC discoverability - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=c0fabd0> |
| 17:15 |
|
wlayton joined #evergreen |
| 17:17 |
* bshum |
waves at wlayton :) |
| 17:24 |
|
mrpeters left #evergreen |
| 18:00 |
dbs |
berick: fantastic! |
| 18:03 |
berick |
certainly adds pressure to angular-ize (i.e. websocket-ize) legacy UI's |
| 18:03 |
ldw |
MrBaggins: I sent a reports howto to the mailing list in 2011, so some of it may not be applicable now, you can find it here: http://georgialibraries.markmail.org/message/ptlz36kle63spqqb?q=Liam+report+date:201109+ |
| 18:04 |
berick |
hm, patron reg throws the same warning, but still renders OK |
| 18:04 |
berick |
perhaps it's only deprecated, but not yet prevented, and vandelay is just broken for some other reason |
| 18:05 |
berick |
oh well, more testing later |
| 18:06 |
dbs |
all I've been seeing is deprecation notices, but I thought maybe you were on canary or some such advance build |
| 18:12 |
MrBaggins |
Thanks for that ldw, every bit helps |
| 18:23 |
jonadab |
As far as asciidoc and people wanting .txt files so they can double click, it should be straightforward to make the build process turn the former into the latter. |
| 00:03 |
|
mrpeters joined #evergreen |
| 00:07 |
|
mrpeters left #evergreen |
| 04:59 |
pinesol_green |
Incoming from qatests: Test Success - http://testing.evergreen-ils.org/~live/test.html <http://testing.evergreen-ils.org/~live/test.html> |
| 07:16 |
|
graced joined #evergreen |
| 07:27 |
kmlussier |
@coffee |
| 07:27 |
* pinesol_green |
brews and pours a cup of Ka'u Coffee Natural Dried, and sends it sliding down the bar to kmlussier |
| 15:01 |
dbwells |
#info dbwells = Dan Wells, Hekman Library (Calvin College) |
| 15:01 |
remingtron |
#info remingtron = Remington Steed, Hekman Library (Calvin College) |
| 15:01 |
phasefx |
#info phasefx = Jason Etheridge, ESI |
| 15:02 |
bshum |
Alrighty, well, as others come along, feel free to add yourselves. We'll move on for now. |
| 15:03 |
bshum |
#topic Action items from last meeting |
| 15:03 |
bshum |
#info dbwells will separate the cstore billing code into a separate bug; jeff and berick will help with testing |
| 15:03 |
kmlussier |
#info kmlussier is Kathy Lussier - MassLNC |
| 15:03 |
bshum |
Any report on that? |
| 15:03 |
berick |
#info dbwells posted collab/dbwells/1198465_cstore_fines_gen -- I have not done any testing yet |
| 15:04 |
jeff |
i can confirm that dbwells has a working branch with said separation and that i'm still planning to test -- sooner than later. |
| 15:04 |
eeevil |
#info eeevil = Mike Rylander, ESI |
| 15:04 |
Bmagic |
#info bmagic is Blake GH - MOBIUS |
| 15:04 |
dbwells |
I did split it, but I also did a bad job at bothering people about it. |
| 15:04 |
eeevil |
(I added a couple things to the agenda at the last moment ... sorry) |
| 15:04 |
bshum |
Alrighty, I think we can leave it off the meeting action items. Hopefully you guys can test it out before the beta cutoff? :) |
| 15:04 |
kmlussier |
We have it loaded on a test server at MassLNC, but I've only looked at it quickly. I'm planning to take a closer look this week. |
| 15:05 |
bshum |
#info Testing to continue. |
| 15:05 |
berick |
one sec... |
| 15:05 |
dbwells |
kmlussier: I did push an update earlier today to fix penalty generation. Just FYI. |
| 15:05 |
kmlussier |
dbwells: OK, thanks. I' |
| 15:13 |
gmcharlt |
but I want to ask others: any other pending stuff that folks would want to target for a March release? |
| 15:13 |
* bshum |
assumes the pound proxy example config is in reference to getting websockets happier with standard ports? |
| 15:13 |
gmcharlt |
yep |
| 15:14 |
bshum |
Sweet, gmcharlt++ # looking forward to testing those things out. |
| 15:15 |
bshum |
#info gmcharlt with new WIP for improving osrf_control --diagnostic and pound proxy example config for websockets in upcoming OpenSRF release. |
| 15:16 |
bshum |
... anything else on OpenSRF? |
| 15:16 |
bshum |
Do we want to assign an action for 2.4.1 cutting? |
| 15:17 |
|
bibliophylum joined #evergreen |
| 15:17 |
gmcharlt |
bshum: go for it |
| 15:17 |
gmcharlt |
you have... THE POWER |
| 15:38 |
gmcharlt |
berick: well, a quick review to be clear |
| 15:38 |
Bmagic |
Im sure no one looked at this one https://bugs.launchpad.net/evergreen/+bug/1174498 |
| 15:38 |
pinesol_green |
Launchpad bug 1174498 in Evergreen "Payment by billing type breakdown" (affected: 7, heat: 38) [Wishlist,In progress] - Assigned to Jeff Godin (jgodin) |
| 15:38 |
bshum |
jeffdavis: I actually got my Overdrive API test key last week. So I might look more closely soon, but not sure if it'll be in time for Friday :\ |
| 15:38 |
gmcharlt |
jeffdavis: I'd look at it when you mentioend it (last month? how time flies); have you made any tweaks since then? |
| 15:39 |
jeff |
Bmagic: i have concerns about payment by billing type interaction with some other billing changes -- it currently can be wrong about things with regard to future-dated overdues. |
| 15:39 |
jeffdavis |
gmcharlt: no, no changes since then |
| 15:42 |
gmcharlt |
jeffdavis: I think some concerns about work-for-hire were expessed -- but I think that's OK for it living as a contrib |
| 15:42 |
kmlussier |
We also have some people lined up here to look at 1410369. |
| 15:42 |
bshum |
In the interests of finishing up this meeting, I'm going to ask us to delay discussions on specific features / requests for 2.8 post-meeting. I'll give it a moment to wrap up last typing. |
| 15:42 |
gmcharlt |
berick: you've got a deal; I'll start testing that tomorrow |
| 15:43 |
berick |
gmcharlt++ # point me at the branch when it's published |
| 15:43 |
bshum |
berick: Anything else about 2.8 beta before we move on? |
| 15:43 |
gmcharlt |
berick: will do |
| 15:43 |
berick |
and kmlussier's testing is welcome :) |
| 15:43 |
berick |
bshum: one last thing. |
| 15:43 |
kmlussier |
berick: The more the merrier. :) |
| 15:44 |
berick |
i want to reiterate that any testing we can do with master between Friday and next Wednesday (beta release) is greatly apprecaited |
| 15:44 |
berick |
to reduce the amount of beta bugs |
| 15:44 |
berick |
or bug fix merges, for that matter |
| 15:45 |
berick |
i'll try to finalize the DB upgrade script well before wednesday, too, for eyes |
| 15:45 |
jeff |
berick++ |
| 15:45 |
bshum |
#info Beta release cut on Wednesday, February 25 |
| 15:45 |
bshum |
#help Please help test between beta deadline and cut to minimize bugs in beta. And also help with bug fixes in general. |
| 15:46 |
bshum |
berick++ |
| 15:46 |
bshum |
Alrighty. |
| 15:46 |
bshum |
#topic OPW Progress Report |
| 15:46 |
dbwells |
The OPW group would like to update the community on the progress of the Evergreen Style Guide project. Our intern julialima_ has joined us today to give the update. |
| 15:47 |
julialima_ |
We have been working very hard and we have made a lot of progress. |
| 15:47 |
julialima_ |
We are focused mainly in ensuring consistency and providing the best user experience we can. Of course it is a working progress, we are still testing some ideas and thinking new solutions for different situations. We still have 3 weeks until my internship is finished so we have a lot of time to try new things. |
| 15:48 |
julialima_ |
You can find the UI style guide in https://github.com/JuliaLima/Evergreen/tree/patch-1/docs/style_guide, remember that we are working on it, and also you can check in my blog for updates about our progress: http://lima-julia.tumblr.com/EG-style-guide. |
| 15:48 |
julialima_ |
Feel free to contact us and give your opinion, we need your feedback in order to improve our work, it is very important for us. |
| 15:49 |
kmlussier |
julialima_ / dbwells: I can't remember. Have the links to the blog and github repo been posted to the mailing list yet? |
| 15:49 |
bshum |
#link Style guide: https://github.com/JuliaLima/Evergreen/tree/patch-1/docs/style_guide |
| 15:49 |
bshum |
#link Blog: http://lima-julia.tumblr.com/EG-style-guide |
| 16:29 |
yboston |
berick: got a minute |
| 16:36 |
jeffdavis |
1/away afk |
| 16:46 |
|
julialima_ left #evergreen |
| 16:48 |
mdriscoll |
Dyrcona: I have been testing your code for LP#957466 today. Are you still here for a question? |
| 16:50 |
Dyrcona |
Yes. I'm here. |
| 16:50 |
mdriscoll |
If a record has a user in a 905 $u, an update to an existing record will use the 905 $u as the editor, but for a new record, the 905 $u is ignored. |
| 16:51 |
mdriscoll |
I'm not sure if it's important for update and create to work the same way. |
| 10:58 |
* jeff |
nods |
| 10:59 |
jeff |
down-then-over vs over-then-down |
| 10:59 |
jeff |
consistency is important there |
| 10:59 |
mdriscoll |
Dyrcona: I would be happy to test your branch when you get it done |
| 11:01 |
dbwells |
Okay, feel free to keep talking about layout, but we have more questions! :) |
| 11:01 |
jeff |
do the single row and grid examples allow presenting more information when you're looking at the "form" not because you're editing, but because you're reviewing / seeking information? item status single-row display vs item attribute editor single-row display being one example -- would we consider moving the item status display move to single column? |
| 11:03 |
dbwells |
jeff: That's a good point. I don't have a good answer, but I do think the concept of dual purpose display/edit screens should be carefully considered. |
| 12:36 |
|
maryj joined #evergreen |
| 12:38 |
goood |
dbs: or they could just consider the "collab" branch as the "master" to target, and submit branches against that. not seeing the difference, really |
| 12:41 |
dbs |
Right. So why aren't bug fixes and small features being merged to master on a regular basis again? |
| 12:42 |
goood |
dbs: because folks don't test (save kmlussier) or merge, and I get smacked whenever I merge my own code, and get the side eye when I merge my coworkers' code without further review |
| 12:44 |
goood |
s/further review/non-esi review, for whatever personal reasons various folks may have/ |
| 12:49 |
|
nhilton joined #evergreen |
| 12:57 |
|
bmills joined #evergreen |
| 13:09 |
mrpeters |
Alrighty guys -- I'm in Action/Trigger hell and have come to beg for mercy! See http://pastie.org/9945103 -- an event_def to mark an item lost 3 hours after it goes overdue. Nothing fancy, pretty similiar to the stock 90 Day Lost notice. Each time the event runs, it ends with a state 'error' but I'm struggling to find more information about what caused the error. Pertinent osrfsys logs are at http://pastie.org/private/l62y4g3wodwvxhw2iwrdg -- Eve |
| 14:13 |
dbwells |
My sense when he mentioned it last week was that he felt obligated to work on it since he had already done some work on it. My guess is he'll be very happy to see you've completed it. |
| 14:15 |
Dyrcona |
I felt obligated to work on it when a Backstage import busted up someone's work. :) |
| 14:15 |
Dyrcona |
That's cool. Collaboration is nice. |
| 14:24 |
Dyrcona |
While testing that change, I noticed something unusual. |
| 14:25 |
Dyrcona |
Any time I typed a key in the marc editor, regular or flat text, while editing a record to import, a JavaScript error window would pop up. |
| 14:26 |
Dyrcona |
TypeError: tab is undefined |
| 14:26 |
Dyrcona |
Still doing it, actually. |
| 14:26 |
Dyrcona |
MARC edit works fine on regular bibs. |
| 14:27 |
Dyrcona |
Guess I'll do a git clean and see what happens. |
| 14:27 |
kmlussier |
dbs: Thanks for letting me know about mlnc4. I'll fix it up now. |
| 14:28 |
dbwells |
Dyrcona: I saw that too, when testing master with a 2.7 client. I know it stems from some changes ldw wrote a while back, and I thought it might be related to me needing a new client. |
| 14:28 |
dbwells |
Are you seeing it with a master client? |
| 14:28 |
Dyrcona |
Yes. |
| 14:29 |
Dyrcona |
My script builds a fresh client when I build all of Evergreen. |
| 14:29 |
Dyrcona |
Guess that's a bug. |
| 14:45 |
|
Tony__ left #evergreen |
| 14:46 |
|
akilsdonk joined #evergreen |
| 14:50 |
Dyrcona |
Hmmm. I have that commit, so something about it must still be wrong. |
| 14:53 |
kmlussier |
dbs / eeevil: Would it be worthwhile to discuss the web client merging at the next dev meeting? I don't know what the answer is, but I personally would love to see the code make it into master faster. |
| 14:54 |
kmlussier |
One thing I'm up against is that I have a few people on hand to help test the circ changes, but we aren't geared up to test cataloging yet. However, I don't really know if the other circ code in that collab branch is dependent on some of the cataloging features that come before it. |
| 14:55 |
kmlussier |
So signoffs there may not be helpful. |
| 14:56 |
kmlussier |
And mlnc4.mvlcstaff.org is up and running again. |
| 14:57 |
* kmlussier |
should add something to her calendar to update it monthly. |
| 15:03 |
Dyrcona |
Well, the diff isn't telling me much, and neither is JavaScript console. |
| 15:03 |
Dyrcona |
I may have to call on Venkman. |
| 15:04 |
Dyrcona |
If he's still around somewhere. |
| 16:22 |
kmlussier |
berick: Have you set a meeting date yet? |
| 16:37 |
kmlussier |
@quote random |
| 16:37 |
pinesol_green |
kmlussier: Quote #94: "<bshum> we should do something about those cupcakes - they're going to go to waste" (added by csharp at 05:57 PM, September 25, 2014) |
| 16:38 |
kmlussier |
bshum does love his cupcakes |
| 17:04 |
|
nhilton joined #evergreen |
| 17:09 |
|
artunit joined #evergreen |
| 17:10 |
kmlussier |
Must be Friday afternoon. I had two client windows open, and I tried testing the new feature on the system that did not have the code loaded. Explains why it didn't work. |
| 17:13 |
|
mmorgan left #evergreen |
| 18:13 |
|
mrpeters left #evergreen |
| 18:20 |
|
dreuther joined #evergreen |
| 11:11 |
csharp |
berick: ewww |
| 11:11 |
kmlussier |
berick++ |
| 11:18 |
Dyrcona |
heh |
| 11:22 |
berick |
eeevil: kmlussier: going to merge working/user/kmlussier/web-client-sprint1-bug-fixing-signoff1 (plus 9804b5c54d12e6f2d1ae7b907a0de80d84ff978f) after light testing. any issues/concerns before I do that? |
| 11:24 |
kmlussier |
I have no concerns |
| 11:25 |
berick |
note for later, we should make the grid less chatty in the console. |
| 11:25 |
berick |
kmlussier: thanks |
| 11:25 |
goood |
berick: just concerns over rebasing the remainder... but that can be a tomorrow problem, I suppose |
| 11:29 |
jeff |
oh and of course the lovely thing about running services on high ephemeral ports... |
| 11:29 |
jeff |
(well, ONE of the things)... sometimes someone else gets there first. :P |
| 11:29 |
berick |
goood: rebasing collab/miker/web-client-sprint1-bug-fixing-rebased-collab ? |
| 11:33 |
berick |
just tried an expirement, rebasing that ^-- to my about-to-merge master branch. i haven't tested it, but it rebased without conflict. |
| 11:33 |
berick |
so there's that |
| 11:42 |
hopkinsju |
Does Evergreen actually use the quality field in biblio.record_entry? I can't think of anything, and we were thinking about reusing that column. |
| 11:44 |
jeff |
hopkinsju: have you considered biblio.record_note? |
| 11:44 |
|
sandbergja joined #evergreen |
| 11:55 |
bshum |
mmorgan: Now you're getting it! :) |
| 11:55 |
kmlussier |
Yeah, probably. I don't think it matters much. I guess I've just been looking at them far too much over the past year. |
| 11:55 |
berick |
bshum: git filter-branch -f --msg-filter "sed '0,/^/s//LP\#1402797 /1'" origin/master.. |
| 11:56 |
bshum |
If you come up with better categories for things, I'm happy to help test/push that along. One of the things I always wish I had more time to do would be to go back and actually add meaningful descriptions to all YAOUS entries. |
| 11:56 |
berick |
(then had to clean up a few double-tagged commits) |
| 11:56 |
bshum |
berick: Ooooh, fancy! |
| 11:56 |
kmlussier |
bshum: Yeah, adding better descriptions has crossed my mind too. |
| 11:56 |
* berick |
credits gmcharlt with the original suggestion way back when |
| 11:59 |
yboston |
Hello, I can't remember what the rules are for the deadlines we use for having a code change make it to the next release. I see the calendar says that next Friday is the "2.8 Beta Cut-Off" I can't remember what that one is for, I think it is meant for testing and signing off bug fixes? I beleive that the previous "2.8 Feature LP Target Deadline" is for havign signed off changes make it to the beta? |
| 11:59 |
bshum |
kmlussier: "Charge lost on zero" confused me for years. Still does sometime... |
| 12:00 |
berick |
yboston: the 2.8 cut off is for new features only. we *always* accept bug fixes |
| 12:00 |
bshum |
bug_fixes++ |
| 12:00 |
bshum |
:) |
| 12:00 |
yboston |
berick: sorry, that was the one rule I did remember, that bug fixes are always accepted :( |
| 12:00 |
kmlussier |
Is it difficult to update records that are provided in the Concerto set? I keep thinking it would be nice to add prices for all of the copies in there. |
| 12:01 |
kmlussier |
So that when you mark them lost in testing, a bill is added to the patron record. |
| 12:01 |
berick |
yboston: no worries, did I answer your question, though? |
| 12:01 |
bshum |
kmlussier: I'm sure it's not hard to do that. |
| 12:01 |
yboston |
so is there still time to submit (signed off) changes for 2.8? |
| 12:04 |
berick |
yboston: documenting / targeting new features in launchpad. the absence of such documention does not prevent new features getting into 2.8, though |
| 12:04 |
berick |
the purpose was to get as much info out as early as possible |
| 12:04 |
yboston |
berick: thank you very much, I had assumed I missed the boat on a new feature |
| 12:06 |
berick |
yboston: speaking of, do you still have any expectation of testing bug #1171984 ? |
| 12:06 |
pinesol_green |
Launchpad bug 1171984 in Evergreen "add support in Vandelay for overlaying authorities during import using match sets" (affected: 5, heat: 22) [Wishlist,Confirmed] https://launchpad.net/bugs/1171984 |
| 12:06 |
yboston |
berick: so sorry |
| 12:07 |
pinesol_green |
Showing latest 5 of 45 commits to Evergreen... |
| 12:08 |
berick |
and I thought we were immune down here ;) |
| 12:08 |
berick |
yboston++ |
| 12:09 |
|
bmills joined #evergreen |
| 12:10 |
kmlussier |
berick: If yboston doesn't get to that one, I might be able to. I was just looking at it today. |
| 12:10 |
kmlussier |
Or, I should say, eyeing it as a possible thing to test. |
| 12:11 |
berick |
kmlussier: awesome |
| 12:11 |
* berick |
pushed a rebased-to-master branch |
| 12:11 |
yboston |
kmlussier: I will let you know if I start testing it so you don't have to, then again maybe the more tsts the better |
| 12:13 |
|
sandbergja joined #evergreen |
| 12:13 |
* berick |
needs to come up w/ a conference proposal |
| 12:13 |
|
akilsdonk joined #evergreen |
| 12:37 |
geoffsams |
Of Course I haven't looked at the specific pages that have been brought up either |
| 12:41 |
|
maryj joined #evergreen |
| 12:47 |
kmlussier |
gsams: 98/100 sounds pretty good |
| 12:48 |
gsams |
kmlussier: That's for the advanced search page at least |
| 12:49 |
gsams |
Since I'm 4 days after an upgrade I haven't had time to do more testing |
| 12:49 |
kmlussier |
I'm interepreting that as 98 out of 100. |
| 12:49 |
gsams |
that is correct |
| 12:49 |
gsams |
The only usability changes it suggested were related to tap targets spacing and size |
| 12:52 |
gsams |
basically, Basic Search and Browse the Catalog are too close, The search input boxes are too close, and we have a link at the bottom that is too close to other things |
| 12:56 |
goood |
hopkinsju: quality is use for lead metarecord selection. can you give some context to your "scoring algorithm"? what are you scoring, how do you interpret a score value, and what questions are answered by the score |
| 13:00 |
|
krvmga joined #evergreen |
| 13:01 |
bshum |
Hmm, c4l lightning talk about "keyboard accessibility" makes me think about how much testing we're doing on the catalog / web client. |
| 13:01 |
bshum |
So much stuff to test. |
| 13:03 |
|
sandbergja left #evergreen |
| 13:05 |
kmlussier |
bshum: Well, there currently isn't much to test in the web client in terms of keyboard accessibility. I don't think the keyboard shortcuts have been included yet. |
| 13:06 |
kmlussier |
yboston: Are you still around? |
| 13:07 |
|
afterl joined #evergreen |
| 13:33 |
yboston |
kmlussier: I am back at my desk |
| 13:34 |
goood |
berick: the removal of dropdown-toggle on the actions in the grid (commit d37e4d0bba) means, for ng 1.2.22 (on webby), that the menu stays open after the launching the action. Is that just an "old ng" problem? |
| 14:11 |
kmlussier |
yboston++ |
| 14:11 |
yboston |
bug #1171984 |
| 14:11 |
pinesol_green |
Launchpad bug 1171984 in Evergreen "add support in Vandelay for overlaying authorities during import using match sets" (affected: 5, heat: 22) [Wishlist,Confirmed] https://launchpad.net/bugs/1171984 |
| 14:12 |
yboston |
this is a feature that needs to be tested out for inclusion in EG 2.8. Again there is one week left to have someone test it and "sign off" on it |
| 14:12 |
yboston |
Kathy or I will try to do the signing off |
| 14:13 |
yboston |
I am bringing this up partially to showcase how improvements are made to EG |
| 14:13 |
yboston |
that is all for me on authoritites. I can move to the nexr past meeting's action item |
| 14:13 |
|
Cybrarian joined #evergreen |
| 14:14 |
yboston |
moving on |
| 14:14 |
yboston |
3) jeffdavis to put out a call for current patron loading scripts that are already in use. |
| 14:14 |
yboston |
I am not sure if jeffdavis is available now, will wait a bit before defring |
| 14:15 |
yboston |
moving on |
| 14:15 |
yboston |
#action jeffdavis to put out a call for current patron loading scripts that are already in use. |
| 14:15 |
yboston |
at this point there are now more outstanding past meetign action items |
| 14:15 |
yboston |
we can now shift to talk about other issues |
| 14:16 |
yboston |
for example, thoughts on the earlier discussion on call number search |
| 14:16 |
yboston |
or something completely different |
| 14:16 |
Cybrarian |
PAC flavors? |
| 14:16 |
DonB__ |
I sent out a couple of emails to the list regarding call numbers |
| 14:17 |
DonB__ |
It seems there is broad consensus in two areas |
| 16:52 |
* Dyrcona |
typoed Integer as Interger last night. |
| 16:52 |
Dyrcona |
That'll do it. |
| 16:52 |
RoganH |
I did it once before which is why I suddenly thought of it. |
| 16:53 |
RoganH |
I discovered that one before testing though. Ug. |
| 16:57 |
RoganH |
That did it. Thanks for the ideas all. |
| 16:57 |
RoganH |
Dyrcona++ |
| 16:57 |
RoganH |
kmlussier++ |
| 17:04 |
gsams |
kmlussier++ |
| 17:04 |
kmlussier |
gsams: Awesome! |
| 17:04 |
RoganH |
Good night! |
| 17:07 |
pinesol_green |
Incoming from qatests: Test Success - http://testing.evergreen-ils.org/~live/test.html <http://testing.evergreen-ils.org/~live/test.html> |
| 17:07 |
|
mdriscoll left #evergreen |
| 18:02 |
|
bbqben joined #evergreen |
| 18:12 |
|
mrpeters left #evergreen |
| 05:09 |
pinesol_green |
Incoming from qatests: Test Success - http://testing.evergreen-ils.org/~live/test.html <http://testing.evergreen-ils.org/~live/test.html> |
| 07:32 |
|
graced joined #evergreen |
| 07:53 |
|
rjackson_isl joined #evergreen |
| 07:55 |
|
akilsdonk joined #evergreen |
| 10:59 |
berick |
with subsequent calls, it compares the previous value to the new value |
| 10:59 |
berick |
if they are different, it calls the $watch function |
| 10:59 |
berick |
that was provided by us |
| 10:59 |
phasefx |
rock |
| 10:59 |
phasefx |
berick: do you mind if I pick those into the sprint1 collab branch? |
| 11:00 |
phasefx |
(after testing) |
| 11:00 |
berick |
phasefx: no, i don't mind |
| 11:00 |
phasefx |
rock |
| 11:00 |
berick |
also, re: $watch, that's one reason you want to avoid using $watch unless necessary, because it causes extra copies of objects to have to persist. |
| 14:06 |
eeevil |
(sorry if that seems overly blunt ... just meant to convey my thoughts succinctly ;) ) |
| 14:06 |
|
DPearl joined #evergreen |
| 14:09 |
eeevil |
kmlussier: unrelated to the web client, I pushed a change for the don't-change-0-balance-lost-xact branch, per the situation you identified ... not sure if you saw that |
| 14:10 |
kmlussier |
eeevil: Sure, I'm okay with pushing them in as is because, as far as I know, nobody is using the web client yet. I'm just having a hard time managing my own testing of the new code. |
| 14:10 |
kmlussier |
eeevil: I did see that. Thanks! :) |
| 14:11 |
* bshum |
has been likewise confused by the status of that collab branch |
| 14:11 |
bshum |
Is there a specific block of commits we should focus on getting into master first? Or are we just expecting to push the whole thing enmasse? |
| 14:11 |
kmlussier |
Also, all the code I've reviewed so far has improved the web client, so +1 to getting it in. |
| 14:12 |
* bshum |
also assumes none of it is being backported to rel_2_7 then. |
| 14:12 |
bshum |
Though, they are "bug fixes"? Meh |
| 14:14 |
eeevil |
kmlussier: right. IIRC, the consensus was folks should wait until they can do OneBigSwitch(tm) to the web client. now, whether folks will... :) |
| 14:14 |
* kmlussier |
doesn't remember that consensus |
| 14:15 |
kmlussier |
Would it help if I signed off on the commits I already tested? |
| 14:15 |
kmlussier |
I know I had further feedback on some of those commits, but I only saw improvements in what I tested. |
| 14:15 |
DPearl |
I'm attempting to build a Concerto database; it worked some time ago, but there was a concerto.sql file to read in. Things have changed and concerto.sql is no longer there. The eg_db_config.pl is gone and replaced by an .in file, so current instructions to build the db don't work! Where should I look for the new doc? |
| 14:15 |
eeevil |
kmlussier: I imagine it would, or at least note them. I'm not sure how berick wants to handle that merge |
| 14:18 |
bshum |
eeevil: Can sprint1 fixes be un-entangled from the collab from sprint2 stuff? Or do we have to test all the things before the branch can be moved forward in any way? |
| 14:18 |
kmlussier |
OK, I'll do some sign-offs. But I still would like to advocate for putting the new cataloging code into a different branch from the circulation fixes. |
| 14:19 |
kmlussier |
Ha ha. What bshum said. |
| 14:19 |
* bshum |
is just trying to help, not trying to be difficult. I just work better in more concrete blocks of things to look at. |
| 14:42 |
eeevil |
the cataloging changes really need everything else (or, substantially), and since nobody has jumped in to help with coding things they find broken, I chose to reduce the overhead of making progress. I can start a new branch called "sprint2" if that would help, but it'll just be a clone of the sprint1 branch. will it provide a lot of help to remove any sprint2 code in another copy of the collab branch? (I'll personally |
| 14:42 |
eeevil |
consider that branch a dead end, fwiw, because I don't see much point in shuffling commits all over the place when there's really only going to be just a couple developers working in tight coordination) |
| 14:44 |
eeevil |
now, if folks started to show an interest in getting involved in either initial coding of features beyond the current plan, or bug fixing (to learn angularjs, or just because), then the multi-branch-parallel-to-master world would make sense |
| 14:52 |
kmlussier |
Heh, I don't think anyone wants me diving into angularjs, but I'll do what I can to sign off on stuff as soon as I test it, which might make things more manageable. |
| 14:54 |
Dyrcona |
My bosses have other ideas how I should spend my time right now. |
| 14:54 |
dbs |
Having small branches is probably the best way for people to learn how to start digging in. |
| 15:11 |
eeevil |
dbs: that's probably true when the small branches are tracking a canonical master branch, but in this case we're talking about (at least) two branches tracking both master and another "fix" branch, which has to track master, too, leading to periodic manual, cascading rebases ... AFAICT. I can do that or write code :). if folks think of the collab branch as the "master for web client -- for now", then they can branch from |
| 16:00 |
kmlussier |
Yikes! beta cutoff is 1 1/2 weeks away? |
| 16:13 |
gmcharlt |
kmlussier: time flies when we're having fun |
| 16:13 |
kmlussier |
gmcharlt: Fun? |
| 16:13 |
gmcharlt |
yes! |
| 16:13 |
gmcharlt |
docs! \o/ |
| 16:13 |
gmcharlt |
patches! \o/ |
| 16:14 |
gmcharlt |
regression tests \o/ |
| 16:14 |
gmcharlt |
too much coffee! \o/ |
| 16:16 |
dbwells |
I was still thinking the cutoff was the 18th, but I now see it got pushed to the 20th. Two extra days! \o/ |
| 16:51 |
pinesol_green |
Incoming from qatests: Test Success - http://testing.evergreen-ils.org/~live/test.html <http://testing.evergreen-ils.org/~live/test.html> |
| 17:04 |
|
eeevil joined #evergreen |
| 17:05 |
|
sarabee joined #evergreen |
| 17:11 |
|
mmorgan left #evergreen |
| 11:55 |
tsbere |
kmlussier: The unit tests are for things that don't require the network to be up. "This utility function should always do the same thing given the same input and doesn't talk to anyone else while doing so" |
| 11:55 |
phasefx |
kmlussier: the unit tests are testing pieces of code that are very self-contained.. the live tests require a larger environment in a known state (a running system with stock test database) |
| 11:56 |
phasefx |
kmlussier: so you end up testing some interaction between multiple moving parts |
| 11:59 |
kmlussier |
phasefx: So is that the same as integration testing? |
| 11:59 |
phasefx |
kmlussier: I think so |
| 12:02 |
|
mmorgan joined #evergreen |
| 12:12 |
gsams |
Just upgraded from 2.3.5 to 2.7.3 and we are noticing problems with Overdrive authentication. The logs are showing successful attempt to access but Overdrive appears to be blocking anyway. |
| 12:17 |
|
bmills joined #evergreen |
| 12:19 |
kmlussier |
Agreed! Let it be resolved that csharp should get rest. :) |
| 12:20 |
jeff |
gsams: likely OverDrive was relying on the patron screen message field in the SIP response being "OK" -- that is no longer sent. |
| 12:21 |
kmlussier |
gsams: Congrats on your upgrade! Unfortunately, I now no longer have a go-to place to test an early release of Evergreen to verify whether a bug is new or not. ;) |
| 12:21 |
jeff |
...and I always encourage folk to NOT use SIP2 for external vendor auth, but I realize it's hard to change because inertia. |
| 12:23 |
kmlussier |
phasefx: OK, so here's my follow-up question. How do we know that we're testing all the right things with the live tests? Would it be helpful for an end user like myself to supply a set of common scenarios that should be part of the live tests? |
| 12:23 |
gsams |
kmlussier: Hehe, I wish I could still be there for that, but alas we needed these fresh new features and to actually stay up to date. |
| 12:23 |
jeff |
kmlussier: ideally, those scenarios are the input to the kinds of tests you speak of. |
| 12:24 |
kmlussier |
I just happen to have a set of test cases around overdue fines and billing that I use quite frequently. |
| 12:24 |
* jeff |
chuckles |
| 12:24 |
gsams |
jeff: now I'm curious, what would you suggest. I've always thought SIP2 was odd, but I'm not really sure why I've felt that way. I don't know enough about it to begin with. |
| 12:25 |
* jeff |
just threw a dozen or so people at the rel_2_7 (plus some cherry-picked fixes) web staff client with only a few things that broke (mostly due to permission-related quirks) |
| 12:32 |
gsams |
Though, I like to do things the difficult way, so I wouldn't be opposed to conversion |
| 12:33 |
tsbere |
jeff: Given some of the alternatives (or lack thereof) some vendors provide to SIP2 I am not sure there is a good way to avoid starting. >_> |
| 12:34 |
jeff |
tsbere: i'm trying to gather some information on those choices -- especially for those we don't have a relationship with. i'd be interested in what you have, if you don't mind sharing. |
| 12:34 |
berick |
jeff++ # browser client testing |
| 12:35 |
tsbere |
jeff: Most of the ones I know of off the top of my head (AKA, people I actually dealt with, rather than non-technical people dealing with them first and deciding on SIP2 without ever finding out about other options) were "SIP2, don't actually validate against your system, or provide us with a URL that spits out custom specific-to-us output that we aren't willing to negotiate much on" |
| 12:35 |
tsbere |
jeff: Oh, and for half of them the URL couldn't be https. <_< |
| 12:45 |
|
sandbergja joined #evergreen |
| 12:46 |
|
sandbergja joined #evergreen |
| 12:46 |
|
sandbergja left #evergreen |
| 12:50 |
jeff |
berick: thanks! I'm immediately being pulled into something else today, but I've a handful of things to follow up on, and we're planning to give all staff test accounts and have them start kicking the tires later this week. |
| 12:50 |
jeff |
(just with concerto as a dataset, to start) |
| 12:52 |
jeff |
is anyone here having staff use the web staff client on a production system yet? |
| 12:52 |
bshum |
jeff: We aren't planning to use any web client bits in active production until more modules are complete. |
| 12:52 |
bshum |
By more I mean specifically cataloging and perhaps even acquisitions :\ |
| 12:53 |
jeff |
so no general usage at service desks for patron account management, placing holds, etc. |
| 12:54 |
bshum |
Not for us anyways. |
| 12:54 |
bshum |
At least not yet. |
| 12:54 |
* bshum |
imagines everyone will brave this stuff differently. |
| 12:57 |
kmlussier |
jeff: Nobody is using it here yet. Based on our preliminary testing, I think there is more work to be done before front-line staff will be ready to use it. eeevil and berick have been making good progress on bug fixes recently, so maybe soon. :) |
| 12:58 |
jeff |
i've applied some of those fixes to this test system :-) |
| 12:58 |
* bshum |
might also wait to see if there's anything that can be done as far as running websockets on different ports. |
| 12:58 |
bshum |
Or rather, not running it on separate ports |
| 12:58 |
kmlussier |
However, I think we'll need keyboard shortcuts before our circ staff is ready to use it. They heavily rely on the keyboard. |
| 13:00 |
bshum |
And they don't always consult us when they change their equipment either. |
| 13:00 |
bshum |
And suddenly the ILS stops working. |
| 13:00 |
bshum |
brb, c4l break time :) |
| 13:00 |
* tsbere |
should try and get websockets working again for testing |
| 13:01 |
kmlussier |
tsbere: That would be great. I would love to try it out on a non-Concerto database. |
| 13:02 |
tsbere |
kmlussier: And the system(s) I was going to see about getting it to work on.......run on Concerto databases. |
| 13:02 |
kmlussier |
tsbere: But it's a first step, right? :) |
