IRC log of #schooltool for Friday, 2012-09-21

*** th1a has quit IRC00:07
*** replaceafill has quit IRC00:47
*** khildin has quit IRC02:26
*** th1a has joined #schooltool04:53
*** th1a has quit IRC09:06
*** povbot has joined #schooltool09:32
*** menesis has joined #schooltool11:55
*** menesis has quit IRC13:46
*** menesis has joined #schooltool14:42
*** th1a has joined #schooltool16:05
*** replaceafill has joined #schooltool16:26
th1ahi replaceafill.16:30
replaceafillhey th1a16:30
th1aQuick question here then maybe switch to a hangout?16:31
th1aSo is there something about filldown that bypasses the normal check for invalid scores?16:32
replaceafillnot really16:33
th1aI don't understand what the issue is with invalid scores.16:34
replaceafillwhen you type an invalid score, you get the red background, right?16:34
replaceafilltelling you is invalid16:35
replaceafillyou can then click Save16:35
replaceafilland the red background and <input /> field go away16:35
replaceafillshowing the invalid score as a regular <td> element16:35
replaceafill*that* gives you the impression that the invalid value was saved16:35
replaceafillbut in reality it was taking from the request only16:36
replaceafillnot from the gradebook16:36
replaceafillnot from the stored data16:36
replaceafillso, if you refresh the page again16:36
replaceafill(with nothing in the request this time)16:36
replaceafillyou'll get the value *from* the gradebook16:36
replaceafill(the previous valid value)16:37
th1aYes, that's definitely bad16:37
th1aIn the gradebook we store invalid scores, don't we?16:37
th1aI mean it is pretty much a toss up.16:37
replaceafilli think it shouldn't even activate "Save" when there are invalid <input/>16:37
th1aNo, it should.16:38
replaceafillno, that's the point, we don't store invalid16:38
replaceafillit seems like we do16:38
replaceafillbut we don't16:38
replaceafillyou can try it out16:38
th1aWhat about if we:16:38
th1aI meant in the gradebook gradebook.16:38
th1aI think we might store invalid grades there.16:39
replaceafillhhmm it's the same problem in the gradebook/cando16:39
th1aSo what if we:16:39
th1a1) pop a js warning "Some invalid scores were not saved."16:40
th1a2) Show them as red input fields with the invalid scores.16:40
th1aBut save the valid ones.16:40
replaceafillshould the popup be modal?16:41
replaceafilli can work on that before moving to projects16:41
replaceafilli was thinking also of adding the warning message16:41
th1aOh, ok.16:41
replaceafillwhen you have unsaved scores16:41
replaceafilland want to leave the gradebook16:41
th1aYes, it is supposed to be there if it isn't.16:42
th1aCan you explain this proxy issue to me?   I think making course skills "proxy" global skills was a huge mistake on my part.  It was put there to allow rating same (course) skill in several different sections, with changes being shared immediately.  But that's not a very useful scenario I guess.16:47
th1aThat's quoting yvl.16:47
replaceafillah yes16:47
replaceafilli think the original use case was:16:49
replaceafilloverride required/optional16:49
replaceafillat the course level16:49
replaceafillthat's the "sharing" part yvl mentions (i think)16:49
th1aI guess I don't understand the difference between copies and proxies in this case.16:50
replaceafillcopies at the section level would make them different "requirements"16:51
replaceafillwe save requirements in the evaluation16:52
replaceafilland you could find out the section from the evaluation16:52
th1aWhat is a proxy in comparison?16:53
replaceafilli think this could also give dwelsh the section level skills he wanted at some point16:53
replaceafillthe proxy represents the global skill16:53
replaceafilljust overriding some of its "original" attributes16:54
replaceafillbut if you do some url calculation on it (for example)16:54
replaceafillyou'll get pointed to its original container16:54
replaceafillin this case the course skills16:54
replaceafill"it's easy to figure out section and course (since the rated skills would be stored in section itself)"16:56
th1aOK... so does this change seem reasonable to you?16:57
replaceafillagain, gives us skills at the section level16:57
replaceafillhhmm good question16:57
th1aWe have enough info to infer them in the small number of existing cases.16:58
th1ainfer the sections.16:58
th1aIn evaluations.16:58
th1aI guess that's not the issue though.16:58
replaceafilli think *that* would be the issue16:59
replaceafillassuming someone is already grading multi term skills ;)17:00
th1aWe could infer it, it would just be a little hacky, right?17:00
replaceafilli don't see how to tell them apart17:01
replaceafillfrom the evaluation17:01
replaceafilldatetime is not good imho17:01
replaceafillyou have (student, skill [global], evaluator, datetime)17:01
th1aIf you have the student, evaluator and date, you should be able to guess the section.17:02
replaceafillyes, not exactly right, but it could work17:03
replaceafillhacky as you say17:03
th1aI think it would work well enough considering that most CanDo users haven't been trained yet.17:03
replaceafillah yes, and considering they're using single term years17:04
replaceafilli guess you have to use the hacky approach only when you see multiple linked sections17:04
th1aIt isn't that hacky since we know for sure that everything should be in the first one.17:05
th1aI mean, it is hacky.17:05
th1aBut not complicated.  ;-)17:05
th1aOK.  Give me 10 minutes and then hangout?17:06
replaceafilli'll get my stuff17:06
*** ignas has joined #schooltool17:29
th1areplaceafill:  Do you want this permissions bug from Glenda or should I give it to someone else.18:58
th1aYou've got the biggest pile right now.18:58
replaceafillis it a bug?18:58
replaceafillor should we just use schooltool.view for the options?18:59
replaceafillbtw, is the code for other projects (thailand, australia) available somewhere?19:00
replaceafilli'd like to see the "read anything/but not edit" implementation :)19:00
th1aThat's an idea, not an implementation.  ;-)19:01
replaceafillah ok19:01
th1aWe *definitely* should want to move away from using edit to view though.19:01
th1aBecause I have a feeling that we'll be trying that change.19:01
replaceafilli think the problem here is that students have access to the worksheet19:02
replaceafilli don't know if the students gradebook helps19:02
replaceafilli think we could use a gradebook_security.txt selenium test for sure19:03
replaceafillfor this19:03
replaceafill(if we change the permissions of the options to schooltool.view)19:03
replaceafilli can investigate ;)19:03
th1aCan students see the whole worksheet?19:04
th1aThat would be VERY wrong.19:04
replaceafilli don't think so, because it requires schooltool.edit19:04
replaceafilllet me check19:04
th1aSo what do they see?19:04
replaceafillan adapter19:04
replaceafillah ok19:08
replaceafillyes, i think we could change the options to schooltool.view19:08
replaceafilli do think a selenium test is worth :)19:09
*** ignas has quit IRC19:10
replaceafillglenda is requesting protection by resource19:11
th1aThat's not happening.19:11
*** ignas has joined #schooltool19:11
replaceafillth1a, ok, i'll change the permissions of the options then19:13
replaceafillsince i'm now working on the gradebook19:14
*** ignas has quit IRC19:50
*** paulproteus has quit IRC20:07
*** paulproteus has joined #schooltool20:11
th1aaelkner: ayt?21:29
*** khildin has joined #schooltool21:47
*** menesis has left #schooltool22:03
*** menesis_ has joined #schooltool22:04

Generated by 2.15.1 by Marius Gedminas - find it at!