gary_posterYou are right, that is from mochikit.  The merge we want is on the Y object.  You can see it in the console with YUI().merge({foo:4, bar:5}, {bar:7,bing:8})00:46
gary_posternight for real :-)00:46
=== _mup__ is now known as _mup_
gary_posterbenji, hi.  "save" nominally works, now even more than it did last night.  Specifically, the save button is hooked up, it saves to the server, and changes to the name and filter are reflected dynamically after a save.12:31
gary_posterAs Danilo mentioned, there are a number of other related issues, and I'm wondering how many of them I should claim as part of this particular effort/"branch".  Here's a list off the top of my head.12:31
gary_poster* state is not loaded properly, especially for the radio button/checkbox divided thing (that is, if your notification level is "everything but comments")12:32
gary_poster* patching fails if you try to use the accordion widget (form_data.importances is undefined)12:33
gary_poster* the panel is not reset entirely across usages12:34
gary_poster(no details available in my head for that one, just a vague memory of encountering the problem)12:34
gary_posterI think there are some other ones, but that's what comes to mind now12:35
gary_posterdo you have a preference as to whether I land as is, and then try to fix some of those; or you tackle some of those (maybe they are in progress already?); or I try to fix them before I land?12:36
gary_posterIf you are not around to reply, my default will be to commit as is, merge with ~yellow's branch and push, and then try to fix some of those other issues.12:36
* gary_poster proceeds to do that.12:37
benjigary_poster: sorry, I got distracted waiting for the list, pre-coffee attention span is lacking12:39
gary_posternp :-)12:39
benji+1 on committing what you have (especially with TODOs, I plan on doing a TODO/XXX purge before the end)12:40
gary_posterpushed to ~yellow12:43
gary_posterbenji, I'd like to start keeping the punch list on the kanban board, please.  I have marked the bits that are specific to "unsubscribe in anger" as low.  I've added the two concrete bugs I mentioned above, as well as "Add JS tests".  I think you are working on that one?  If so, and you have no other objections, I'll move it to "coding" for you, and I'll move "load state properly and fully into overlay" into "coding13:03
gary_posterIf you could then add your other punchlist items to Feature Work 1 tasks I would appreciate it.13:03
benjigary_poster: correct and +1 on "Add JS tests"; I'll add my other items as cards.  Do you want the non-required ones low and the required ones regular?13:04
gary_posterbac, benji, gmb, flacoste made a suggestion on how we might run our morning meeting a bit differently.  In sum, I'll be reviewing the kanban board and asking questions of you guys and asking for details.  Hopefully that will be faster, it will leverage the kanban board better, and it will maybe even help us with the collaboration thing (we'll see; brief details on call)13:05
gmbCool, works for me.13:05
gary_postercool thanks13:05
bacgood experiment13:05
gary_posterbenji, yes, that would be great, thanks13:06
benjigary_poster: which part of the backlog should the punchilst items go in?  I've forgotten which of the subdivides was for what.13:13
gary_posterbenji, feature list 1: tasks13:17
benjioh, ok13:17
bacgary_poster: do we have a means of testing IE?  i can set up a windows VM here if needed.13:18
gary_posterbac, the direction from Francis is to simply remove the "no IE" clauses from our JS13:19
gary_postertesting is extra niceness13:19
gary_posterhis argument is that Firefox is the only browser that is truly officially supported13:20
gary_posterBecause that's where the Windmill tests live13:20
gary_posterWe fix webkit because it is in use13:20
gary_posterbut we don't test any of these exhaustively13:20
gary_postereven FF is "most recent version on our test box"13:21
gary_posterIE should be given a chance to try and work13:21
gary_posterespecially when the functionality has no alternative13:21
gary_posterAlso, the "no IE" was added at a time that YUI 3 had a very poor IE story13:21
gary_posterNow AIUI it does not support IE 613:21
gary_posterbut later versions of IE are supported reasonably well13:22
gary_posterand IE 8 is the primary IE by far on LP13:22
gary_posterSo anyway, please feel free to argue, but I was alright with it myself13:22
benjigary_poster: tasks added, I think there's one or two that I added that you've already done so a glance over them would be appreciated.13:22
gary_posterack, looking13:23
benjiI think we need a small but visible warning to people using unsupported browsers.  I can only imagine the feeling of shoddy workmanship a corporate type visiting with IE would get.13:24
gary_posterI can see your position13:25
benji[in general, not just our latest work ;)]13:25
benjiwe did something like that on a previous project and it helped a great deal to both manage expectations and push people toward using supported browsers13:28
gary_posterunderstood. :-) I'd be happy if you were to advocate that in some appropriate forum.  The only argument against it that I can think of off hand is that, depending on how you define "supported" the list could be *really* small.13:29
gary_posterFF works fine for me on Chrome and Safari in addition to FF, but our support of Chrome/Safari is essentially catch-as-catch-can.  I dunno anyone who uses older versions of FF on the team.13:29
gary_posterSaying "we'll treat problems in browser X as high priority" might sound nice, but as we know from the size of the open bug collection in our bugtracker, that could be argued to be just hot air.13:29
gary_posterbac benji gmb, mumble/kanban in 1-ish13:29
gary_posterthat was not an example of "faster" but hopefully it will become so13:45
gary_posterok, bac, so, you want me to just verify that the functionality works; or look at the code...?  Give me an idea of what you are looking for, please13:46
gary_posterstandard review?13:46
bacgary_poster: just look the presentation, really13:46
gary_posterok cool, trying13:46
bacgary_poster: i just put up two <a> links.  nothing fancy13:47
bacsanity check, really13:47
gary_posterbac, I would say that it is functional and reasonable.  In a perfect world, I'd have some CSS tweaks to change several things about the pane: "Select all" and "Select none" would be smaller...and if LP used underlines to show links like the rest of the world then I'd advocate that too, but nevermind.13:51
gary_posterI'd also put in some padding to the whole pane, as well as some horizontal padding for each of the checkbox items.13:52
gary_posterSo in context, then, my take is "good, and let's maybe send the whole thing to Huw for some polish soon, or spend some time on that ouselves"13:52
gary_posterGood enough, or do you want to drillinto any of that?13:52
benjigary_poster: I have a ZF meeting on my calendar for today, any idea if that's right?13:56
gary_posterbenji, might have been cancelled, because the board was all elected?13:56
gary_posterI mean, nominations equaled spots available13:56
gary_posterso voting was not necessary13:57
benjicould be; I don't even remember getting the vote email, I wonder if they're still using my zope.com address13:57
bacgary_poster: thanks for looking at it.  i agree with your comments but think it may be best to send the whole thing off to huw for tweaking.14:01
gary_postercool, bac14:01
* gmb -> out for a run14:39
benjigary_poster: ZF meeting /is/ on (the time zone change made me be off by an hour); #zope-foundation15:06
gary_posterbenji, ah, k15:06
benjigary_poster: are there any other ZF members at Canonical that we should remind?15:07
gary_posteris sidnei?15:07
bacjml is doing a TDD presentation today at noon:  https://wiki.canonical.com/KnowledgeSharing/TDD15:12
gary_posterbac, you were asking for confirmation of whether the thunderdome was going to be in Ireland.  I asked Francis.  Still no confirmation, but he said it was a reasonable time to check back in with Marianna about it.17:21
=== Ursinha is now known as Ursinha-lunch
gary_posterbenji, ~yellow now has proper rendering of data (loading of state) in edit forms.  It may have fixed "patching fails if you try to use the accordion widget" on the way; not sure.18:51
gary_posterI'm now going to a card I added, "Normalize "any status" and "any importance" so that if all checkboxes are checked, or none, they are the same in summary display and checkbox redraw."18:51
gary_posterNotice I added another card too: "Make help links work (tags, subscription name)"18:51
gary_posteror, bac, is there something I can do to help you?  "Make sure that clicking add subscription after an add or cancel displays a clean form " would suit me fine.18:53
bacgary_poster: that is a good one18:53
gary_posterok, doing that18:53
bacgary_poster: i talked to curtis and getting our link in that portlet is best done by creating a hidden link.  that's going to require some surgery on the Link rendering18:54
gary_posterbac, brain surgery, or ear piercing?18:55
bacgary_poster: i hope it isn't too bad.  will need to add a new attribute to ILink and teach the renderer to include display:none or something similar18:55
gary_posterok, yeah, doesn't sound awful18:56
bacshould be straightforwardish but all of the abstraction around menuing makes my head hurt18:56
gary_posteryeah, I hear you. :-/18:56
bacgary_poster: so i plan to make a separately landable branch to do that and then wire it into our branch18:56
gary_postersounds perfect18:56
gary_posterbac, if you open an overlay with "Subscribe to bug mail," make some changes, click cancel or click away to make the overlay vanish, and then click on "Subscribe to bug mail" again, what do you think should happen? 1) your changes are remembered or 2) you get a fresh start?  I have an opinion, but I'd like to hear yours and compare notes.19:17
gary_posterI'm going to make a decision, but I'll keep quiet on what it is until I hear from you...or until I have to land. :-)19:21
bachey gary_poster19:46
gary_posterhey bac19:46
gary_posterI hope we agree ;-)19:46
bacsorry, my house is all tore up and a guy is here installing new floors.  sometimes it is distracting.19:46
gary_posterlol, np19:46
bactwo rooms worth of stuff have been crammed into my office.  :(19:47
gary_posterhey, same here!19:47
bacgary_poster: i guess a cancel should wipe out your work and you get a fresh start19:47
gary_posterstuff that was in the baby-room-when-we-don't-have-a-baby was dumped out here19:47
bacsame for reopening after a successful submit19:48
gary_posterbac, cool, agree, that's what I did.  Almost done.19:48
bacis click-away a different scenario?19:48
baci guess not19:48
gary_posterIt could be--I see why you ask, I think--but making it one might be annoying19:49
baci can just see the 'i did lots of stuff and then it ate my work' bug19:49
bacprobably from wgrant. :)19:49
gary_posterI still think the desired behavior in that case is ambigious at most19:49
bacwe're having an NC launchpad dinner tomorrow19:49
gary_posterand so implementing the easier one makes sense19:50
gary_posterNC launchpad, huh?19:50
bacwell, sackett wanted to try lantern19:50
gary_posterIf that's an invitation, I could see if Karyn is up for it and we could get a babysitter19:51
bacoh, you're more than welcome if you can possibly come19:51
baci assumed you were not doing a lot of social stuff these days19:51
gary_posterno, not so much, but K has wanted to go to Lantern19:51
bacoh.  well find out and i'll up the reservation.  it is currently at 6:1519:52
bacand i promise they will not have green beer19:52
gary_postercool.  I should know in half an hour or less; I suspect the answer is probably no, but thank you for mentioning it.  We'd definitely like to get together more often, though things will probably be more challenging, not less, for at least a few months after the delivery.19:53
gary_posterbac, enjoy.  We won't be able to do it this time.  K says maybe we can have you guys over sometime soon.20:01
bacthat sounds great20:02
=== Ursinha-lunch is now known as Ursinha
benjifrom the looks of it, it's not safe to run the tests on the ~yellow branch; right?20:22
gary_posterI'm probably guilty20:24
gary_posterI have no idea how to run the tests20:24
gary_posterwhich is probably bad, what with me changing things and stuff20:24
gary_posterhow do I run the tests, benji?  I'll see if I can fix in a sec20:24
benjigary_poster: I was just running bin/test -c lp.bugs.browser and got a wall of errors20:25
gary_posterah, those tests20:25
gary_posteryeah, I got scared of those earlier.20:25
benji(the JS tests are run like so xvfb-run ./bin/test --layer=RegistryWindmillLayer -cvvt test_yuitests)20:25
benjiwhile I have your attention, did you write person_is_team_admin?20:26
gary_posteryeah, I've almost exclusively been touching the JS lately20:26
gary_posterbenji, yes, and it is fixed in my branch.  sorry about that.20:26
gary_posterI can commit in just a sec20:26
benjiheh, ok; sounds good20:27
gary_posterbenji, pushed, along with fixes for a another related item or two you might have encountered.20:34
benjicool, pulling now20:35
gary_poster(the deletion of filters within subscriptions had been a bit hosed)20:35
gary_posterI'll see if I can see what lp.bugs.browser does...20:35
gary_posterbenji: AttributeError: 'BugSubscriptionListView' object has no attribute 'structural_subscriptions' and AttributeError: 'TargetSubscriptionView' object has no attribute 'structural_subscriptions' are trivial.  I had set up an attribute for you to use when building your views, and you ignored it. :-P  I ripped it off locally, and forgot to remove the test.20:41
gary_posterThat said, I've seen "UnknownEntryAdapter: No IEntry adapter found for Interface (web service version: devel).  Encountered as a result of the entry interface None, field ''." when I run LP.  That one is mildly scary to me and has been around for a little while.  I'll look into it.20:41
benjiyeah, the IEntry one worried me a bit too20:42
gary_posterbenji, looking at the diff with devel, this seems like the most likely culprit.20:57
gary_poster@export_factory_operation(Interface, []) # Really IBugSubscriptionFilter20:57
gary_posterThat's on def addBugSubscriptionFilter(subscriber, subscribed_by):20:57
gary_posterI assume there's supposed to be a fix for a circular import.  I'm not entirely sure how to do that, but I think I know where to look.  If you already have an idea of what to do, though, I'm all ears.20:58
gary_posteryeah, I think I see it...21:03
gary_posteryeah, benji, that was it, and I have a fix.  I'll push it and a merge with devel in just a few, hopefully.21:10
gary_posterbenji, pushed.  bin/test -c lp.bugs.browser passes and devel is merged21:21
gary_posteroff to dinner21:21
benjigary_poster: cool; that was less painful than I expected21:22
=== Ursinha is now known as Ursinha-afk

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!