*** khildin has quit IRC | 01:06 | |
*** khildin has joined #schooltool | 12:12 | |
*** yvl has joined #schooltool | 12:58 | |
*** veloutin has joined #schooltool | 15:54 | |
*** replaceafill has joined #schooltool | 16:21 | |
*** th1a_ is now known as th1a | 16:31 | |
th1a | hi replaceafill, yvl. | 16:31 |
---|---|---|
yvl | good morning | 16:31 |
th1a | Is menesis on vacation now? | 16:31 |
replaceafill | good morning/afternoon | 16:32 |
*** veloutin has left #schooltool | 16:33 | |
yvl | I think so | 16:33 |
th1a | OK. He said he was doing the release and going on vacation, and I saw the release, so... | 16:34 |
th1a | replaceafill: How are you doing? | 16:35 |
replaceafill | good, recovered now | 16:35 |
replaceafill | lost my voice for 2 days | 16:35 |
replaceafill | i had a serious throat infection | 16:36 |
replaceafill | th1a, did you check the url i sent you last week | 16:37 |
th1a | Ah. | 16:37 |
th1a | Yes. | 16:37 |
th1a | Looks very good. | 16:38 |
replaceafill | ah cool | 16:38 |
th1a | Two things: | 16:38 |
replaceafill | yes? | 16:38 |
th1a | Ideally, if you just wanted to punch in a value an move on, there would be some way to do it. | 16:39 |
replaceafill | you mean, without displaying the combobox? | 16:40 |
th1a | I was thinking more without it capturing the return. | 16:40 |
th1a | Just when it came up. | 16:40 |
th1a | Or alternately, if it made a best guess about selecting what should be active. | 16:41 |
th1a | So if I enter "a" the menu comes up but "A" is active so if I hit return, I go on. | 16:41 |
replaceafill | i think there's an option to select a value automatically | 16:43 |
replaceafill | i mean, in the combobox | 16:43 |
th1a | That might work. | 16:44 |
replaceafill | i just didn't use it, because the way A+, A, and A- are sorted | 16:44 |
replaceafill | didn't feel "natural" | 16:44 |
th1a | Basically, if there is a perfect or very close match, you just want to keep going. | 16:44 |
replaceafill | it would if the order was "A, A+, A-..." and so on | 16:44 |
replaceafill | but currently the order is not alphabetical | 16:45 |
replaceafill | well, i'll activate the option and show you, and will remove the return capture | 16:46 |
th1a | Can you sort? | 16:46 |
replaceafill | sure | 16:46 |
replaceafill | it's just using the scoresystem order right now | 16:46 |
replaceafill | but this cries for being sorted alphabetical imho | 16:46 |
th1a | Well... that's the right way to do it. | 16:46 |
th1a | No... | 16:47 |
th1a | I don't think so. | 16:47 |
replaceafill | ok | 16:47 |
replaceafill | what's the second thing? :) | 16:47 |
th1a | The round corners. | 16:47 |
replaceafill | ah | 16:48 |
replaceafill | the combobox style is default jquery-ui | 16:48 |
replaceafill | including the corners | 16:48 |
th1a | Sure. | 16:48 |
replaceafill | the options have black borders too | 16:49 |
replaceafill | (i didn't like that) | 16:49 |
th1a | Yeah. I assume you can override that. | 16:49 |
replaceafill | yes | 16:49 |
replaceafill | i will | 16:49 |
th1a | OK. "Flourish it." (TM) | 16:50 |
replaceafill | :D | 16:50 |
th1a | When your UI is stinky, spray some Flourish (TM) on it!" | 16:50 |
replaceafill | th1a, this is the part of the year we talk about cando xls imports... | 16:51 |
replaceafill | dwelsh is back in action! | 16:51 |
th1a | Yes, I was trying to remember what the custom import issue was? | 16:52 |
replaceafill | and he's asking if we're going to support his format for custom lists | 16:52 |
replaceafill | ah ok | 16:52 |
replaceafill | he has this 2011 format for importing a single course (skills) | 16:52 |
replaceafill | that's what cando 2011 used | 16:53 |
replaceafill | for him it's natural for the user to write *that* format | 16:53 |
replaceafill | not the one we currently use | 16:53 |
replaceafill | last year i helped him importing every custom list | 16:53 |
replaceafill | i wrote a script for it (similar to the one i wrote for jelkner) | 16:54 |
replaceafill | i haven't talked to him yet, but i think he either wants my support again doing that or he want us to include that format in cando | 16:55 |
th1a | Is it more human readable, you mean? | 16:55 |
replaceafill | i'm not sure | 16:55 |
replaceafill | well, yes | 16:55 |
th1a | I guess it is up to you. | 16:56 |
th1a | If it is just dwelsh, maybe just running the script is easiest. | 16:56 |
replaceafill | he asked me to meet today, so i'll ask | 16:57 |
replaceafill | that's it from me | 16:57 |
th1a | OK. Back to 100% health? | 16:58 |
replaceafill | yes | 16:58 |
replaceafill | and i don't have classes today and tomorrow | 16:58 |
replaceafill | so i'll be here | 16:58 |
yvl | good to hear that you're ok replaceafill :) | 16:58 |
replaceafill | thanks yvl! :) | 16:58 |
th1a | Great to have you back replaceafill. | 16:58 |
th1a | yvl? | 16:59 |
yvl | new relationship implementation is dropping relationships sometimes | 16:59 |
yvl | as in - all relationships :) | 17:00 |
th1a | Ach. | 17:00 |
yvl | looks like persistence problem, but those are sometimes tricky to track down | 17:00 |
yvl | luckily there are not many places to look at | 17:01 |
yvl | other than that... | 17:01 |
yvl | I almost almost have it presentable :) | 17:01 |
yvl | javascript dialog is not finished / broken a bit | 17:02 |
th1a | OK. So... focus on the bug. We'll probably have to punt this to the next release. | 17:02 |
yvl | ok | 17:02 |
th1a | Unless you get the bug fixed by Wed, I'd say. | 17:02 |
th1a | That would be the blocker, ofc. | 17:03 |
yvl | yes | 17:03 |
th1a | All right, so dig into that. | 17:04 |
yvl | sure | 17:04 |
th1a | Thanks yvl. | 17:05 |
th1a | I am flying to Scotland in one week. | 17:05 |
th1a | So the distraction level around here is going to be extremely high. | 17:05 |
th1a | I -- and the family, for a year -- makes the distraction level VERY high. | 17:05 |
th1a | But I'll be around, so don't hesitate to ask a question. | 17:06 |
replaceafill | i'll try not to set up meetings with you and dwelsh :D | 17:06 |
th1a | Yeah. That would be good. | 17:06 |
th1a | We'll probably meet Mon/Thurs next week. | 17:07 |
th1a | I'll have to figure out what time the meetings are! | 17:07 |
replaceafill | th1a, do you plan to change the meeting time when you're in your new house? | 17:07 |
th1a | I don't think so. | 17:07 |
replaceafill | ah ok | 17:07 |
th1a | I don't think there is a reason to. | 17:07 |
yvl | 2:30 PM I guess :) | 17:08 |
th1a | That might have to change when school starts. | 17:09 |
th1a | OK. Thanks guys! | 17:11 |
th1a | See you Wednesday. | 17:11 |
* th1a drops the bag of gravel. | 17:11 | |
replaceafill | thanks guys | 17:11 |
replaceafill | good to see you again :) | 17:11 |
yvl | thanks guys | 17:12 |
yvl | see you soon! | 17:12 |
replaceafill | th1a, sorry for not sending the invoice last night (i dropped pretty early), i'll send it in a moment | 17:12 |
th1a | ok | 17:13 |
*** pgulley has joined #schooltool | 17:22 | |
pgulley | replaceafill! | 17:22 |
pgulley | I just read over the meeting from yesterday | 17:23 |
replaceafill | hello pgulley | 17:23 |
replaceafill | ah good | 17:23 |
pgulley | sorry I couldn't be there. | 17:23 |
replaceafill | we need to talk UI reorganization i guess | 17:23 |
pgulley | Yes- and import/export | 17:23 |
replaceafill | did you see my email? | 17:24 |
replaceafill | comments on your branch | 17:24 |
pgulley | yes, but jelkner seemed to have some differing thoughts with regards to how it should work | 17:25 |
replaceafill | right | 17:25 |
pgulley | Should it then just be by-teacher, instead of system-wide? | 17:26 |
pgulley | it doesn't quite address the original issue, but it's what he wants. | 17:26 |
replaceafill | i'm not sure | 17:29 |
replaceafill | i guess you need to talk to him about it | 17:30 |
replaceafill | the sharing/reusing part is still confusing to me | 17:30 |
pgulley | yeah, same | 17:30 |
pgulley | okay, I'll sort that out with him | 17:31 |
pgulley | but, in the mean time- UI | 17:31 |
replaceafill | regarding UI reorganization, i think you should start by filtering of quizzes | 17:31 |
replaceafill | students should be able to filter quizzes by course | 17:33 |
pgulley | okay | 17:34 |
replaceafill | also i'd include terms somehow | 17:34 |
pgulley | would that be something browser-side? | 17:34 |
replaceafill | not really | 17:35 |
replaceafill | we have this filter functionality for tables in place already | 17:35 |
replaceafill | ajaxy | 17:35 |
pgulley | ahh | 17:36 |
pgulley | alright | 17:36 |
replaceafill | you see, all those tables in the quizzes page | 17:36 |
replaceafill | are different "pieces" | 17:36 |
replaceafill | viewlets | 17:36 |
pgulley | right | 17:37 |
replaceafill | look at browser/deployedquiz.zcml | 17:38 |
replaceafill | name="takeable_quizzes_table" | 17:38 |
replaceafill | right below that | 17:38 |
replaceafill | there's another viewlet | 17:38 |
replaceafill | named "filter" | 17:39 |
replaceafill | which currently is set to donotfilter | 17:39 |
replaceafill | so, nothing is displayed/done | 17:39 |
replaceafill | but you can insert your own html and functionality | 17:39 |
replaceafill | and name it "filter" | 17:39 |
replaceafill | and it'll be displayed above the table | 17:39 |
replaceafill | there are plenty of examples | 17:39 |
replaceafill | in your schooltool branch, look for: | 17:40 |
replaceafill | course.browser.section.SectionsTableFilter | 17:41 |
replaceafill | and visit the /sections page in your instance | 17:41 |
pgulley | ahhh | 17:43 |
pgulley | okay, cool | 17:43 |
replaceafill | i was also thinking that maybe the quizzes to take and the quizzes taken tables should be in different pages | 17:43 |
replaceafill | but you can start with the filter | 17:44 |
replaceafill | see the Terms part of the /sections filter | 17:44 |
replaceafill | i'd use something like that (checkboxes) for the quizzes tables too | 17:45 |
replaceafill | enabling only the "current" term | 17:45 |
pgulley | alright | 17:45 |
replaceafill | so students see quizzes only for that term by default | 17:45 |
replaceafill | if they need other terms, they mark the other checkboxes and so on | 17:45 |
replaceafill | makes sense? | 17:45 |
pgulley | yes, all clear | 17:46 |
replaceafill | good :) | 17:46 |
pgulley | awesome! | 17:47 |
pgulley | This makes sense, shouldn't be too much. | 17:47 |
pgulley | I'll fix all of the things you pointed out with the import/export | 17:47 |
pgulley | talk to jeff, | 17:47 |
pgulley | then jump into this UI work | 17:47 |
replaceafill | cool | 17:47 |
replaceafill | use a separate branch for the ui work | 17:47 |
pgulley | okay | 17:47 |
replaceafill | starting from trunk ofc | 17:47 |
pgulley | ofc | 17:48 |
replaceafill | i'll be around, let me know if you have questions | 17:48 |
pgulley | excellent! thanks a bunch. | 17:49 |
pgulley | Glad you're feeling better | 17:49 |
replaceafill | np | 17:49 |
replaceafill | thanks! :) | 17:49 |
replaceafill | th1a, mind if i double check the permissions using an instance? | 18:16 |
replaceafill | (not just the code) | 18:16 |
th1a | That's fine. | 18:16 |
th1a | I just figured they are all .edit or .view | 18:16 |
replaceafill | i mean, i can read the changes in the remap, but i want to see the cando side of it | 18:16 |
replaceafill | yes | 18:16 |
th1a | Also, that was my excuse for making you do it while I pack. | 18:17 |
replaceafill | :)) | 18:17 |
replaceafill | hhmm it seems like we have some navigational problems | 18:45 |
replaceafill | clerks have permission to add courses | 18:45 |
replaceafill | but how do they get to the /courses view | 18:45 |
replaceafill | or how do they see the "Add -> Course" link | 18:45 |
th1a | hi replaceafill. | 19:08 |
replaceafill | hey th1a | 19:09 |
replaceafill | updating the sheet | 19:09 |
th1a | What can't they see? | 19:09 |
replaceafill | the links | 19:09 |
th1a | Why? | 19:09 |
replaceafill | the links context i think | 19:09 |
th1a | You just need to be clear on if you're looking at a bug or "problem." | 19:09 |
replaceafill | it's a data model problem! (joke) | 19:10 |
replaceafill | these views | 19:10 |
replaceafill | oops | 19:10 |
replaceafill | freenode... /courses, /sections, /terms | 19:10 |
replaceafill | anything starting with / is a command... | 19:10 |
replaceafill | are registered for the application | 19:10 |
replaceafill | i remember yvl complained about it | 19:10 |
replaceafill | and i think this is why | 19:11 |
replaceafill | the security declaration lands explicitly on the containers | 19:11 |
replaceafill | coursecontainer, sectioncontainer | 19:11 |
replaceafill | allowing the clerk to modify those | 19:11 |
replaceafill | but the views are registered on a different context, including the "add new..." links | 19:11 |
replaceafill | imho the fix is to remove the /courses, /sections links and move the views to use traversal properly | 19:13 |
replaceafill | not sure how much work it is though | 19:14 |
th1a | I don't understand what has changed. | 19:14 |
th1a | Is there some permission the Clerks lack now? | 19:14 |
replaceafill | giving the clerks "edit" on the app would solve it i think | 19:15 |
replaceafill | but it seems like a workaround to me | 19:15 |
replaceafill | just my 0.02 | 19:15 |
th1a | OK, it is all lumped together on "edit" on the app, which would, I'm guessing, let them do things like add and delete years. | 19:16 |
replaceafill | correct | 19:17 |
th1a | OK. Well, the "right" fix is probably necessary then. | 19:17 |
replaceafill | +1 | 19:18 |
th1a | Can you file a bug for that? | 19:19 |
replaceafill | but even with the right fix, we still have some navigational issues | 19:19 |
th1a | And note it in the spreadsheet as a bug. | 19:19 |
replaceafill | the courses view is supposed to be accesed from /manage,, right? | 19:19 |
replaceafill | should clerks have access to that? | 19:19 |
replaceafill | ah ok | 19:20 |
th1a | Clerks definitely have to view /manage. | 19:20 |
replaceafill | but not all of it, correct? | 19:20 |
th1a | That was always the premise. | 19:20 |
replaceafill | no school year add for example, right? | 19:21 |
th1a | Yeah, but it would just be a couple things. | 19:21 |
replaceafill | in the meantime i'm filling the spreadsheet ignoring this | 19:22 |
th1a | Well... mark the places where there are bugs. | 19:23 |
th1a | So they know. | 19:23 |
replaceafill | kk | 19:23 |
replaceafill | as usual... i wonder if i'm missing something... :) | 19:24 |
th1a | Don't start fixing them though (unless there are a few trivial ones). | 19:24 |
replaceafill | right | 19:24 |
replaceafill | no, i'll definitely run this by yvl before doing anything | 19:24 |
th1a | yvl may be doing it later this week, depending on how the relationship bugfix goes. | 19:24 |
replaceafill | ah ok | 19:24 |
th1a | The clerk needs to be able to do all the routine data entry and run all reports. | 19:25 |
th1a | The clerk is the main day to day user of the application. | 19:25 |
replaceafill | hey! that's what i told dwelsh, almost same words ;) | 19:25 |
th1a | I told him too. ;-) | 19:25 |
th1a | One issue is that neither the clerk nor the site manager technically should have rights to do all the imports. | 19:28 |
replaceafill | only the va import works off the schoolyear | 19:30 |
replaceafill | the global one and skills work off the app | 19:30 |
th1a | Well, you understand my point though? | 19:31 |
replaceafill | yes | 19:31 |
th1a | I mean, it seems like there should already be a check at the object level that is throwing a warning? | 19:32 |
th1a | Or does the site manager actually ave rights for all the objects in the spreadsheet. | 19:32 |
replaceafill | glenda's spreadsheet? | 19:33 |
replaceafill | not, not on all of them | 19:33 |
th1a | I mean, the standard import spreadsheet. | 19:34 |
th1a | Does the site manager have permission to create all those objects? | 19:34 |
replaceafill | not really | 19:35 |
replaceafill | again, course container for example | 19:35 |
replaceafill | edit just for clerks | 19:35 |
replaceafill | but these permissions (in security declarations) are for views | 19:36 |
replaceafill | i mean, practically | 19:36 |
th1a | OK. | 19:37 |
th1a | So in theory we could add permissions at the object level but that would be a lot of work just to tidy up this issue. | 19:38 |
th1a | In practice, we just need to decide who's going to be allowed to run the imports. | 19:38 |
replaceafill | right | 19:38 |
th1a | Since you can't delete data in the imports, realistically it could just be both site managers and clerks. | 19:39 |
replaceafill | also, i'd remove the super-user remark in the security page | 19:41 |
replaceafill | to me, the default user is not that super anymore | 19:41 |
th1a | What has changed? | 19:42 |
replaceafill | i mean, you start with a fresh database, right? | 19:43 |
replaceafill | he can add the school year | 19:43 |
replaceafill | but as soon as you try to add a course | 19:43 |
replaceafill | you get the forbidden page | 19:43 |
replaceafill | that's a clerk's job | 19:43 |
replaceafill | what's changed is that some tasks are only for the clerks now, when in the past the administration crowd contained the 3 | 19:44 |
th1a | Yes, but by default you're going to be both. | 19:44 |
replaceafill | he's not | 19:45 |
th1a | I think I sent an email about that last week. | 19:45 |
th1a | But no, it hasn't happened yet. | 19:45 |
th1a | You should have a couple emails about that. | 19:45 |
replaceafill | but your mail was about evolution, right? | 19:45 |
th1a | I think I also said that the manager should start in clerks and site managers. | 19:46 |
replaceafill | ah | 19:46 |
replaceafill | didn't get that part :) | 19:46 |
th1a | Because you're right, otherwise people would lose their minds as soon as they try to add a course. | 19:46 |
replaceafill | ah, found that email | 19:48 |
replaceafill | th1a, i'm done with glenda's spreadsheet | 20:16 |
th1a | Do you want to just throw it on Google? | 20:16 |
replaceafill | ah sure | 20:16 |
replaceafill | shared | 20:19 |
replaceafill | cando 2013 groups... :S | 20:19 |
replaceafill | i added in blue one permission they didn't consider, terms | 20:19 |
replaceafill | and with yellow, the parts with navigational problems | 20:20 |
replaceafill | th1a, bug filed: https://bugs.launchpad.net/schooltool/+bug/1208549 | 20:45 |
* replaceafill goes to run some errands, bb in ~1h | 20:53 | |
*** khildin has quit IRC | 20:55 | |
th1a | replaceafill? | 21:40 |
*** pgulley has quit IRC | 23:27 |
Generated by irclog2html.py 2.15.1 by Marius Gedminas - find it at mg.pov.lt!