IRC log of #schooltool for Tuesday, 2014-12-02

*** menesis has quit IRC02:19
*** mobert has quit IRC06:09
*** yvl has joined #schooltool12:21
*** yvl has quit IRC13:13
*** yvl has joined #schooltool13:19
*** menesis has joined #schooltool13:28
*** menesis has quit IRC14:36
*** khildin has joined #schooltool15:15
*** menesis has joined #schooltool15:32
*** replaceafill has joined #schooltool16:11
*** khildin has quit IRC16:16
th1ahi replaceafill.16:34
replaceafillhey th1a16:34
th1aSo, where are we today?16:36
replaceafilli merged some of the 2.9 fixes back to trunk16:36
replaceafillmostly papercuts i found during the live searches work16:37
replaceafilli also fixed the bug i told you about yesterday, the one in the report card16:37
replaceafilli'll make a gradebook bugfix release today16:37
replaceafilland i found my notes on the importer change16:38
replaceafillcan we discuss that now?16:38
replaceafillso this change will only affect the Persons sheet16:40
replaceafilland the only column required would be Username16:41
th1aI mean, it could affect resource too, since they also have custom attributes.16:41
th1aBut it is much, much less of a priority.16:41
replaceafilli don't think the current resource importer uses custom attributes16:42
th1aOh, ok.16:42
th1aI think we had that conversation.16:42
replaceafilli think so, that's where i got the notes :)16:42
replaceafillmy question is titles vs IDs16:43
replaceafillshould we just use IDs for all the columns and be done with it?16:43
th1aIs that what we decided before?16:44
replaceafilli don't have a decision recorded16:44
th1aIs there a good reason not to I've forgotten.  ;-)16:45
replaceafillcan you (re)explain how do you think titles (or IDs) should work?16:46
replaceafilli do have:16:46
replaceafill- there may be duplicated titles16:47
th1aYeah, that's the obvious problem.16:47
replaceafillimho titles also have the l10n issue16:49
replaceafillso far that hasn't been a problem because standard fields like "First name", "Prefix", "Last name", etc16:50
replaceafillhave used positions16:50
replaceafilland users have control over the demo field titles16:50
th1aOK, just ID then.16:50
replaceafilland we use the empty data sheet to show the IDs of the standard fields?16:51
replaceafilluser can see/set demographic field IDs16:52
th1aLet's see...16:52
th1aOld sheets won't work on the new one, because they have titles.16:52
th1aThat was probably the concern...16:52
replaceafillyeah, probably16:53
th1aWell, that's a good 3.0 change.  ;-)16:53
th1aIt is a major release!16:53
replaceafilli remember you wanted some logic for detecting titles and IDs16:54
replaceafillbut i don't remember exactly how16:54
replaceafilland i didn't write it down, sorry16:54
replaceafilli think to avoid that problem16:54
th1aAh, screw it.16:54
th1aIt is 3.0!16:54
th1aMajor revision.16:55
replaceafillthere are other importer bugs that we could kill? like16:55
replaceafilland maybe
th1aYes and yes.16:57
* replaceafill done16:57
th1aOK, can you update the empty instance mobert was using to have the final school page code?16:58
replaceafillah ok16:59
th1aIt has the changes to teacher and student (count + search)?16:59
replaceafilli think a catalog change would break it17:00
replaceafillwill try anyway17:00
th1aYou can nuke the db.17:00
replaceafillah ok17:01
th1aOK.  I have some accounting to take care of and then I'm going to try to set up a selenium benchmark.17:03
replaceafillfor the cubebox?17:03
th1aI have the USB stick.17:03
replaceafillok, i'll go get a quick breakfast17:03
th1aThanks replaceafill.17:04
replaceafillok, thanks th1a17:05
replaceafillth1a, instance updated17:07
th1areplaceafill:  btw, mobert is very much in favor of having clerks be directed to the school page when they log in.17:27
replaceafillth1a, ah, instead of going to the calendar17:33
replaceafillbased on the active year?17:34
*** yvl has quit IRC17:35
* th1a goes to sit with Julia a while... she's still sick.17:40
*** khildin has joined #schooltool17:41
*** khildin has quit IRC18:26
*** khildin has joined #schooltool18:38
*** mobert has joined #schooltool18:40
th1ahi mobert.18:47
moberthey th1a18:47
th1aSo for, say, adding a person.18:47
th1aChange "To create a new Person account, you must be logged in as a member of Clerks or Site Managers. Select the School tab at the top of the page, then click on People to end up here:"18:48
mobert"In version 3.0 or after, click on the plus sign (screenshot). In earlier versions, navigate to ..."18:49
th1aYeah, But you don't really need to say "in earlier versions" even.18:49
th1aBecause it still works.18:49
th1aWe don't need to add a lot of verbosity.18:49
mobertHmm. But if there's an easy one-click way that only works after version 3.0, and a more cumbersome way that works now...18:51
mobert... I think there's value to spelling out that the more cumbersome way works now ...18:51
mobert... even if it will still work in the future.18:51
th1aI don't think it is necessary to spell it out.18:53
th1aI guess it is just three words.18:53
th1aSure, fine.18:53
th1aDo it that way.18:53
th1aThat should be enough to get you started.18:57
th1aHm... can we push this back to matt-book?18:57
mobertYep, since I'll be making changes that don't apply to the current release.18:58
mobertAnd then we can publish the new documentation on the day the new version goes live.18:59
mobertIs there anything else you want me to work on in the main branch? Stuff that can go live before 3.0 release?19:00
mobertNever mind, th1a. I'll just start working on the 3.0 docs with the new School page, and I'll push the changes to the matt-book repo.19:05
th1aYeah, nothing else for the 2.x branch I don't think.19:21
mobertBTW, I'm looking at -- very cool19:34
th1aConfusing, but in a productive way.19:35
th1aThe local storage part is confusing.19:35
mobertThe whole node.js implementation of sfw is a bit confusing.19:37
mobertIt looks great though. And it plays very well in mobile browsers!19:37
th1aYeah, it took me a while to figure out that links were opening off screen to the right though.19:38
mobertYup, and you can just swipe left-to-right to return to the previous page.19:39
mobertI found it more intuitive since I had already viewed it in a regular browser.19:39
mobertLuckily, you haven't written much there yet, so I won't spend all day looking at it! :)19:40
*** menesis has quit IRC21:23
*** menesis has joined #schooltool22:00
th1areplaceafill:  Attendance by student seems to be broken.22:18
replaceafillth1a, ah22:18
th1aYou get a blank dialog -- there is a new bug for it I just confirmed while writing my benchmark test.22:18
replaceafilli saw the bug22:18
replaceafillok, i'll check it22:18
replaceafillth1a, could you zip the logs?22:18
th1aOK, but I have to go get Vivian in a minute, so after a bit.22:19
replaceafillah np22:19
replaceafilli'll set it up22:19
*** replaceafill has quit IRC22:22
*** replaceafill has joined #schooltool22:22
replaceafillth1a, hhm can't reproduce22:34
replaceafillare you using sample data?22:34
replaceafillah nm, got it22:35
replaceafillthe test doesn't catch it because celery is disabled :(22:35
replaceafillth1a, fixed:
replaceafillwill make a bugfix release tonight22:58
th1aAh, cool.22:59
th1aOK... I think I spent enough time with this trying to make a benchmark script that I know how it performs off the micro sd.23:02
th1aIt is a bit frustrating because for all the dynamic stuff you'd have to make selenium run slower.23:02
th1aAt which point you're not sure what you're testing, etc.23:02
replaceafilldynamic stuff?23:02
th1aProbably just doing the big import is as good as anything.23:02
th1aLike, trying to do a fill down,23:03
th1aI'm afraid there's no apparent improvement moving the zodb to the usb stick.23:21
th1aI mean, not that you'd expect it to set the world on fire.23:22
*** khildin has quit IRC23:28

Generated by 2.15.1 by Marius Gedminas - find it at!