IRC log of #schooltool for Wednesday, 2012-04-11

*** th1a has joined #schooltool01:44
*** alga has quit IRC02:19
*** th1a has quit IRC02:56
*** replaceafill has quit IRC03:04
*** th1a has joined #schooltool03:34
*** aks has joined #schooltool05:27
th1aaks:  Just the person I need to talk to.05:28
th1aaks:  I've been wondering what the status of SchoolTool in Nepal is.  Do you have 2.0 running?05:40
*** aks has quit IRC06:00
*** aks has joined #schooltool06:00
*** paulproteus has quit IRC06:22
*** paulproteus has joined #schooltool06:26
*** th1a has quit IRC06:46
*** alga has joined #schooltool11:58
*** menesis has joined #schooltool13:57
*** ignas has joined #schooltool14:15
*** th1a has joined #schooltool15:24
*** aks has quit IRC16:14
*** replaceafill has joined #schooltool16:21
th1ahi replaceafill, aelkner.16:31
th1aand menesis.16:31
replaceafillgood morning/afternoon16:31
th1ayvl is apparently sick.16:31
th1a... has been sick.16:31
* replaceafill is sick too :(16:31
menesisyes, yvl is sick16:33
menesisand will likely be for the rest of the week16:33
th1aOK, let's start by looking at:
th1aThe fact of the matter is that other than report cards, almost all the other existing reports are just a header and a table.16:35
th1aSo I'll probably ask Vinny to do an example of a simpler tabular report.16:35
th1aAnd fully annotate the examples.  And that should be sufficient for us I think.16:35
th1aNote that we don't literally have to include all these components (like the overview) but it is nice to have the style examples and ideas.16:37
th1aI guess my question to aelkner and replaceafill is if you feel like you could start using the examples to create these reports and make others in the same style.16:39
replaceafillth1a, i thought we were going to create a "central" style for reports16:39
replaceafilland start using that16:40
replaceafillthe "css for reports" task16:40
replaceafillor "css for pdfs" (don't remember the name)16:40
th1aSo... does anything in particular leap out as being missing to start doing that?16:40
aelknerit looks like the comments section of the first report could be the outline activities in our layout16:41
replaceafilli'd like to see a landscape version16:41
th1aGood point!16:41
replaceafilli think the gradebook should produce landscape reports16:41
replaceafill(at least the worksheet pdf one)16:42
th1aI think I should ask him for a generic table grid & textarea.16:42
th1aOK.  Do you feel like this is a good foundation replaceafill?16:44
replaceafillth1a, i think so, i didn't get the Verdana change though16:44
replaceafillwe're still using the ubuntu font, right?16:45
th1aUh... Maybe this is a debian glitch...16:45
replaceafillno, i mean "I used verdana for everything because I felt it’s a lot more websafe than ubuntu and it looks similar."16:45
th1aOh, well no, we aren't using verdana.16:46
th1aIs that just in the web version?16:46
th1aThe printed examples use ubuntu anyhow don't they?16:46
* th1a didn't pay too much attention to the web example.16:46
replaceafilli think both use the same font16:47
replaceafilland both seem like Verdana16:47
* replaceafill always watch the "l" :)16:47
th1aThe J is different.16:47
th1aOh, he just uses it in the header.16:48
th1aWell, anyhow, thanks for pointing that out.16:48
replaceafillah, true16:48
replaceafillthe report card is the most complex report we have, right?16:49
th1aOr... maybe not.  I guess that's not the Ubuntu J.16:49
th1aBut yes, the report card is the most complex one.16:49
th1aWe just need some simple generic examples.16:49
th1aI think we can move on.16:50
aelknerok, so i took it upon myself (since i needed to add new tests to course)16:51
aelknerto move all the export ftests to stests (except sample_data.txt)16:52
aelknerthe ftests only run at level 2, so if something gets broken, it isn't caught right away16:53
aelknernow they are selenium tests, so they will run at the standard test level16:53
aelknerthis way, if someone does something to break the importer, they will catch it16:53
aelkneranyway, i also coded the FlatSectionsTable export, replacing the old sections export16:54
aelkneras we discussed, and added the unit tests for that16:54
aelknernext, i will work on the report card xls export16:56
aelknerwhat should the report request link be called?16:56
aelknerExport Report Card XLS?16:56
aelknerwe have Export as XLS for the main one16:57
th1aOh, they're probably both wrong.16:57
aelknerthat being from the School tab, and exporting everything16:57
th1aBasically I'd like to move away from putting the format in the link.16:58
aelknerpeople tend to think in terms of formats16:59
th1aPut it under term and call it "Report Sheets"16:59
th1aNo... they don't think "I need to see an XLS."17:00
aelknerif they are passing around files in email, they need to know what type of file to look for17:00
th1aBasically going forward when there are multiple versions of the same report I'd like to have you choose in the dialog.17:00
aelknerso they get into the habit of thinking of the file type that way17:00
th1aPick the data you want, then the format.17:00
aelknerwell, that's different17:01
replaceafillusers see the format in the request dialogs also, so they know17:01
aelknerso, for now, it won't ask, but later it will17:01
th1aAs it becomes more necessary.17:01
th1aThe report system is going to get more complicated as needed.17:01
aelknerso term view, Report Sheets, for now17:02
aelknerwe won't change the Export as XLS for now, right?17:02
th1aIt isn't a priority.17:02
aelkneroh, one last thing, i realized that i hadn't added teachers and students to the groups17:03
aelkneras i had just done for the old sections sheets17:03
aelknerso i did that17:03
aelknerthat's tested as well17:03
aelknerthat's it for me17:03
th1aSo what's the time frame on the report sheet export?17:05
aelknerthat should take a day, so i'll need more tasks for the rest of the week17:06
th1aOK, the question is what CanDo task would you be able to do without yvl around.17:07
th1aEsp. since he's missing time, we need you rolling, but that's hard to start without him.17:07
th1aBut either skills import or ttw skills creation are the two obvious next steps.17:08
aelknerskills import would be the best one to start with since my head in in import already17:08
aelknerreplaceafill, has yvl pushed a usable cando to trunk?17:09
replaceafillthere's no trunk for cando yet17:09
th1aIt is in svn.17:09
aelknerhow do you keep in sync with yvl?17:10
replaceafillaelkner, i haven't had the need to sync with him yet17:10
replaceafilli took his branch when he left last time17:10
replaceafillto do the gradebook work17:10
aelkneryvl just pushed to lp:~justas-pov/schooltool/cando17:11
replaceafillyes, but those don't affect what i'm doing17:11
replaceafillhe's working on models afaict17:11
aelknerhe just created an importer placeholder :)17:12
aelknertitled Import Skills XLS :)17:12
th1aImport Skills will do.17:12
replaceafillhe even made a little refactoring to the xls importer in core17:12
aelkneri guess he didn't anticipate th1a's decision17:13
aelknerit was idea to refactor core without communicating wth me17:13
aelknerthat's all i've been doing17:13
aelknerunless it's a small change that i can easily merge17:14
replaceafilli think it is aelkner17:14
aelkneri'll have to see17:14
aelkneri'll check out his core branch17:14
th1aI would say the main thing is you should at least be ready to ask yvl some questions and hit the ground running on Monday with the importer.17:15
th1aIdeally actually start it.  But at least grok as much as you can about the new CanDo model.17:15
aelknerthat's exactly what i was about to suggest17:16
aelknersounds like a plan17:16
th1aOK.  Good.17:16
aelknerthe model he wrote is pretty solid, right?17:16
aelkneri mean, it has already come into shape17:17
th1aThe gradebook works.17:17
aelknerwhat did he say he was about to do?17:17
th1aSo all the main pieces must exist.17:17
replaceafilli added a couple of new attributes to skills though17:17
replaceafillfor the gradebook17:17
replaceafilllabel and description17:17
replaceafillthere's also a difference in the worksheet base class17:18
replaceafillyvl created a generic one for cando17:18
replaceafillbut my gradebook changes still depend on the old one17:18
aelknerif i branched yvl's cando and merged your's i shouldn't get any conflicts, should i?17:19
th1aI think there may be a couple loose ends in the model but I think the next step will be ttw skills creation.17:19
replaceafillaelkner, the problem with my cando branch is that depends on my flourish and flourish.gradebook branches17:19
aelknerreplaceafill, it sounds like i will get conflicts17:19
aelknerthis is why i asked about trunk :)17:20
aelknerwe should have one17:20
aelknerbut anyway, i could start with yvl's branch, and look at yours some, too17:21
replaceafillthat's what i think you should do17:21
aelknerperhaps on monday, we cold discuss getting a trunk together17:21
th1aWhy wouldn't you just use replaceafill's?17:21
replaceafillth1a, i think that if aelkner doesn't need the gradebook functionality, he should start with yvl's17:22
replaceafilland just look at my couple of attribute changes i mentioned17:22
replaceafillor he'll get into the branches dependencies trouble17:22
replaceafillfor example:17:22
replaceafillmy flourish branch has now jquery 1.917:22
aelkneri could use a working gradebook, but it doesn't have to work at the stae of the arrt17:22
replaceafillbut i'm not sure if we're going to stick to that one17:22
aelkneryvl's has a working gradebook, right?17:23
replaceafillyvl's branch is just the data model17:23
aelknerso your's is more up to date17:23
aelkneri think th1a was right to suggest using yours17:23
replaceafillfrom the gradebooks pov, yes17:23
replaceafillif you do, remember to use my gradebook and trunk branches then17:23
replaceafillbut i suggest that if you don't need the gradebook functionality for your work, stick to yvl's17:24
th1aIt seems like replaceafill's will become the main working branch next week.17:24
replaceafillth1a, correct17:24
replaceafillbut i'd like yvl's opinion on the jquery changes first17:24
replaceafillthe thing is that we decided to use the new jquery tooltip, in the unstable 1.9 version17:25
aelknerreplaceafill, i can create a new sandbox and branch your core, gradebook and cando17:25
replaceafillaelkner, sure17:25
aelkneras th1a said, that will probably end up as trunk, so it will create less conflicts17:25
replaceafillth1a, but the way the tooltips are behaving now, seems like it can be done without the jquery's functionality17:26
replaceafillso, that's a change i may revert at some point17:26
th1aThe whole thing seems like a toss up to me, but if I was doing it I'd just make a clean sandbox exactly like replaceafill's.17:27
menesisreplaceafill: the releases will stick with jquery 1.7.1 and jqueryui 1.8.1817:27
replaceafillyes, i think it's a toss up too17:27
replaceafillmenesis, we discussed at the sprint that for the time cando was put into production, 1.9 could be out17:28
replaceafillagain, i don't think the tooltip change is strong enough for the move17:29
replaceafillbut we'll see17:29
replaceafilli've been trying to implement the functionality the way th1a asked without tooltips from jquery17:29
replaceafilljust plain js17:29
menesismaybe, but it is not released yet, so will not be available17:30
menesisin ubuntu17:30
th1aThis would be in October release regardless.17:30
th1aBut it still might not be in Ubuntu by then.17:30
th1aOK, aelkner, pick one.17:30
th1aSix of one, half dozen of the other.17:31
aelknerpick what?17:31
aelkneri thought we agreed to me using replaceafill's17:31
aelkneri had jsut suggested it and noone objected17:31
replaceafillno objections frome me17:32
th1aOK, thanks aelkner.17:32
th1amenesis, anything to report?17:32
th1aWe are planning a final release before precise, correct?17:32
menesisin fact, Final Freeze is tomorrow17:33
th1aOoh, so...?17:34
menesisso I wanted to make a release today17:34
menesisaelkner, replaceafill: anything you have finished but not yet merged to trunk?17:34
replaceafillmenesis, nothing from me17:35
menesisth1a: it is two weeks until ubuntu release17:35
aelknermenesis, i have many importer changes17:35
th1aThat's for next release.17:35
menesisduring final freeze, all uploads have to be approved17:35
th1aThe CanDo style importer changes.17:35
aelknerwhen is final freeze?17:35
th1aaelkner:  This would only be bugfixes.17:35
menesisbut since we are in universe, it is a formal approval17:36
aelknerth1a, i made many changes to importer, but they are not only for cando17:36
aelknerwell, the flat sections table could be considered only cando as they are only customer so far17:36
menesisyes, Students and Teachers sheets are already merged17:36
th1aThe wholesale changes to the importer are NOT for this release.17:36
th1aIt is just too late in the release process for such big changes.17:37
aelknerwell, i was about to merge to trunk by week's end, should i not?17:37
aelknermenesis, would that throw you off?17:37
menesisif it is big changes for cando/next release, then yes17:38
th1aI'd like this done quickly by aelkner or menesis
aelknermenesis, you could test merge with my branch to see17:39
aelkneri haven't added to CHANGES.txt yet17:39
th1aIs this resolved?
th1aThere is this
menesisaelkner: will look, but I'd prefer to hold off merging features for one week17:40
aelknerso i won't merge and push to trunk yet17:41
menesisth1a: I can change the parents string 97367917:41
aelknerhow about we talk about this again on monday17:41
aelknerth1a, i haven't changed it yet17:41
th1aThis import stuff should not be going in.  There isn't really any discussion.17:41
th1aWe need the fixes to 944471 but not the other import changes.17:42
th1aI'd like this in
aelknerinto the release, yes, but it needs to get into trunk soon so that we can pile changes upon it17:42
aelknerotherwise we build up the conflicts17:42
th1aI'd like this in
th1aThis fix:
th1aThis fix:
th1aThis fix:
menesisI will create a stable branch for myself and you can merge everything to trunk17:44
aelknermenesis, thanks17:44
menesisbut I'd like to be sure without doing work now, that what we have, works17:45
menesisuntil I do final 2.1.0 release next week17:45
aelkneri won't merge to trunk right away, we can discuss again monday, how about that?17:46
menesisreplaceafill: did not understand, have you solved 944471? does it need any code changes?17:46
menesisaelkner: yes17:46
menesisthe custom demographic fields import17:47
th1aOr did we not get to it?17:47
aelknerare we discussing
aelknerthat one s assigned to me, but replaceafill is working with Carlos, no?17:47
th1aI guess we only got the workaround?17:48
replaceafillCarlos solved his problem afaik17:48
th1aOK, now I'm remembering.17:48
menesisas I understand custom demo fields work17:48
aelknerso that is no longer a valid bug17:48
th1aWe talked about fixing it but I guess forgot aelkner?17:48
menesisbut there were issues17:48
th1aNO IT IS A BUG.17:48
th1aRemember talking about needing a two-pass importer?17:49
aelknermenesis just said they work, that's what i expect17:49
replaceafillaelkner, you changed the importer to add the persons to the teachers/students group, right?17:49
th1aRegardless, I guess we're not going to have the fix in the release since we haven't fixed it.17:49
replaceafilli remember that was the issue with that bug17:49
aelknerreplaceafill, yes17:49
th1aBut that needs to be done before aelkner moves on to CanDo anyhow.17:49
replaceafilliirc the data is importer, but not visible, because the person was not in the correct group17:50
replaceafillso the limited demo fields were not shown17:50
replaceafillaelkner changed that, so the limited demo fields *should* now be shown17:51
aelknerah, yes, that lead to the idea to automatically add them to the groups17:51
aelkneranyway, i found the merge to trunk with that feature, march 2617:52
th1aIt doesn't solve the fundamental problem -- it still should work only with the persons sheet.17:52
th1aAre the attributes just hidden until the group is set or are they not added at all?17:52
aelknerthe persons sheet has all demos17:52
replaceafillth1a, just hidden17:52
replaceafillth1a, you can't tell the groups from the person sheet17:53
replaceafillunless you sniff the data or something17:53
aelknerno, ot possible17:53
aelknerthe columns need to be fixed by group17:53
replaceafill"this column should only be set for teachers, so if this person has data on it, it is a teacher"17:53
th1aCan't we just make the person view show attributes that have values, even if the group isn't set?17:53
aelknerth1a, the way it works now is correct17:54
replaceafillth1a, i thought of that at the beginning, but then you loose the functionality of limited demo fields17:54
aelknerall demos are columns after the standard columns17:54
th1aThere is no reason in particular to hide the values.17:54
aelknerthe only sheets that limit the columns are the new Teachers and Students sheets17:54
th1aDon't we hide all demos that have no values?17:54
aelknerthose take the current limit groups into account17:55
aelknerth1a, values are irrelevent during mport17:55
th1aThe bug is really just this -- I added an attribute, now I can't see it.17:55
aelknerthe columns have to be fixed17:55
aelkneryou have to export to see it17:55
aelknerif they add a demo17:56
th1areplaceafill:  I don't understand what functionality we would lose if we'd show all attributes that *have values*17:56
th1aIs it because some have default values?17:56
replaceafillth1a, suppose your persons have attribute "foo", all with data17:56
aelknerok, what are we talking about 'show attributes'?  exporting or importing?17:57
replaceafillbut then you decide, this should only be used for students17:57
th1aViewing the data!17:57
replaceafillthen you make it limited17:57
th1aIn the web interface.17:57
th1areplaceafill:  That's not really a use case.17:57
replaceafillit was for cando17:57
replaceafilland the teacher_id17:57
th1aMaking an existing value limited is not a use case.17:57
replaceafilland the report_id for students17:57
th1aDid students have teacher_id's assigned?17:58
replaceafillyes, because it was the same demo field17:58
replaceafillalready with data17:58
th1aWell, that's stupid and not a use case.17:58
th1aNobody is going to start using SchoolTool today with one field and different semantics by group.17:58
replaceafillthat's what i thought, but demo fields fix that17:58
aelknerwell, the important thing is that you two know what you're talking about17:58
aelknerbecause you've lost me17:59
th1aaelkner:  The actual PROBLEM is that people are importing data and then they can't see it in the web interface.17:59
menesisif persons are in correct groups then demo fields are shown correctly17:59
replaceafillth1a, what about this:17:59
th1aBut it is there, so all we have to do is show it.17:59
replaceafillyou see the demo field in the person index view17:59
replaceafilland you say, i'll edit it17:59
replaceafillbut you don't see it in the edit form17:59
replaceafillbecause it's limited17:59
th1aThat is still an improvement.17:59
th1aAdd them to the right group and edit it.18:00
replaceafillok, i'm out :)18:00
th1aBasically, this is a situation where a common case will generate confusion and bug reports -- importing people without groups.18:00
th1aAnd there are some not very important corner cases that come up with the simple solution.18:00
aelknerkeep this in mind:18:01
th1aI'd much rather handle the "How do I delete this "Teaching Certification: NO" field I accidentally imported for a student question than the "I can't import my demographic data" question.18:02
replaceafillth1a, +118:02
aelknerwell, that18:02
aelknerthat's a different issue entirely18:02
th1aNo, THAT'S THE ISSUE.18:02
th1aThe problem is not that we aren't storing the data, right?  We're just not displaying it.18:03
th1aSo just display it!18:03
aelknerso even if the user i snot in the group, but has the demo value18:04
replaceafill-1, but ok :P18:04
aelknerdisplay it in the person view18:04
replaceafillaelkner, yes18:04
aelkneri see what th1a means, it stops hiding the data issue :)18:04
aelknerdid i say issue?18:04
aelkneri mean situation :)18:04
th1aAs long as it isn't a data MODEL issue.18:05
th1aWhich it isn't.18:05
aelknerso that's a small change to the person view, whose is it?18:05
aelknerwhich bug is this?18:05
th1aThe export should work the same way too: it should have demographic fields even if they aren't supposed to be there based on group.18:06
th1areplaceafill:  Can you do this?18:06
replaceafillit should be ready for tomorrow right?18:06
aelknerth1a, the export already does that18:06
th1aaelkner:  OK, great.18:06
th1aOK, reassigned to replaceafill.18:06
menesisth1a: other bugs you pasted are fix committed, and merged to trunk18:07
th1aOK... where were we... ah,18:07
th1aThanks menesis!18:07
replaceafilljust one quick question18:07
th1aFor some reason I thought this would be a quick meeting.18:07
menesisexcept import bug with school day names,
replaceafillthe person index view change has to be done by tomorrow, right?18:07
menesisreplaceafill: in a week18:08
replaceafillmenesis, ah ok, thanks18:08
replaceafillok, my report18:08
replaceafillafter i showed th1a my tooltip changes, we agreed on putting the tooltip above the navbar18:09
replaceafilland do the same old functionality18:09
replaceafillyou hover the skill id and you get the div updated18:09
replaceafillso i started doing that and i realized we don't need tooltips for that18:10
replaceafill(as i mentioned earlier)18:10
replaceafilland that would remove the jquery 1.9 dependency18:10
replaceafillunless you guys see we could use tooltips in some other places18:10
replaceafillth1a, do you?18:11
menesisI'd like jqueryui 1.9 reverted18:12
replaceafilldone then18:12
replaceafilli'll revert that change18:12
replaceafilli also got rid of the vertical scrolling in the tertiary navbar18:12
menesistooltips - maybe, but I think in most cases title attr is enough18:12
replaceafillwhen you have too many worksheets18:13
replaceafillmenesis, agree18:13
replaceafillnow, i'm in the middle of making that new tertiary nav scrollable18:13
replaceafillthe last unfinished task i started is auto save18:14
replaceafillth1a, here i have a question18:14
replaceafillif we implement auto-save on cells18:14
replaceafillwe're going to get rid of the Save button?18:14
replaceafillor should we leave it for the user to save a bunch of changes "quickly"18:14
replaceafillassumming auto save takes like 2-3 seconds on each cell after the last edit18:15
th1aI think we decided it should still exist.18:15
menesisGmail autosaves drafts, but it has a Save now button18:15
replaceafillah ok18:15
replaceafilli'll leave it then18:16
replaceafillth1a, last question18:16
replaceafilli can't find that bug18:16
th1aWe'll refine it after we look at it a while.18:16
menesisI think it does not hurt if a user clicks just to make sure everything i saved before leaving18:16
replaceafillbut there's a bug about exporting the journal data into xls18:16
th1aFrom jelkner?18:16
replaceafillno, that's the pdf one :)18:16
replaceafillit was someone who asked about it, i'm not sure if there's abug18:17
aelknerjelkner asked for it18:17
aelkneryes, he created a bug18:17
replaceafilljelkner asked for the pdf one i think18:17
replaceafilli'm talking about exporting the journal into xls18:18
menesisis the xls bug18:18
replaceafillah menesis thanks18:18
replaceafillthat's the one18:18
replaceafillduh! it's under the journal project :D18:18
* replaceafill was looking at trunk bugs18:19
replaceafillth1a, should i finish that one this week too?18:19
th1ayou should use schooltool-project when looking at bugs.18:19
replaceafilli remember you said you wanted it in this release18:19
replaceafillth1a, i usually do, don't know how i was redirected to only schooltool bugs18:19
th1aIf you can do it in a day or two and menesis can get it in next week, sure.18:20
replaceafillcan i move to this two bugs now and leave my cando changes for a while18:20
replaceafilli think my cando changes are not very important right now18:20
replaceafilli mean, high priority18:21
replaceafilli just want to get rid of these bugs before continuing :D18:21
th1aWell, we need to get cracking on CanDo though.18:22
th1aEsp. after this release.18:22
replaceafillth1a, i think i'm just waiting for the skills hierarchy to start the skills gradebook18:22
th1aIf your gradebook changes are becoming trivial, there's certainly more substantial stuff to be done.18:22
replaceafill*that's* the right wording18:23
replaceafillthey seem trivial18:23
replaceafillcosmetic i'd say18:23
th1aOh yes, skills hierarchy.18:23
th1aAre they not in a hierarchy now?18:24
th1aCould you work on the skills tree view?18:24
th1aThat's certainly non-trivial.18:24
replaceafillnot without yvl's changes18:24
replaceafilland i think he's on that18:24
th1aOK... so that is the major loose end at this point.18:24
replaceafillthat's why i was thinking of doing bugs instead18:24
th1aWhich is essentially blocking aelkner now too.18:25
* th1a couldn't remember what that was.18:25
th1aSo basically we're entering the worst case scenario after a good start.18:25
th1aThe data model isn't done and we're all stuck.18:25
replaceafillor i could port the gradebook view to the generic worksheet18:26
replaceafillthat i can do!18:26
menesisyvl won't finish the model while he's sick18:26
th1aWell wait, what about gradebook ajax?18:26
th1aAre you going to be done with that?18:26
replaceafillth1a, true! i can make the tabs ajaxy18:27
replaceafilli mean the worksheets18:27
th1aYes, do that.18:28
replaceafilland that's not trivial :D18:28
replaceafillok, that's it from me18:28
th1aaelkner should probably still prep for skills importer even though the hierarchy may not be done.18:28
th1aOK... thanks guys.18:29
th1aHave a great week & weekend!18:29
* th1a drops the bag of gravel.18:29
replaceafillthanks everybody18:29
aelknergreat weekend all18:30
*** replaceafill has quit IRC18:59
*** replaceafill has joined #schooltool19:01
replaceafillhhmm, is it me or the selenium tests in the journal are being translated...?19:27
replaceafillah got it, the stesting.zcml is incorrect19:30
replaceafillit's funny to see how stests have evolved when you update an old one :)19:35
*** menesis has quit IRC19:43
*** ignas has quit IRC20:07
replaceafillth1a, zyt?21:15
replaceafillhola Tomás :)21:15
replaceafilli'm working on this one:
replaceafilland i was wondering about the header of the columns for the periods21:15
th1aaelkner: ayt?21:15
replaceafillshould we use the same logic as the journal21:16
replaceafilli mean21:16
th1aWe don't need to truncate them if that's what you're asking.21:16
replaceafillif the period has its own title, use that for the export21:16
replaceafillbut we use the period title if the user has set the timetable like that, right?21:17
replaceafillor export a date if he hasn't21:17
th1aUh... I'm not sure what the alternative is?21:18
replaceafilluse a date always21:18
replaceafillthat's what the journal uses for the tooltip (title atr)21:19
th1aFor this we can add two rows in the header if it makes things more explicit.21:19
replaceafillID, first name, last name, date, date date, date...21:20
replaceafill           period, period, period, period...21:20
replaceafillsomething like that?21:20
th1aperiods below?21:20
replaceafillor the other way around?21:21
replaceafilldates below21:21
replaceafillah, i see21:21
replaceafillthe journal uses the short title always21:21
replaceafilli need to set up a different timetable to see the other case21:23
th1aJust bear in mind that we should be optimizing this for automated processing, right?21:23
replaceafillah true21:23
replaceafillin that case we should use the date instead of just the day i think21:24
replaceafillthe journal view uses:21:24
replaceafill02 | 03 | 04 | ...21:24
replaceafillthe Day21:24
replaceafillD | D | D | ...21:24
replaceafillthe Period21:24
replaceafillwhen you said two rows i actually thought of 321:25
replaceafillday, date and period21:25
replaceafillmy bad21:25
replaceafilli'll code the date and the period and show it to you21:26
replaceafillsorry for the (usual) confusion21:26
th1aOh... yes, for export, you really want the date in this case.21:26
th1aHeck, maybe it should be the date all the time.21:26
replaceafillif this is for automated processing, maybe "they" should take care of converting the date to a period ;)21:27
th1aBecause you don't want to get the wrong "C Day"21:27
replaceafillhow does the journal work if you teach the same section twice in a day in non-consecutive periods21:28
* replaceafill goes to see21:28
replaceafilli bet two columns with the same day, date and different periods...21:28
replaceafillth1a, zyt?22:36
replaceafillhelp me with the XXX parts please22:37
replaceafillReports group in the sidebar22:37
th1aPerhaps we could call this XLS Export.  ;-)22:38
replaceafillthen there's a XXX in the request dialog22:39
th1aI'm not sure what my rule of thumb is here.22:39
*** ignas has joined #schooltool22:40
th1aI guess I see a distinction between a report which is just a representation of the thing in a different format and a report that is pulling in a certain set of data.22:41
th1aI think the dialog is fine.22:42
th1aAnd maybe we should stick with Journal Export to keep aelkner from killing me.22:42
replaceafillmaybe we should create an Export group for representations22:42
replaceafilli'll remove the XXXs then22:43
replaceafillwhat about the content of the xls22:43
replaceafilldoes it look ok?22:43
th1aI guess the main point is I don't want to see growing lists of format identifiers in the reports sidebar.22:43
replaceafillor we could put the export options (representations) in the gear menu someday :)22:44
th1aNot there...22:44
th1aThe export looks ok to me.22:44
replaceafilli'll clean it up and push to trunk then22:44

Generated by 2.15.1 by Marius Gedminas - find it at!