| *** aelkner has joined #schooltool | 02:07 | |
| *** aks has joined #schooltool | 06:19 | |
| *** Lumiere has quit IRC | 06:48 | |
| *** Lumiere has joined #schooltool | 06:55 | |
| *** itsurkg has joined #schooltool | 07:23 | |
| *** itsurkg has left #schooltool | 07:23 | |
| aks | yvl: ping | 11:09 |
|---|---|---|
| yvl | hey, aks | 11:09 |
| aks | yvl: while starting schooltool instance, I get pkg_resources.DistributionNotFound: mechanize<0.2dev | 11:10 |
| aks | yvl: but the package is already installed | 11:10 |
| yvl | could it be that you got mechanize 0.2.5 installed? | 11:11 |
| aks | yvl: mine is 0.2.2 | 11:11 |
| yvl | looking at schooltool versions.cfg, our requirement is mechanize = 0.1.11 | 11:13 |
| aks | yvl: ok | 11:14 |
| yvl | do you have any software that needs mechanize 0.2.5? | 11:14 |
| yvl | it would be easiest just to downgrade it | 11:14 |
| aks | yvl: yeah trying out downgrading | 11:15 |
| yvl | thanks | 11:15 |
| aks | yvl: actually I built the required version of mechanize, but apparently there was a higher version of the package on fedora repositories | 11:15 |
| yvl | ouch | 11:15 |
| * yvl wonders which package has the <0.2dev dependency | 11:17 | |
| aks | yvl: do you have any idea which package needs roman? | 11:17 |
| aks | yvl: on Fedora the package python-docutils provides the roman dependency, but the "start-schooltool-instance" complains about not finding roman | 11:18 |
| yvl | ok, it seems that zope.testbrowser is the one giving you the trouble. | 11:23 |
| yvl | the good thing is, that you don't actually need it for the a live instance | 11:24 |
| aks | yvl: I've fixed the mechanize problem | 11:25 |
| aks | yvl: now I'm into the roman problem | 11:25 |
| yvl | I think that one is because roman.py is simply stored in a different place | 11:27 |
| yvl | probably in /usr/lib/pythonXX/pymodules or somewhere | 11:27 |
| yvl | sorry, I'm not familiar with your setup to suggest a solution :( | 11:29 |
| yvl | if python-docutils provides the dependency, and is already in Fedora repositories, it should be available in system python | 11:30 |
| yvl | you can try running sys python interpreter, "import roman", "print roman.__file__" | 11:31 |
| yvl | and try to go from there | 11:31 |
| aks | yvl: the output is /usr/lib/python2.7/site-packages/roman.pyc | 11:32 |
| yvl | hmm, can you remind me how your deployment works | 11:33 |
| yvl | does it tweak python sys.path somewhere? | 11:34 |
| aks | yvl: no | 11:34 |
| aks | yvl: it's pretty standard | 11:34 |
| aks | yvl: I've not done any changes, just tried to package into RPM format what was there | 11:34 |
| yvl | then I'm baffled | 11:35 |
| yvl | when we run dev version, we make virtual python env and filter out system site-packages | 11:36 |
| yvl | but that's probably not what your setup is doing | 11:36 |
| aks | yvl: yes, mine does not make virtual env, it uses the standard one | 11:36 |
| yvl | right | 11:36 |
| yvl | so it should see roman.py | 11:36 |
| aks | yvl: definitely | 11:37 |
| aks | yvl: but unfortunately it is not able to see in this case | 11:38 |
| yvl | well, you can try putting "import pdb; pdb.set_trace()" line in code ( zope / schoooltool ) nearest to traceback | 11:40 |
| yvl | if you run the server in *not* deamon mode | 11:40 |
| yvl | you'll get interactive console | 11:40 |
| yvl | form there you can look at sys.path, try import roman | 11:41 |
| aks | yvl: which file | 11:41 |
| yvl | can you show me the traceback? | 11:41 |
| yvl | that fails to import roman.py | 11:41 |
| aks | yvl: ok | 11:41 |
| aks | yvl: http://fpaste.org/v39V/ | 11:43 |
| yvl | damn | 11:47 |
| yvl | so, it seems that roman is not installed as an egg | 11:47 |
| yvl | hmm | 11:48 |
| yvl | I wonder how menesis solved that | 11:48 |
| aks | yvl: no idea myself | 11:52 |
| yvl | looking into it... | 11:54 |
| aks | yvl: in the previous build, I had solved the problem using a customized build of python-docutils, and then installing the python-roman package | 11:56 |
| yvl | hmm, I see menesis removed zope.app.renderer dependency on roman | 11:58 |
| yvl | ah! | 11:59 |
| yvl | here it is : https://launchpadlibrarian.net/60412603/zope.app.renderer_3.5.1-0ubuntu6.diff.gz | 11:59 |
| yvl | he indeed patches zope.app.renderer setup.py before building | 12:00 |
| yvl | hope this helps, aks | 12:02 |
| aks | yvl: so I need to apply the patch as well? | 12:02 |
| yvl | yes | 12:02 |
| yvl | wait - how are you extracting dependencies? | 12:03 |
| yvl | basically - zope.app.renderer must end without roman dependency in it's EGG_INFO/requires.txt | 12:04 |
| yvl | so you may have to patch the egg itself, depending on how you build things | 12:05 |
| yvl | either that or setup.py | 12:05 |
| aks | yvl: I'm patching setup.py | 12:08 |
| yvl | ok | 12:10 |
| aks | yvl: that solved the roman issue. Thanks | 12:13 |
| yvl | cool :) | 12:13 |
| aks | yvl: encountered another problem | 12:16 |
| aks | yvl: see http://fpaste.org/94sD/ | 12:17 |
| yvl | looking... | 12:17 |
| aks | yvl: the configure.zcml file is at http://fpaste.org/vpO0/ | 12:17 |
| yvl | hmm, line 9 <includePlugins package="schooltool" file="plugin.zcml" /> | 12:19 |
| yvl | must go after line 11 <include package="z3c.autoinclude" file="meta.zcml" /> | 12:19 |
| aks | yvl: ok | 12:19 |
| yvl | can you please remind me the link to your patch of the configure.zcml? | 12:20 |
| aks | yvl: see http://fpaste.org/oQ8D/ | 12:21 |
| yvl | right, that's what causes it | 12:22 |
| yvl | lines 7 and 11 in the patch | 12:22 |
| yvl | just remove them ;) | 12:22 |
| yvl | also line 16 looks odd | 12:23 |
| aks | yvl: in the patch file? | 12:23 |
| yvl | yes | 12:23 |
| yvl | you could try building without this patch | 12:24 |
| yvl | and if build is broken, maybe keep only lines: | 12:24 |
| aks | yvl: ok trying out a build without the patch now | 12:24 |
| yvl | # | 12:25 |
| yvl | - <includePlugins package="schooltool" file="plugin.zcml" /> | 12:25 |
| yvl | sorry, wrong line | 12:25 |
| yvl | I meant lines 12 and 14 | 12:25 |
| yvl | # | 12:25 |
| yvl | - <include package="schooltool.help" /> | 12:25 |
| yvl | + <include package="schooltool.requirement" /> | 12:25 |
| yvl | but only if build fails | 12:25 |
| aks | yvl: ok | 12:25 |
| aks | yvl: the build worked and the service started in daemon mode | 12:39 |
| aks | yvl: will be checking inside if anything fails | 12:39 |
| yvl | great! | 12:39 |
| aks | yvl: When I try to downlod the gradebook as PDF it says "PDF support is disabled. It can be enabled by your administrator." | 13:08 |
| aks | yvl: I have pyPdf installed though | 13:08 |
| yvl | reportlab_fontdir is probably commented out in schooltool.conf | 13:14 |
| yvl | at the moment we use ttf-liberation fonts in Ubuntu distribution | 13:15 |
| aks | yvl: ok, I've that font installed | 13:15 |
| aks | yvl: how can I enable it? | 13:15 |
| yvl | find schooltool.conf, and look at reportlab_fontdir line | 13:16 |
| yvl | and set a correct path to the fonts there | 13:16 |
| yvl | (schooltool.conf might be named differently... but it's probably somewhere near your Data.fs and is named .conf) | 13:17 |
| aks | yvl: yeah, got it | 13:17 |
| yvl | (it's the config_file specified in paste.ini) | 13:19 |
| aks | yvl: yeah, got it working, thanks | 13:20 |
| yvl | cool :) | 13:20 |
| aks | yvl: I'm learning a lot about schooltool | 13:20 |
| aks | yvl: I do not have a "Print Report card" in manage>person>person-X | 13:22 |
| yvl | I think it moved to [PERSON-X] / Reports / Request Report Card | 13:24 |
| aks | yvl: so that means I'll have to login as a student? | 13:24 |
| aks | yvl: found it, thanks | 13:25 |
| yvl | ;) | 13:25 |
| yvl | reports are important part of the school, so we (were and are) upgrading them | 13:25 |
| aks | yvl: :), I'm awaiting the next release with easy interface (hope i could see the mockups/screenshots soon) | 13:26 |
| * yvl hopes you'll like it ;) | 13:27 | |
| aks | yvl: intervention module does not seem to be documented at book.schooltool.org, is it? | 13:28 |
| yvl | I don't think so | 13:29 |
| yvl | apart from a short note here: http://book.schooltool.org/features.html#intervention | 13:29 |
| yvl | it's quite new | 13:29 |
| aks | yvl: ok | 13:30 |
| aks | yvl: Also I would like menesis to test my new RPMs for v1.6.0b1 which is available at http://dev.olenepal.org/schooltool-f14/ | 13:30 |
| aks | yvl: I'll be making builds for Fedora 9, 13, and F15 shortly if there is no bug in current packaging | 13:31 |
| aks | yvl: the current build is for Fedora 14 | 13:31 |
| yvl | hmm, menesis does not have a Fedora installed at hand | 13:32 |
| yvl | I'll bring this up at daily meeting today | 13:32 |
| *** menesis has joined #schooltool | 13:32 | |
| aks | yvl: menesis uses VM based Fedora to test, so do I for packaging and testing | 13:32 |
| yvl | maybe Douglas has an install :) | 13:32 |
| yvl | (replaceafill) | 13:32 |
| aks | yvl: might be | 13:32 |
| aks | yvl: one simple bug, In the detailed student report, the heading in the attendance detail spans beyond the cell boundaries | 13:37 |
| yvl | can you please file a bug at https://bugs.launchpad.net/schooltool.gradebook ? | 13:38 |
| yvl | and attach the screenshot ;) | 13:38 |
| aks | yvl: ok, I'll do that tomorrow | 14:03 |
| yvl | thanks! | 14:04 |
| *** aks has quit IRC | 14:19 | |
| *** th1a has joined #schooltool | 16:17 | |
| *** replaceafill has joined #schooltool | 16:18 | |
| *** yvl has quit IRC | 16:21 | |
| *** yvl has joined #schooltool | 16:25 | |
| *** ignas has joined #schooltool | 16:26 | |
| th1a | aelkner has power? | 16:30 |
| th1a | hi yvl, menesis, replaceafill, aelkner. | 16:31 |
| replaceafill | good morning/afternoon | 16:31 |
| menesis | hi. | 16:31 |
| yvl | morning guys | 16:31 |
| th1a | Irene was pretty weak by the time she hit us. | 16:31 |
| aelkner | morning | 16:32 |
| menesis | th1a: good to hear | 16:32 |
| aelkner | yes, i got power back last night in time to do stuff | 16:32 |
| yvl | glad to hear | 16:32 |
| menesis | aelkner: how long have you been without power? | 16:33 |
| aelkner | my complex lost power even though most neighborhoods around me didin't | 16:33 |
| aelkner | it was about 12 hours we were without it | 16:34 |
| th1a | We lost power for about 5 senonds. | 16:34 |
| th1a | Enough to make all the clocks blink. | 16:34 |
| th1a | OK, yvl, how'd it go today? | 16:36 |
| yvl | a bit slow :) | 16:36 |
| yvl | I helped aks with building ST | 16:36 |
| yvl | http://schooltool.pov.lt/irclogs/%23schooltool.2011-08-29.log.html#t2011-08-29T13:30:17 | 16:37 |
| yvl | replaceafill, I think you tested Fedora packages last time? | 16:37 |
| replaceafill | yes | 16:37 |
| yvl | so if you have some spare time, it would be cool to check those out | 16:37 |
| replaceafill | ah ok, will do | 16:38 |
| yvl | other than this, I replaced gradebook activity z3c option storage with a simple container | 16:38 |
| yvl | + evolution | 16:38 |
| yvl | now I'm working on the views | 16:38 |
| yvl | that's it from me. | 16:39 |
| replaceafill | gradebook categories ;) | 16:39 |
| yvl | right, those :))) | 16:39 |
| th1a | Yeah... | 16:39 |
| th1a | ? | 16:40 |
| aelkner | what's all the simlicons about, is there a joke i'm missing? | 16:40 |
| aelkner | i meant smilicns | 16:41 |
| yvl | umm, no joke there :) | 16:41 |
| aelkner | just a bunch of smiley developers | 16:42 |
| th1a | Did you work on them? | 16:42 |
| aelkner | me? | 16:42 |
| menesis | I am happy to get rid of z3c.optionstorage dependency | 16:42 |
| yvl | we'll have to keep it for one more release probably | 16:42 |
| th1a | Did yvl work on gradebook categories? | 16:43 |
| yvl | or enhance evolution to use mocking | 16:43 |
| aelkner | i didn't know we were getting rid of that, but it sounds like a good idea | 16:43 |
| yvl | th1a, povbot was out, so no IRC logs, but as far as I remember, you mentioned - yvl could do the activity categories page | 16:43 |
| yvl | yes, it's an old abandoned package we have to maintain aelkner | 16:43 |
| yvl | and it gives us no benefit whatsoever | 16:44 |
| aelkner | true | 16:44 |
| aelkner | that's from the original gradebook implementation | 16:45 |
| yvl | right | 16:45 |
| th1a | Oh, I see, you moved the categories out of optionstorage? | 16:45 |
| yvl | yes | 16:45 |
| th1a | OK, I didn't catch the connection. | 16:45 |
| th1a | What about the page itself? | 16:45 |
| yvl | just started it, some hour ago or so | 16:46 |
| yvl | sorry, it's really a slow day for me | 16:46 |
| th1a | OK. | 16:46 |
| th1a | We're getting to the end of the loose ends though, so we want to get them all put to bed asap. | 16:46 |
| th1a | How's that for a mixed metaphor. | 16:48 |
| th1a | OK anything else, yvl? | 16:48 |
| yvl | nope | 16:48 |
| th1a | Thanks. | 16:49 |
| th1a | replaceafill? | 16:49 |
| th1a | I guess we were working together Friday afternoon, so there shouldn't be much. | 16:49 |
| th1a | ;-) | 16:49 |
| replaceafill | th1a right, i just fixed some of the styles i missed that day | 16:50 |
| replaceafill | http://69.164.203.135:6660/scoresystems | 16:50 |
| th1a | Did you check our work into trunk? | 16:50 |
| replaceafill | yes | 16:50 |
| replaceafill | i don't think this is much, but here you go: http://img97.imageshack.us/img97/7717/guidelinesf.png | 16:51 |
| replaceafill | that's our column history | 16:51 |
| replaceafill | i took 16px from the current content space | 16:51 |
| replaceafill | the only remaining issue is the third-nav | 16:52 |
| replaceafill | but i have an idea on how to fix it | 16:52 |
| replaceafill | i'll work on that first thing today | 16:52 |
| th1a | OK. | 16:53 |
| replaceafill | th1a i was thinking of doing flourish work today and start with cando tomorrow | 16:54 |
| replaceafill | if that's ok | 16:54 |
| th1a | That should be fine. | 16:54 |
| replaceafill | ok | 16:54 |
| replaceafill | that's it from me | 16:54 |
| th1a | Thanks replaceafill. | 16:54 |
| th1a | aelkner? | 16:54 |
| aelkner | i moved Report Sheet Templates from the School's main page to the Customize sidebar | 16:55 |
| aelkner | then i added Report Sheets for getting to the manage deployed sheets | 16:55 |
| aelkner | as manager, http://69.164.203.135:36660/manage | 16:55 |
| aelkner | i put the table at the top followed by the form for deploying another sheet | 16:56 |
| aelkner | it has the tertiary control as the other manage views have | 16:56 |
| aelkner | a couple of things: | 16:56 |
| aelkner | if you don't select a template and hit submit, it just does nothing and returns to the same place | 16:56 |
| aelkner | it could have a red error message in that case | 16:57 |
| th1a | Sure, why not. | 16:57 |
| aelkner | otherwise, there is a data model issue to deal with, but first let | 16:57 |
| aelkner | let's go over the cosmetics | 16:57 |
| th1a | Didn't we add hiding report sheets at some point? | 16:57 |
| aelkner | we did | 16:58 |
| aelkner | wiat, let me check | 16:58 |
| aelkner | no, not yet | 16:58 |
| aelkner | i think you decided against it at the time | 16:59 |
| aelkner | perhaps because the data is not some whim of a teacher who could just hide a worksheet any time | 16:59 |
| aelkner | without impacting the report card generation | 16:59 |
| th1a | OK, we'll come back to that next release probably. | 16:59 |
| aelkner | any cosmetic concerns before i bring up the data model issue? | 17:00 |
| th1a | We should have all terms listed with checkboxes, because you might have four or six terms. | 17:01 |
| th1a | And want to deploy to half of them. | 17:01 |
| aelkner | i don't understand, all terms are already listed | 17:01 |
| th1a | A message if you don't have any RST's would be very helpful. | 17:01 |
| th1a | Of course it is possible to do it this way, | 17:02 |
| aelkner | RST? | 17:02 |
| th1a | Er | 17:02 |
| th1a | report sheet templates. | 17:02 |
| aelkner | ah, yes, that, what should the view have in that case? | 17:02 |
| aelkner | no form? | 17:02 |
| th1a | but if you were deploying to terms 2, 4 , and 6 you'd have to do it three times. | 17:03 |
| menesis | th1a: hiding report sheets feature was implemented. and released. but only from trunk | 17:03 |
| menesis | it is not merged to flourish | 17:03 |
| menesis | sorry | 17:03 |
| th1a | Ah! | 17:03 |
| aelkner | ahha! | 17:03 |
| th1a | I thought so. | 17:03 |
| th1a | Could we correct that today? | 17:03 |
| aelkner | didn't think to check back there | 17:03 |
| menesis | I am looking at all the different branches since friday and will merge what is missing | 17:04 |
| menesis | including gradebook changes in trunk | 17:04 |
| th1a | If there are no templates, you could just replace the drop-down with an explanation and a link to the RST index. | 17:04 |
| aelkner | but keep the form? | 17:04 |
| th1a | I guess you could just replace the whole form. | 17:05 |
| aelkner | ok, sounds right | 17:05 |
| aelkner | any other similar suggestions? | 17:05 |
| th1a | Make sure you have a "there are none" type message if there are no currently deployed sheets. | 17:05 |
| th1a | Otherwise, seems fine. | 17:05 |
| aelkner | click on the other year | 17:06 |
| menesis | and a link to report sheet templates | 17:06 |
| aelkner | i copied the This year idea from /courses | 17:06 |
| th1a | kk | 17:06 |
| aelkner | it also didn't say 2008-2009 has no... | 17:06 |
| aelkner | because the tab has the year, right? | 17:06 |
| aelkner | although the School view has a year tab and still says the year in the messages | 17:07 |
| aelkner | is that because the view is so long it helps the user to redundantly see the year in question? | 17:08 |
| th1a | Helping the user keep track of the year is good. | 17:08 |
| th1a | People will make mistakes when they start looking at the wrong year. | 17:08 |
| th1a | That's one of the potential problems with the current design. | 17:08 |
| aelkner | sure, but what about /courses and /report_sheets not showing the year? | 17:08 |
| th1a | ? | 17:09 |
| aelkner | is that ok because they are simpler views? | 17:09 |
| aelkner | go to /courses for a year with no courses | 17:09 |
| aelkner | it says This year has no courses defined | 17:09 |
| th1a | Oh. | 17:09 |
| aelkner | not $YEAR has no course defined | 17:09 |
| aelkner | what's you preference there? | 17:10 |
| th1a | aelkner: I have no strong feeling about it, but probably stating the year explicitly is better in every case. | 17:10 |
| aelkner | i could change all the / views to do that | 17:10 |
| aelkner | ok, how about the data model issue now? | 17:10 |
| th1a | In fact, we should probably put the years in all the index titles, like "Courses for 2008-2009" | 17:10 |
| aelkner | ah, yes | 17:10 |
| aelkner | let me be the one to go over those views and ake those adjustments | 17:11 |
| th1a | OK | 17:11 |
| aelkner | i like having th year in the title | 17:11 |
| aelkner | it teaches the user | 17:11 |
| aelkner | ok, the data model issue | 17:11 |
| aelkner | back in the day, i was not sure what to do about multiple deployments, so i allowed it | 17:12 |
| replaceafill | i think /courses has that right? http://69.164.203.135:6660/courses | 17:12 |
| replaceafill | "Courses for 2008-2009" | 17:12 |
| aelkner | ok, but still needs the $YEAR has no courses | 17:12 |
| aelkner | i'll go over all those views as one commit | 17:13 |
| aelkner | getting back to the data model issue... | 17:13 |
| aelkner | the idea is this, looking at the table i just coded for the new skin, you see it has one row per template | 17:14 |
| aelkner | but we don't stop the user from deploying the same template twice | 17:14 |
| menesis | I think Courses for $YEAR is troublesome for translators | 17:14 |
| menesis | also, $PERSON's... Calendar | 17:14 |
| th1a | aelkner: Can you just file that as a bug? | 17:16 |
| menesis | on the other hand, explicitly saying what year you are looking at is better | 17:16 |
| menesis | than in the old skin when often it was not visible anywhere | 17:16 |
| aelkner | th1a, so we should deal with that after the release, right? | 17:16 |
| aelkner | it would require evolution and decisions on our part | 17:17 |
| th1a | aelkner, Yes, put it off. | 17:17 |
| th1a | But please file a bug. | 17:17 |
| th1a | menesis: How is it troublesome? | 17:17 |
| menesis | https://bugs.launchpad.net/schooltool.gradebook/+bug/643676 | 17:18 |
| menesis | There's no way to know which report sheet templates have been deployed already | 17:18 |
| menesis | this one is solved I see | 17:18 |
| menesis | but I know that it is still possible to deploy the same report sheet many times | 17:18 |
| menesis | th1a: it is hard to translate to Lithuanian at least... but there were strings like 'Courses of $year' and similar before | 17:20 |
| menesis | so nevermind | 17:20 |
| th1a | Oh, yes. | 17:20 |
| th1a | RIght. | 17:20 |
| aelkner | th1a, but filed: https://bugs.launchpad.net/schooltool/+bug/836732 | 17:21 |
| aelkner | bug | 17:21 |
| th1a | Good point though menesis. Thanks for reminding me of the possessives issue. | 17:22 |
| th1a | aelkner: Any other views in your queue? | 17:22 |
| aelkner | i need to simplify the report sheet template activities view to not have th title form | 17:22 |
| aelkner | and put a pencil next to the title in that view to get to the edit | 17:23 |
| aelkner | but that won't take long | 17:23 |
| th1a | I want to go over intervention with you and clean up the text and add some "What is This?" | 17:24 |
| aelkner | i noticed that the Interventions link from the person page is broken, so i should look into that | 17:24 |
| aelkner | ah, ok, we could do that next | 17:24 |
| th1a | replaceafill: Can you get aelkner up to speed on adding the right sidebar? | 17:24 |
| aelkner | i just need a revision link | 17:24 |
| replaceafill | it's very simple really | 17:24 |
| replaceafill | aelkner just look for what-is-this in the code :) | 17:25 |
| aelkner | sure | 17:25 |
| replaceafill | i called most of the viewlets like that | 17:25 |
| replaceafill | and it's just a template | 17:25 |
| replaceafill | with the header and the body | 17:25 |
| aelkner | is it Inline like the tertiary ones? | 17:25 |
| replaceafill | registered for the ipagerelated manager | 17:25 |
| replaceafill | aelkner no | 17:25 |
| replaceafill | it's a .pt file for each | 17:25 |
| replaceafill | with the whole structure | 17:26 |
| aelkner | i love how those tertiary classes don't even need a template | 17:26 |
| aelkner | we should do that kind of thing more often | 17:26 |
| replaceafill | hhmm we need a template for the body | 17:26 |
| replaceafill | of the viewlet | 17:26 |
| replaceafill | i did want to remove the title part though | 17:26 |
| replaceafill | doing something like: | 17:26 |
| replaceafill | title="What is This?" | 17:26 |
| replaceafill | body_template=".../foo.pt" | 17:26 |
| replaceafill | but i got the template name rendered in the output | 17:27 |
| replaceafill | so, for speed reasons, i went with the whole template approach, which is not very different anyway | 17:27 |
| replaceafill | <flourish:viewlet | 17:28 |
| replaceafill | name="what-is-this" | 17:28 |
| replaceafill | class="schooltool.skin.flourish.page.Related" | 17:28 |
| replaceafill | template="templates/f_sections_related.pt" | 17:28 |
| replaceafill | view=".section.FlourishSectionsView" | 17:28 |
| replaceafill | manager="schooltool.skin.flourish.page.IPageRelatedManager" | 17:28 |
| replaceafill | permission="schooltool.edit" | 17:28 |
| replaceafill | /> | 17:28 |
| aelkner | so is it already the way you want it to be, or do you need to make an adjustment? | 17:28 |
| replaceafill | aelkner , you just need to change template and view | 17:28 |
| replaceafill | aelkner i won't make any adjustment | 17:28 |
| aelkner | ok, i'll be able to follow your pattern with no problem | 17:28 |
| replaceafill | yep | 17:29 |
| aelkner | looks clean enough, even with the naming convention for the template | 17:30 |
| replaceafill | :) | 17:31 |
| replaceafill | yvl question | 17:32 |
| yvl | yes? | 17:32 |
| replaceafill | could you share your apache rewrite rules | 17:32 |
| replaceafill | i'd like to see the issues with the buttons you described | 17:32 |
| yvl | ok, I'll email you those | 17:33 |
| replaceafill | thanks | 17:33 |
| yvl | the rules are probably stupid, but anyway - it's wrong that stuff breaks with them | 17:33 |
| replaceafill | why do they break? because we dont use @@absolute_url calls in the templates? | 17:34 |
| replaceafill | or the links | 17:34 |
| yvl | yes | 17:35 |
| yvl | they point to http://localhost/schoolyears | 17:35 |
| yvl | when ST is under http://localhost/schooltool/... | 17:35 |
| aelkner | is this a case where someone says href="something.html" instead of: | 17:35 |
| aelkner | tal:attrubutes="href .string:${context/absolute_url}/something.html"? | 17:36 |
| aelkner | forget the . part before string, a typo | 17:36 |
| yvl | yes | 17:37 |
| yvl | but not href="something.html" | 17:37 |
| yvl | but href="/some/foo.html" | 17:37 |
| aelkner | is that one ok or not? | 17:37 |
| aelkner | href="something.html", i mean | 17:37 |
| yvl | I think so | 17:37 |
| yvl | haven't checked, frankly | 17:37 |
| aelkner | because i like using it, but i'll stop if i have to | 17:38 |
| yvl | I think that's ok | 17:38 |
| aelkner | ok, cool | 17:38 |
| yvl | just that /shcoolyears go to well, the root | 17:38 |
| yvl | which is not always at the root (mine is at http://localhost/schooltool) | 17:39 |
| aelkner | a lot of these links are in our zcml, aren't they? | 17:39 |
| aelkner | for menu items? | 17:39 |
| aelkner | or is it just in the templates that the offending links are rendered? | 17:39 |
| yvl | just templates | 17:40 |
| replaceafill | i loved this one: | 17:40 |
| replaceafill | <flourish:viewlet | 17:40 |
| replaceafill | name="../.." | 17:40 |
| replaceafill | before="../weights.html" | 17:40 |
| replaceafill | title="Worksheets" | 17:40 |
| replaceafill | class="schooltool.skin.flourish.page.LinkViewlet" | 17:40 |
| replaceafill | manager=".gradebook.FlourishGradebookSettingsLinks" | 17:40 |
| replaceafill | permission="schooltool.edit" | 17:40 |
| replaceafill | /> | 17:40 |
| replaceafill | it's like aelkner giving me the finger | 17:40 |
| replaceafill | (in the name) | 17:40 |
| aelkner | i saw your chat with th1a, haha | 17:40 |
| yvl | :D | 17:40 |
| replaceafill | :D | 17:40 |
| aelkner | but .. would be ok for the same reason href="something.html is ok, right? | 17:41 |
| aelkner | it's only a root issue | 17:41 |
| yvl | yes | 17:41 |
| yvl | when you render a href that starts with / | 17:41 |
| replaceafill | yvl do you remember a link that gives the issue? so i can test it? | 17:42 |
| replaceafill | (im setting up apache) | 17:42 |
| yvl | and ../.. is also not the nicest way to give the finger, aelkner | 17:42 |
| yvl | term/browser/templates/f_term.pt | 17:42 |
| aelkner | it's a bit of a limp finger, so i wouldn't be so inclined :))) | 17:42 |
| yvl | schoolyear/browser/templates/f_schoolyear.pt | 17:43 |
| yvl | and so on | 17:43 |
| yvl | the Done link | 17:43 |
| replaceafill | :| | 17:43 |
| replaceafill | bad aelkner! | 17:43 |
| aelkner | heah, i didn't broach the subject :) | 17:43 |
| yvl | ok then... some gravel, th1a? :) | 17:45 |
| th1a | menesis: We'll hopefully have all the strings set tomorrow this time. | 17:45 |
| * th1a drops the bag of gravel. | 17:46 | |
| th1a | aelkner: Meet back here in... 30? | 17:46 |
| aelkner | ok | 17:46 |
| menesis | ok | 17:46 |
| yvl | see you guys tomorrow... or on Wed? | 17:46 |
| th1a | Let's do Mon/Tue/Thurs. | 17:46 |
| th1a | This week, at least. | 17:47 |
| yvl | ok | 17:47 |
| yvl | have a great day, guys :) | 17:47 |
| replaceafill | thanks everybody | 17:47 |
| aelkner | a great day/evening to all | 17:48 |
| th1a | hi aelkner | 18:17 |
| aelkner | hey | 18:19 |
| aelkner | http://69.164.203.135:36660/report_sheets?schoolyear_id=2009-2010 | 18:19 |
| aelkner | i didn't change the This year to 2009-2010 | 18:20 |
| aelkner | i will, but just thought i'd show you without | 18:20 |
| aelkner | is there a point where telling the user which year it is n the form gets annoying? | 18:20 |
| th1a | aelkner, I don't have a strong feeling either way. | 18:20 |
| aelkner | ok, then it's ok as is | 18:21 |
| th1a | Relatively speaking, new users will be the ones who don't have report sheets. | 18:21 |
| th1a | Let's just move on. | 18:21 |
| aelkner | sure | 18:21 |
| aelkner | so i see that adding page related content is technically not difficult | 18:22 |
| th1a | One would hope not. | 18:22 |
| aelkner | it's just a matter of deciding what text goes there | 18:22 |
| aelkner | if at all | 18:23 |
| aelkner | not all views will hae What's this, will they? | 18:23 |
| th1a | No. | 18:23 |
| aelkner | right, so we need to go view by view | 18:23 |
| th1a | We're pretty much done with it except for intervention and report sheets. | 18:23 |
| th1a | But also we need to make other changes in intervention. | 18:23 |
| aelkner | what's the first one you'd like to look at | 18:23 |
| th1a | Start with the teacher dashboard. | 18:24 |
| aelkner | oh, i may need to merge with intervention trunk if someone else made changes | 18:24 |
| aelkner | checking... | 18:24 |
| aelkner | yes, merging... | 18:25 |
| aelkner | ok, i merged, pushed, reloaded my demo instance | 18:27 |
| aelkner | so, for the teacher dashboard, what should the What's this box contain? | 18:28 |
| th1a | This needs a lot more changes, actually. | 18:28 |
| th1a | Let's change "Dashboard" to "Inbox" | 18:28 |
| th1a | And "Inbox" to "Search" | 18:28 |
| aelkner | would "Search Inbox" help avoid confusion with the Search sidebar? | 18:30 |
| th1a | Well, perhaps we should just make it conceptually less confusing. | 18:32 |
| th1a | Why do we need two? | 18:32 |
| aelkner | well, there's two different data sets | 18:33 |
| aelkner | one is the set of all interventions, regardless of who that has the user responsible, the inbox | 18:33 |
| aelkner | the other is the set of links to intervention student views | 18:33 |
| aelkner | both sets are interesting to the user depending on the mode they are in | 18:33 |
| th1a | I think this is one search. | 18:34 |
| aelkner | the inbox is a way of seeing what's going on with ongoing things | 18:34 |
| aelkner | one search? | 18:34 |
| aelkner | two totally different sets of data | 18:34 |
| th1a | Well, if I search for a student with an intervention, I get a link to the intervention and a link to the student views. | 18:35 |
| th1a | If the student doesn't have an intervention I just get the link to the student view. | 18:35 |
| aelkner | no | 18:35 |
| th1a | I'm saying we could do that. | 18:35 |
| aelkner | oh | 18:35 |
| aelkner | you'll need to spell this out a bit more | 18:36 |
| aelkner | we're talking the Inbox view, right? | 18:36 |
| th1a | Yes. | 18:36 |
| th1a | The filter is the key here. | 18:37 |
| aelkner | the data set could be sic messages for one student, two goals and a message for another | 18:38 |
| aelkner | the teacher may not know the exact name, so the filter returns any student that matches the filter | 18:38 |
| aelkner | a simple 'a' returns all students with 'a' in the name or id | 18:39 |
| aelkner | i thought the idea of the inbox was mostly like an email box | 18:40 |
| aelkner | it's sorted by date by default (i believe), reverse order? | 18:40 |
| aelkner | checking... | 18:40 |
| th1a | It is. | 18:41 |
| th1a | As you said though, the two searches are a little confusing. | 18:41 |
| th1a | How about checkboxes for: | 18:41 |
| th1a | Show: | 18:42 |
| th1a | [ ] goals | 18:42 |
| th1a | [ ] messages | 18:42 |
| th1a | [ ] all available students. | 18:42 |
| th1a | with the first two selected by default. | 18:42 |
| aelkner | i'm not sure what that means though | 18:43 |
| aelkner | i admit that i confuse easily, but i think i'm supporting the point you made about the confusion | 18:43 |
| aelkner | here' the thing with confusion | 18:43 |
| aelkner | sometimes the only solution is devide and concur, like in solving algebra equations | 18:44 |
| aelkner | if we explain to the user the data set they are seeing, i don't see why it is so bad to have to inbox And the student search | 18:44 |
| aelkner | ok, but perhaps you don't see why i'm confused with your suggestion above | 18:44 |
| aelkner | perhaps a little flooding (just 8 lines) of a result set you'd like to see would help | 18:45 |
| th1a | Do you understand the first two? | 18:45 |
| aelkner | what are the columns of the table first | 18:45 |
| th1a | Let's assume it is the same. | 18:46 |
| aelkner | so the first two columns would have links that go right to the student intervention venter | 18:46 |
| aelkner | center | 18:46 |
| th1a | y | 18:46 |
| aelkner | but col 3 would go to the intervention itself | 18:46 |
| aelkner | and 4 | 18:46 |
| th1a | y | 18:46 |
| aelkner | hm | 18:46 |
| aelkner | so, if all available students is not checked | 18:47 |
| aelkner | then only those message and goals (when both checked) | 18:47 |
| aelkner | that the user is responsible would appear | 18:47 |
| aelkner | and the student links are built into the reslt set | 18:47 |
| aelkner | then | 18:47 |
| aelkner | if they wanted to see students for whom they are not currently responsible | 18:48 |
| aelkner | they click the third one | 18:48 |
| aelkner | those student with no interventions for the user would have those cells blank as you said | 18:48 |
| th1a | The use case here is the teacher wants to create an intervention for a student that doesn't have one. | 18:48 |
| th1a | Or, I suppose look up other interventions from other teachers for a student. | 18:48 |
| th1a | So it is only relevant ever to students they have a relationship with. | 18:49 |
| aelkner | right, those are two common cases | 18:49 |
| th1a | I'm not sure that there are any others. | 18:49 |
| aelkner | so if all three are checked you get a mixed set of data | 18:50 |
| aelkner | but wait | 18:50 |
| th1a | That's really the question. | 18:50 |
| aelkner | when you say "students they have a relationship with" | 18:51 |
| aelkner | are you saying the result set should only contain students that they teach or advise? | 18:51 |
| aelkner | students for which they come up in the peson list? | 18:51 |
| aelkner | that may be tru already, but i'd have to check | 18:52 |
| th1a | It should be true already. | 18:52 |
| aelkner | let's assume it is | 18:52 |
| aelkner | for now at least | 18:52 |
| aelkner | but your ok with the mixed data set of 25 messages for three students 10 goals for four others and whatever students are left that have neither | 18:53 |
| aelkner | with only one row for each of those | 18:53 |
| aelkner | i guess i'm asking because i think those students would get lost in the rest of the data | 18:53 |
| th1a | Probably we should just have a subheader and put them below. | 18:53 |
| th1a | They can't be sorted by date, obviously. | 18:53 |
| aelkner | ah, yes, that | 18:53 |
| aelkner | and i was abpout to mention that this is a table formatter view, so customizing it is more work intensive | 18:54 |
| aelkner | i'm not even sure how i'd render the tale differently using table formatter | 18:54 |
| th1a | More work intensive than what? | 18:55 |
| aelkner | but your point supports my concern about the mixed result set | 18:55 |
| aelkner | when i make table views without using table formatter, i can do them quickly | 18:56 |
| aelkner | and if you ever ask for something special, it's easy | 18:56 |
| th1a | So... do we need to use the table formatter? | 18:56 |
| aelkner | just add to the page teemplate | 18:56 |
| aelkner | no | 18:56 |
| aelkner | but wait | 18:56 |
| aelkner | my felling is this, table formatter has it's place | 18:56 |
| aelkner | most f the time i don't think it's necessary | 18:57 |
| aelkner | and over engineering to boot | 18:57 |
| aelkner | also, it annoys you because it wastes developer time | 18:57 |
| aelkner | however | 18:57 |
| aelkner | there are cases when it use is essential | 18:57 |
| aelkner | the inbox, like /persons can have hundreds in the result set | 18:58 |
| aelkner | in which case, the table formatter solves more problems than it causes | 18:58 |
| aelkner | it's just a little inflexible to customize | 18:58 |
| aelkner | i'd recommend this | 18:59 |
| aelkner | since the result set is mixed,, why not have a second table below the first? | 18:59 |
| th1a | That would be fine. | 18:59 |
| aelkner | uh, oh | 18:59 |
| aelkner | i just realized, that brings us back to the old skin view | 18:59 |
| aelkner | with the two fieldsets | 19:00 |
| aelkner | you had me split that out to have the students search in the sidebar | 19:00 |
| aelkner | i liked that decision to be honest, and i'm ot sure exactly why you don't | 19:01 |
| th1a | Well, it was two different searches before? | 19:01 |
| aelkner | yes | 19:01 |
| aelkner | it was bad before | 19:01 |
| aelkner | the first search box did not display the result set in place | 19:02 |
| aelkner | it redirected to a view with only student search results | 19:02 |
| aelkner | your decision to make that an explicit sidebar request made it better in my opinion | 19:02 |
| aelkner | at least the user knew they were drawing themselves away from the inbox | 19:02 |
| aelkner | i think that helps them know that they are thinking about a different data set | 19:03 |
| aelkner | just students | 19:03 |
| aelkner | no goals or messages or missing students that don't have any yet | 19:03 |
| th1a | We could put a search field in the sidebar. | 19:04 |
| th1a | OH. | 19:04 |
| aelkner | wait, you're onto somting there | 19:04 |
| th1a | OK, here's the damn problem, | 19:04 |
| th1a | The main content area isn't "Search" it is "Filter" | 19:05 |
| aelkner | back up and mention what you click first before the context are is as you describe | 19:06 |
| aelkner | Search->Students? | 19:06 |
| th1a | En Engles? | 19:06 |
| th1a | Habla espanol? | 19:07 |
| aelkner | btw, i just noticed that the search students view is not a table formatter view in the flourish case | 19:07 |
| th1a | What do you want me to back up to? | 19:07 |
| *** ignas has quit IRC | 19:07 | |
| aelkner | sorry, it is a table formatter view | 19:08 |
| aelkner | oh, i was't sure what you were talking about there | 19:08 |
| aelkner | The main content area isn't "Search" it is "Filter" | 19:08 |
| aelkner | what main context area? | 19:08 |
| th1a | I mean the form in the main content area. | 19:08 |
| th1a | You aren't searching the messages, etc., you're filtering them. | 19:09 |
| aelkner | in the inbox, you mean | 19:09 |
| th1a | (admittedly similar concepts) | 19:09 |
| th1a | Yes. | 19:09 |
| aelkner | ah | 19:09 |
| aelkner | yes, it is only a filter | 19:09 |
| aelkner | good point | 19:09 |
| aelkner | that's true for People, too, right? | 19:09 |
| th1a | Not so much. | 19:10 |
| th1a | Sort of. | 19:10 |
| th1a | But less so. | 19:10 |
| aelkner | yes, the legend says Search | 19:10 |
| aelkner | but the dropdown says filter | 19:10 |
| aelkner | i'm not sure that's so bad though | 19:10 |
| th1a | You search for a person. | 19:10 |
| th1a | You don't filter all the other people. | 19:11 |
| aelkner | so, we could call it Filter in Inbox and still call it Search in the search students view? | 19:11 |
| th1a | That's the idea. | 19:11 |
| aelkner | doing that is much less involved then removing the sidebar and trying to stuff it back into the inbox view | 19:12 |
| th1a | Yes yes. | 19:12 |
| aelkner | so back to the filter box in the inbox | 19:12 |
| aelkner | legend: Filter | 19:12 |
| aelkner | keep Student Name and Show only goals? | 19:13 |
| aelkner | just change the legend? | 19:13 |
| th1a | Change show only goals to | 19:13 |
| th1a | Show only: | 19:13 |
| th1a | [ ] Goals | 19:13 |
| th1a | [ ] Messages | 19:13 |
| aelkner | or Show: rather than Show only: | 19:13 |
| th1a | Or just "Show:" | 19:13 |
| th1a | Yes. | 19:14 |
| aelkner | with both pre-selected | 19:14 |
| th1a | Waitasec. | 19:14 |
| th1a | Search students shouldn't really be any different than a persons search. | 19:15 |
| th1a | You really shouldn't need a different view at all there. | 19:15 |
| *** menesis has quit IRC | 19:15 | |
| th1a | Also, I'm getting a traceback when I try to click on a search result there... | 19:16 |
| aelkner | yes, i mentioned that in the meeting, actually about the sidebar link in the person view | 19:16 |
| aelkner | but it's the same issue | 19:16 |
| aelkner | i'll look into that today | 19:17 |
| aelkner | but getting back to the person search | 19:17 |
| aelkner | there are some key differences | 19:17 |
| aelkner | 1) the result set should only contain students the user can see from an intervention permission standpoint | 19:17 |
| aelkner | 2) the result set being filtered by group seems arbitrary | 19:18 |
| aelkner | finally, 3) the goals only filter is there for a reason i suppose | 19:18 |
| aelkner | unless you don't agree with Lehman's idea there | 19:18 |
| th1a | We can leave it for now. | 19:18 |
| aelkner | you know, that may not be interesting anymore | 19:19 |
| aelkner | so point 3) is not important | 19:19 |
| th1a | Let's not fiddle with that view other than to make sure it works. | 19:19 |
| aelkner | that works for me | 19:19 |
| th1a | OK. | 19:19 |
| th1a | So we also need a helpful no result message for the inbox. | 19:20 |
| th1a | "You have no intervention goals or messages in your inbox." | 19:21 |
| th1a | ? | 19:21 |
| aelkner | the message sounds right | 19:22 |
| aelkner | i just have to figure out how to get it to come out with table formatter | 19:22 |
| aelkner | also, changing the goals only to show: is probably a widget rewrite/reregster task | 19:22 |
| th1a | OK. Get started on that stuff. | 19:23 |
| aelkner | what about What's this? | 19:23 |
| aelkner | do you want to send me an email | 19:23 |
| th1a | No, I want you around today. | 19:23 |
| th1a | We need to finish this. | 19:23 |
| aelkner | i have a doctor's appointment at 3:45 | 19:23 |
| th1a | OK. | 19:24 |
| replaceafill | th1a could you please check the third-navs in http://69.164.203.135:6660/ to make sure they resize ok | 19:24 |
| th1a | We need to go through all the intervention views to make sure they make sense and I understand what they actually do and then I'll do the what's this. | 19:24 |
| aelkner | sure | 19:24 |
| replaceafill | th1a i think i checked them all, but you never know :) | 19:24 |
| aelkner | replaceafill, how do you mean checked them all | 19:25 |
| th1a | looked at? | 19:25 |
| aelkner | oh, sorry, didn't see your first message | 19:25 |
| replaceafill | :) | 19:25 |
| aelkner | way to mix yourself in there replaceafill :) | 19:26 |
| th1a | We'll find them eventually if you missed any. | 19:26 |
| replaceafill | kk, sorry for interrupting | 19:26 |
| aelkner | th1a, so i'll worry about the table formatter changes later | 19:27 |
| aelkner | we can discuss what you just mentioned now | 19:27 |
| th1a | ForbiddenAttribute: ('context', <schooltool.intervention.intervention.InterventionMessages object at 0xd87526c>) | 19:29 |
| th1a | We may have a little basic debugging first. | 19:29 |
| th1a | Try to create a message. | 19:30 |
| aelkner | you can't even go to the student intervention view, right? | 19:32 |
| aelkner | we have a bug with the student intervention view regarding report sheets | 19:32 |
| aelkner | that's because we ddnt' have any at the time, so i could debug it | 19:32 |
| aelkner | i'll need to do that directly | 19:32 |
| aelkner | but how are you adding a essage? | 19:32 |
| th1a | I can get to the intervention dashboard by using search in the intevention sidebar. | 19:33 |
| th1a | Just not the link in the person view. | 19:33 |
| aelkner | did you deploy any report sheets yet? | 19:35 |
| aelkner | to the schoolyear in question? | 19:35 |
| th1a | I don't know. | 19:35 |
| aelkner | i fxed the bug, need to push/load | 19:35 |
| aelkner | ok, loaded into instance | 19:39 |
| aelkner | now, the add messages bug | 19:39 |
| aelkner | th1a, found it, fixed it, loaded it | 19:42 |
| aelkner | try again now | 19:42 |
| aelkner | i accidentally introduced the add message bug while getting add message to work from the section interventions view | 19:43 |
| th1a | It shouldn't let me try to send an email when email is disabled on the server. | 19:44 |
| aelkner | true | 19:44 |
| aelkner | haven't worked out the kinks of sending email yet | 19:45 |
| aelkner | i just got the widget rendered for your cosmetic review | 19:45 |
| aelkner | the issue with email is this | 19:45 |
| th1a | I think "Messages and Observations" need to become "Messages" | 19:45 |
| th1a | And "Goals and Interventions" need to be "Goals" | 19:45 |
| aelkner | ok | 19:45 |
| th1a | Are "Change of Status Messages" considered "Messages?" | 19:46 |
| aelkner | yes, with special flag set | 19:47 |
| aelkner | in order to separate them out | 19:47 |
| aelkner | the idea is that expelling a student is a one time thing | 19:47 |
| aelkner | and shouldn't get lost in all the other messages | 19:47 |
| aelkner | also, easy find these important message this way | 19:48 |
| th1a | Uh... | 19:48 |
| th1a | Oh. | 19:48 |
| th1a | Let's get rid of this. | 19:48 |
| th1a | It is just a special message type? | 19:49 |
| aelkner | flag, yes | 19:49 |
| th1a | Are they ever automatically generated by the system? | 19:49 |
| aelkner | no | 19:49 |
| th1a | Lets shitcan the whole thing. | 19:50 |
| th1a | For now just rip out that section in the student view. | 19:51 |
| th1a | I feel better already. | 19:51 |
| aelkner | here's what i suggest | 19:51 |
| aelkner | we don't change the data model that would break the old skin views, but | 19:52 |
| aelkner | we do change the flourish views to not filter that flag | 19:52 |
| th1a | Just take it out of the new skin. | 19:52 |
| aelkner | so messages will be all messages regardless of whether they have the flag set | 19:52 |
| aelkner | that way, old data doesn't get lost in the new views | 19:53 |
| aelkner | we just take away their abililty to set the flag while adding | 19:53 |
| th1a | Also, I don't know how we ended up with years in the sidebar here instead of tertiary navigation. | 19:53 |
| aelkner | one thing at a time, please | 19:54 |
| aelkner | so the messages view will not filter, showing both types of messages but not discriminating them | 19:54 |
| aelkner | add message will work as it does now, not setting the flag | 19:54 |
| th1a | y | 19:55 |
| aelkner | the change of status section and add views go away | 19:55 |
| aelkner | ok, now the tertiary | 19:55 |
| aelkner | good idea, now that we have done these for a while, it seems more obvious now | 19:55 |
| th1a | We don't have to do that this afternoon. | 19:56 |
| aelkner | well, i can do that one later, but would you want the year in the title? | 19:58 |
| th1a | Sure. | 19:58 |
| aelkner | Interventions for $YEAR for the subtitle? | 19:58 |
| th1a | Sure. | 19:58 |
| aelkner | ok | 19:59 |
| th1a | Why don't you get started on this stuff. | 19:59 |
| replaceafill | aelkner can you confirm a ValueError with the Summary worksheet? | 20:00 |
| aelkner | link, steps to reproduce? | 20:01 |
| replaceafill | create a summary worksheet in a section that doesnt have one and visit the summary worksheet | 20:01 |
| replaceafill | th1a http://69.164.203.135:6660/schoolyears/2008-2009/2008-fall/sections/science_a_2008-fall_teacher001_003/activities/Worksheet/gradebook | 20:03 |
| aelkner | yes, i get the same error | 20:03 |
| replaceafill | th1a teacher001:teacher011 | 20:03 |
| replaceafill | oops | 20:03 |
| replaceafill | same pwd | 20:03 |
| replaceafill | just a quick question (sorry again) | 20:03 |
| replaceafill | is it ok if i make Activity/Points conditional | 20:03 |
| replaceafill | as aelkner suggested (i think) | 20:03 |
| replaceafill | when there are no activities | 20:03 |
| th1a | I have no opinion. | 20:05 |
| aelkner | yeah, i thought it looked confusing | 20:05 |
| aelkner | not important though | 20:05 |
| replaceafill | yvl could you please check http://bazaar.launchpad.net/~schooltool-owners/schooltool/flourish/revision/3086 specifically src/schooltool/skin/flourish/page.py | 20:08 |
| replaceafill | yvl i want to know if the related_manager.viewlets check is safe enough | 20:08 |
| replaceafill | yvl thanks :) | 20:08 |
| aelkner | replaceafill, is yvl even still here? | 20:11 |
| replaceafill | logs aelkner... | 20:11 |
| aelkner | ? | 20:12 |
| aelkner | are you talking to the log knowing that yvl will scan it later? | 20:12 |
| replaceafill | correct | 20:12 |
| aelkner | ah | 20:13 |
| aelkner | th1a, is there any reason /manage should have title, 'Your School for $YEAR' | 20:14 |
| aelkner | it has the tabs like all the other views | 20:14 |
| aelkner | if /courses needs it, why not /manage | 20:14 |
| th1a | Well, you can't actually screw anything up directly from /manage | 20:16 |
| aelkner | but you may get confused about the stats when you changed the year then forgot | 20:17 |
| aelkner | like when you walk away from kb and return | 20:17 |
| th1a | Some of the things on manage aren't actually limited to the year, either. | 20:17 |
| th1a | People, for example. | 20:17 |
| th1a | And the sidebar. | 20:18 |
| aelkner | i thought you were going to move people to the sidebar? | 20:23 |
| th1a | No. | 20:23 |
| th1a | aelkner: So are you getting that batch of changes underway? | 20:27 |
| replaceafill | th1a do you mind if i make http://69.164.203.135:6660/demographics a wide container | 20:29 |
| replaceafill | that table is floating out of the normal content container | 20:29 |
| th1a | Please do. | 20:29 |
| aelkner | th1a, got rid of status message views: | 20:31 |
| aelkner | http://69.164.203.135:36660/schooltool.interventions/2008-2009/student119 | 20:31 |
| aelkner | as manager | 20:31 |
| th1a | I'm getting a login when I try to download a report sheet. | 20:32 |
| aelkner | link? | 20:32 |
| th1a | http://69.164.203.135:36660/schoolyears/2008-2009/2009-spring/sections/english_c_2009-spring_teacher000_004/activities/2008-2009_2009-spring-2/gradebook/student119/view.html | 20:34 |
| th1a | I'm just trying to click on the links in the view you sent me to. | 20:34 |
| th1a | Did you try them? | 20:34 |
| aelkner | if i sent you a link, it was from a browser that had the page rendered, so i don't know what you're asking | 20:35 |
| aelkner | could you send me a link that doesn't give the login box, then tell me what to click? | 20:36 |
| aelkner | oh, and what user needs to be logged in for the link you send me | 20:36 |
| th1a | aelkner: What happens when you click on one of the links on the URL that you sent me to start this thread? | 20:37 |
| th1a | http://69.164.203.135:36660/schooltool.interventions/2008-2009/student119 | 20:37 |
| aelkner | ah, the report sheets links | 20:38 |
| th1a | AH! | 20:39 |
| aelkner | i didn't get to those yet, they point to view.html of the student gradebook context | 20:41 |
| aelkner | i didn't do that one yet | 20:41 |
| th1a | Ah. | 20:41 |
| aelkner | there's a criss-cross that gets started as the features come on line | 20:41 |
| aelkner | we don't actually have any other view.html links in the gradebook itself i think | 20:42 |
| th1a | Well, we need to push to tie these things up. | 20:42 |
| aelkner | my current tasks are: | 20:43 |
| aelkner | 1) create flourish view.html view (thanks for catching that one) | 20:44 |
| aelkner | 2) create tertiary for student interventions, removing select box in side bar | 20:44 |
| aelkner | 3) fix inbox filter widget for shoe goals and/or messages | 20:45 |
| th1a | Don't do 2) yet. | 20:48 |
| th1a | Do 1 & 3, make the text changes in the student intervention view and ping me. | 20:48 |
| th1a | We'll finish this tonight if necessary. | 20:48 |
| th1a | I guess you already made the text changes. | 20:51 |
| th1a | I was blinded by the traceback. | 20:51 |
| aelkner | sorry, too late to stop 2) | 20:51 |
| th1a | np | 20:52 |
| aelkner | it's not far from being done | 20:52 |
| th1a | kk | 20:52 |
| aelkner | i'm choosing the low hanging until i get ready for doc soon | 20:52 |
| th1a | ok | 20:52 |
| replaceafill | th1a about to do this: "The Sections link in /manage should only appear if there are courses and terms defined in that year" | 21:00 |
| th1a | OK. | 21:00 |
| th1a | Then we should do some "What is This" | 21:00 |
| replaceafill | we could do that first if you want | 21:00 |
| th1a | No, go ahead. | 21:01 |
| replaceafill | ok | 21:02 |
| aelkner | th1a, http://69.164.203.135:36660/schooltool.interventions/2008-2009/student119 | 21:07 |
| th1a | Good. | 21:08 |
| aelkner | so 1) and 3) i'll do later | 21:08 |
| aelkner | gong to get ready for doc now | 21:08 |
| th1a | OK -- but we still need to do the text, so I need you to ping me later. | 21:08 |
| th1a | Just send and email. | 21:09 |
| aelkner | what's this text? | 21:09 |
| th1a | The What is This. | 21:09 |
| th1a | We need to put the finishing touches on. | 21:09 |
| aelkner | ok, i'll ping you when i return | 21:10 |
| th1a | OK. | 21:10 |
| * aelkner goes to get ready | 21:10 | |
| replaceafill | th1a done | 21:16 |
| th1a | ok replaceafill | 21:17 |
| replaceafill | let me put the old data.fs back | 21:17 |
| th1a | OK. | 21:17 |
| replaceafill | i changed to test the condition on sections | 21:17 |
| replaceafill | th1a ready | 21:19 |
| th1a | kk | 21:19 |
| th1a | I'm going to look at the gradebook for a sec. | 21:20 |
| replaceafill | it's the sample data btw | 21:20 |
| replaceafill | ok | 21:20 |
| th1a | External scores. | 21:21 |
| replaceafill | teacher001? | 21:21 |
| th1a | Sure. | 21:21 |
| th1a | Move the hint to the WiT? | 21:21 |
| th1a | The first hint. | 21:21 |
| th1a | Or maybe just the first sentence of it? | 21:22 |
| th1a | What do you think? | 21:22 |
| replaceafill | i'd say both | 21:22 |
| replaceafill | i remember we put something about the sys admin in on WiT already | 21:22 |
| th1a | Also it should say "external score sources should" | 21:22 |
| replaceafill | categories i think | 21:22 |
| replaceafill | weight categories | 21:23 |
| replaceafill | http://69.164.203.135:6660/schoolyears/2008-2009/2008-fall/sections/science_a_2008-fall_teacher001_003/activities/Worksheet/weights.html | 21:23 |
| th1a | OK. | 21:23 |
| replaceafill | ok, moving it... | 21:23 |
| th1a | Also, can we make External Scores the third tab? | 21:23 |
| replaceafill | ok | 21:24 |
| th1a | Moving the hint? | 21:28 |
| replaceafill | all of it | 21:29 |
| replaceafill | th1a could you please explain this one: | 21:30 |
| replaceafill | Also it should say "external score sources should" | 21:30 |
| th1a | Not "...external scores should..." | 21:31 |
| replaceafill | is that in the hint? | 21:31 |
| th1a | Yes. | 21:32 |
| replaceafill | hhmm this is the hint i see in trunk: | 21:32 |
| replaceafill | Use external scores to add data from sources outside the SchoolTool Gradebook. External scores must be configured by your system administrator | 21:32 |
| th1a | Oh "must" | 21:32 |
| th1a | Sorry. | 21:32 |
| replaceafill | :D | 21:32 |
| replaceafill | ok | 21:32 |
| th1a | I should have copy/pasted. | 21:33 |
| replaceafill | th1a http://69.164.203.135:6660/schoolyears/2008-2009/2008-fall/sections/science_a_2008-fall_teacher001_003/activities/Worksheet/addLinkedActivity.html | 21:35 |
| replaceafill | two paragraphs for the WiT? | 21:35 |
| th1a | YEah. | 21:36 |
| replaceafill | done | 21:36 |
| th1a | Now I'm getting an ValueError for Linked Column. | 21:36 |
| th1a | Didn't anyone write tests for this shit? | 21:36 |
| th1a | :-D | 21:37 |
| replaceafill | it's seems like the same issue of the Summary worksheet | 21:37 |
| th1a | Can you fix it? | 21:38 |
| replaceafill | let me look | 21:38 |
| replaceafill | maybe aelkner should take a look at it :( | 21:41 |
| th1a | OK, moving on... | 21:42 |
| replaceafill | ok | 21:42 |
| th1a | http://69.164.203.135:6660/gradebook/templates | 21:44 |
| th1a | OK, that's fine. | 21:44 |
| th1a | Can you make the add report sheet form go straight to the edit view for the sheet? | 21:45 |
| replaceafill | kk | 21:46 |
| th1a | Eehh... | 21:49 |
| replaceafill | done | 21:49 |
| th1a | OK move add report activity to the sidebar. | 21:49 |
| replaceafill | i fixed the buttons classes | 21:49 |
| th1a | And add a Done. | 21:49 |
| replaceafill | adding the sidebar... | 21:55 |
| replaceafill | great, the add report activity form is broken by the change to the categories... | 22:01 |
| replaceafill | fixing... | 22:01 |
| replaceafill | th1a done | 22:02 |
| replaceafill | http://69.164.203.135:6660/schooltool.gradebook/templates/Bla | 22:02 |
| th1a | We should come up with some kind of system for automatically testing for regressions. | 22:03 |
| th1a | In theory I'd prefer not to have the add on the side, but it is too cluttered. | 22:04 |
| th1a | Does "Report Activities" need to be that big? | 22:04 |
| replaceafill | well, we've been using h2s for table, title, table | 22:04 |
| replaceafill | want to see an h3? | 22:05 |
| replaceafill | refresh | 22:05 |
| replaceafill | in my opinion h3 is too small there | 22:05 |
| th1a | If that's consistent then h2 is fine. | 22:05 |
| replaceafill | ok, reverting | 22:05 |
| th1a | OK, now, what is this damn thing. | 22:06 |
| th1a | Let's add a paragraph to the RST index. | 22:07 |
| th1a | Or... | 22:07 |
| replaceafill | RST or RSTs? | 22:07 |
| replaceafill | ;) | 22:07 |
| th1a | Change the first paragraph to say: | 22:07 |
| th1a | Report sheets use the SchoolTool gradebook to collect data and commentary from teachers about students in all classes. | 22:09 |
| th1a | How's that? | 22:09 |
| replaceafill | classes vs sections! | 22:10 |
| replaceafill | refresh | 22:11 |
| th1a | Sections! | 22:11 |
| th1a | Sorry. | 22:11 |
| th1a | This is why I can't just write all this stuff down and send it to you. | 22:11 |
| replaceafill | :D | 22:12 |
| replaceafill | refresh | 22:12 |
| th1a | I think that's ok. | 22:12 |
| replaceafill | i dont get the second paragraph | 22:12 |
| replaceafill | well, i do | 22:12 |
| replaceafill | i thought both paragraphs were talking about templates | 22:13 |
| replaceafill | never mind | 22:13 |
| th1a | Probably we need to repeat those in the report sheet indexes too. | 22:13 |
| th1a | Let's add a hint for the Title. | 22:14 |
| replaceafill | refresh, i was changing the buttons | 22:15 |
| th1a | Identifies the report sheet in teacher gradebooks. | 22:15 |
| th1a | for hint ^ | 22:15 |
| replaceafill | ok | 22:15 |
| replaceafill | http://69.164.203.135:6660/schooltool.gradebook/templates/Bla | 22:21 |
| th1a | Period. | 22:22 |
| th1a | OK, edit view WiT now... | 22:22 |
| th1a | A report sheet is a special kind of gradebook worksheet that can be deployed to all teacher gradebooks in the school. | 22:24 |
| th1a | I'm kind of slowly explaining this as the user progresses. | 22:24 |
| th1a | The whole thing will be explained in the book too. | 22:24 |
| th1a | The first sentence above is the first paragraph of the WiT, btw. | 22:24 |
| replaceafill | edit view is the index view of the sheet, right? | 22:25 |
| th1a | Each deployed worksheet will contain the activities defined in a template. | 22:25 |
| th1a | Yes... it says edit. | 22:25 |
| th1a | There isn't really a separate view. | 22:25 |
| th1a | view view. | 22:25 |
| replaceafill | kk | 22:26 |
| th1a | That's the second paragraph ^ | 22:26 |
| th1a | Let me see those two... | 22:27 |
| replaceafill | refresh | 22:29 |
| th1a | make the second sentence say "report activities" | 22:30 |
| replaceafill | done | 22:30 |
| replaceafill | "defined in its template. "? | 22:30 |
| th1a | I feel like stopping before I say "To add a report activity, click "Add Report Activity." | 22:30 |
| replaceafill | :D | 22:31 |
| th1a | replaceafill, yes, it's template. | 22:31 |
| th1a | or | 22:31 |
| th1a | no ' | 22:31 |
| th1a | lol | 22:31 |
| replaceafill | :D | 22:31 |
| th1a | Me no se habla espanol. | 22:31 |
| replaceafill | :)) | 22:31 |
| replaceafill | done | 22:31 |
| th1a | I think that's ok. | 22:32 |
| th1a | Do you? | 22:32 |
| replaceafill | yes, i understand it | 22:32 |
| th1a | Assuming it isn't the only documentation. | 22:32 |
| th1a | OK. | 22:32 |
| th1a | http://69.164.203.135:6660/report_sheets | 22:33 |
| th1a | Let's start with: | 22:34 |
| th1a | Report sheets use the SchoolTool gradebook to collect data and commentary from teachers about students in all sections. | 22:34 |
| th1a | The first step in the process is to create a new report sheet template. | 22:34 |
| replaceafill | same as the templates, right | 22:34 |
| th1a | ANd make "create a new report sheet template" a link to /gradebook/templates | 22:34 |
| th1a | Actually, we should link both ways. | 22:35 |
| replaceafill | http://69.164.203.135:6660/report_sheets | 22:38 |
| replaceafill | ANd make "create a new report sheet template" a link to /gradebook/templates? | 22:38 |
| th1a | Yes? | 22:38 |
| th1a | Can we do that. | 22:39 |
| replaceafill | you mean add the link right? | 22:39 |
| th1a | Yes. | 22:39 |
| replaceafill | i was looking for it :P | 22:39 |
| th1a | It will be a little messy translation wise, I presume. | 22:39 |
| replaceafill | definitely | 22:41 |
| th1a | Just have to work around the html, correct? | 22:41 |
| replaceafill | the string becomes: The first step in the process is to ${link} | 22:42 |
| th1a | Ooh. | 22:42 |
| th1a | Well, the link is just a separate translation string then? | 22:42 |
| replaceafill | yes | 22:43 |
| replaceafill | that's easier to translate i guess | 22:43 |
| th1a | OK, that's not too bad. | 22:43 |
| replaceafill | http://69.164.203.135:6660/report_sheets | 22:44 |
| replaceafill | we should add a "The second step in the process..." paragraph to /gradebook/templates :) | 22:45 |
| replaceafill | pointing to deploy | 22:45 |
| replaceafill | like you said | 22:45 |
| th1a | Yes. | 22:45 |
| th1a | After you create a template, the next step is to deploy it to teacher gradebooks. | 22:46 |
| th1a | OH... REPORT CARD LAYOUT. | 22:46 |
| th1a | Shit, that's what we don't have yet. | 22:46 |
| replaceafill | ah! | 22:46 |
| replaceafill | "The third step in the process.." :D | 22:46 |
| th1a | INdeed. | 22:46 |
| th1a | OK anyhow, add that sentence above, with a link. | 22:47 |
| replaceafill | ok | 22:48 |
| th1a | hm? | 22:49 |
| th1a | http://69.164.203.135:6660/gradebook/templates | 22:49 |
| th1a | ? | 22:49 |
| replaceafill | im changing the template | 22:50 |
| replaceafill | http://69.164.203.135:6660/gradebook/templates | 22:51 |
| replaceafill | th1a done | 22:56 |
| th1a | Once you have a template, use this form to add the report sheet to every section that meets in the specified term or terms. | 22:58 |
| replaceafill | that would be the third paragraph for http://69.164.203.135:6660/report_sheets, right? | 22:59 |
| th1a | Data collected in the report sheets can be added to the standard report card or custom reports. | 22:59 |
| th1a | Yes. | 22:59 |
| th1a | There is the 4th. | 22:59 |
| th1a | With a column this narrow you don't really need multi-sentence paragraphs. | 22:59 |
| th1a | Data collected through report sheets can be included in the standard report card or custom reports. | 23:00 |
| th1a | better wording? | 23:00 |
| replaceafill | http://69.164.203.135:6660/report_sheets | 23:01 |
| replaceafill | it's starting to grow! | 23:01 |
| th1a | Once you have a template, use this form to add the report sheet to THE GRADEBOOK OF every section that meets in the specified term or terms. | 23:03 |
| th1a | Don't actually use caps. | 23:03 |
| th1a | I think that's sufficient. | 23:03 |
| replaceafill | done | 23:04 |
| replaceafill | i also changed the markup of the form a little | 23:04 |
| replaceafill | to give it more space | 23:04 |
| replaceafill | i mean, the fieldset | 23:04 |
| th1a | OK. | 23:05 |
| th1a | Now... | 23:05 |
| th1a | Where's our old skin instance? | 23:05 |
| replaceafill | http://69.164.203.135:7777/calendar | 23:06 |
| th1a | Unfortunately, we've saved one of the gnarliest forms for last... | 23:06 |
| th1a | http://69.164.203.135:7777/schoolyears/2010/layout_report_card.html | 23:07 |
| replaceafill | yikes! | 23:07 |
| th1a | I might have to make aelkner do it. | 23:07 |
| replaceafill | +1 | 23:08 |
| replaceafill | :) | 23:08 |
| th1a | OK. | 23:08 |
| th1a | You're off the hook. | 23:08 |
| replaceafill | kk | 23:08 |
| replaceafill | lunch time ;) | 23:08 |
| replaceafill | 2 pm!!! | 23:08 |
| th1a | OK. | 23:09 |
| th1a | I'm off to cook. | 23:09 |
| * replaceafill goes to get lunch | 23:18 | |
Generated by irclog2html.py 2.15.1 by Marius Gedminas - find it at mg.pov.lt!
