=== Ursinha is now known as Ursinha-afk | ||
=== Ursinha-afk is now known as Ursinha | ||
danilos | gmb, hey, I've got one question: I am looking at the mockups from user testing, and I don't see the case where you *are* marked as the bug supervisor — should that be treated as if you have a structural subscription then? | 11:06 |
---|---|---|
bac | hi gmb | 11:24 |
bac | the fix you made worked great. | 11:24 |
gmb | bac: Cool, glad to hear it. | 11:25 |
bac | there is one oddity i haven't yet investigated. when the overlay is closed, the accordion and some checkboxes remain behind | 11:25 |
gmb | Hmm. | 11:25 |
gmb | That's a bit weird. I didn't see that one but my testing was probably a bit perfunctory. | 11:26 |
gmb | bac: I'm just going to grab a cup of tea, but I'll take another look in a few minutes. | 11:26 |
bac | yeah i only saw it yesterday but not sure if it had been there all along. happens in webkit and FF | 11:27 |
bac | ok thanks | 11:27 |
bac | it'll be a little bit before im really here | 11:27 |
danilos | gmb, do we already have some standard code that returns all the subscriptions for a (person, bug) pair? :) if not, do you know what code should I reuse to get at least some bits of it? | 11:56 |
gmb | danilos: I guess you're after structural subscriptions there, aren't you? | 11:57 |
danilos | gmb, nope, quite the contrary | 11:58 |
danilos | gmb, I need things like "direct subscription", "through duplicate", "you are bug supervisor", "through team membership",... | 11:58 |
gmb | danilos: Ah. Take a look at lp.bugs.interfaces.bug. There's a bunch of methods there that will be useful, but nothing that does everything you need all in one method. | 11:59 |
gmb | danilos: Thinks like isSubscribed(user), isSubscribedToDupes(user), getSubscribersForPerson(user).... etc. | 12:00 |
danilos | gmb, right, I just want to re-use as much as possible, thanks | 12:00 |
danilos | gmb, btw, if you've got a minute to glance over https://pastebin.canonical.com/44215/, I wonder if I am covering all the bases :) | 12:10 |
danilos | I thought of more supervisor actions ("leave the team" and "I don't want to be supervisor" if you are not product owner) | 12:15 |
* danilos -> food | 12:41 | |
gary_poster | hey benji, looks like Huw came through for the CSS. Any word on the triangle dealie? | 13:06 |
benji | nope, none yet; once I try out his CSS I'll reply with a "thanks that worked great... how about the other thing"-type message | 13:07 |
gary_poster | heh, ok benji, sounds good :-) | 13:08 |
danilos | benji, hi, are you working on implementing the structural subscriptions part of mock-ups on https://dev.launchpad.net/LEP/BetterBugSubscriptionsAndNotifications/Testing/EditingRound2? | 13:25 |
gmb | bac: I haven't been able to reproduce the problem you mentioned. Can you send me a screenshot | 13:25 |
gmb | ? | 13:25 |
bac | gmb, sure | 13:26 |
gmb | danilos: I think you've covered everything with that list. | 13:27 |
benji | danilos: I'm doing the list of user/team subscriptions for a bug (but I don't think it's mocked up on the slides you reference, it's at https://dev.launchpad.net/LEP/BetterBugSubscriptionsAndNotifications/Testing/SubsListRound1/Slide1 instead) | 13:27 |
danilos | gmb, cool, thanks | 13:28 |
danilos | benji, right, thanks | 13:28 |
bac | gmb: good : http://people.canonical.com/~bac/overlay-opened.png | 13:29 |
bac | gmb: bad http://people.canonical.com/~bac/overlay-closed-messy.png | 13:29 |
gmb | OIC. Hang on... | 13:29 |
bac | to go from good to bad i hit the 'cancel' button | 13:29 |
gary_poster | oopsie | 13:30 |
gary_poster | mumble nowish, bac benji danilos gmb | 13:30 |
danilos | https://dev.launchpad.net/LEP/BetterBugSubscriptionsAndNotifications/DirectSubscriptionsOnBug | 13:39 |
gmb | bac: So, http://pastebin.ubuntu.com/574977/ sort of solves the problem... I think it has to do with the way we deal with expanding nodes, but I'm not sure *why* | 14:10 |
gmb | (That's not a solution, really, but it does make the controls vanish) | 14:10 |
bac | interesting. and it'd need to be done for submit too, i guess | 14:10 |
gary_poster | The "kill them all" apprach to DOM manipulation :-) | 14:11 |
gmb | bac: Yeah (it'd be nice if FormOverlay had a 'hide' event that we could tie onto, but anyway...) | 14:12 |
gary_poster | well, perhaps the "SHUT UP ALREADY!" approach to DOM manipulation... | 14:12 |
danilos | gary_poster, gmb, anyone else who cares: I'll start using the IBug:+subscribe page for the work I am doing (description and direct subscription actions); if you feel I should leave that alone and provide a different page (eg. because we want to continue have +subscribe gracefully degrade), just let me know :) | 14:22 |
gary_poster | danilos, we have a page for this already that gmb started and that benji is working on... | 14:23 |
danilos | gary_poster, do we? | 14:23 |
gary_poster | yeah | 14:23 |
gary_poster | it has nothing on it ATM :-P | 14:23 |
gmb | danilos: NOOOOOooooo. That page is used to render the overlay for direct subscriptions. Use Bug:+subscriptions | 14:23 |
gmb | (Which is what gary_poster is talking about) | 14:23 |
gary_poster | :-) | 14:23 |
danilos | gmb, gary_poster: but I can't find it :) | 14:23 |
danilos | gmb, gary_poster: ok, that's the kind of info I was looking for | 14:24 |
gary_poster | cool | 14:24 |
danilos | gary_poster, that actually confuses me a bit; benji told me he's working on https://dev.launchpad.net/LEP/BetterBugSubscriptionsAndNotifications/Testing/SubsListRound1/Slide1, so just to clarify, https://dev.launchpad.net/LEP/BetterBugSubscriptionsAndNotifications/Testing/EditingRound2 is part of that as well? | 14:25 |
* gary_poster looks | 14:25 | |
gary_poster | benji, you are working on the bottom part of https://dev.launchpad.net/LEP/BetterBugSubscriptionsAndNotifications/Testing/EditingRound2/Slide2 I thought? | 14:26 |
gary_poster | ("Other subscriptions that may send you email about this bug") | 14:26 |
benji | danilos: I don't see anything on those slides that (as far as I'm aware) I'm working on (i.e., I'm not doing the "If you don't want to receive..." part) | 14:26 |
gary_poster | yeah you are :-) | 14:27 |
gary_poster | the one you are looking at is a part of the full one; you are only implementing the non-direct subscriptions, though | 14:28 |
gary_poster | which is pretty much the same | 14:28 |
gary_poster | across the two mockups | 14:28 |
gary_poster | danilos is working on the explanation and direct subscription part | 14:28 |
gary_poster | does that ring a bell, benji, or at least are you satisfied that this jibes with what you were already doing? | 14:29 |
benji | gary_poster: exactly, I'm doing the non-direct bits | 14:30 |
gary_poster | Great. Danilos, you cool too? | 14:30 |
danilos | gary_poster, yep :) | 14:30 |
gary_poster | yay :-) | 14:30 |
gary_poster | gmb, halloo. call now, or ping me when you are ready? | 14:30 |
danilos | thanks all | 14:30 |
gmb | Oh, bother, forgot about that. | 14:31 |
* gmb gets his mumble on. | 14:31 | |
gary_poster | :-) | 14:31 |
=== Ursinha is now known as Ursinha-lunch | ||
gmb | bac: D'you have a second to mumble? | 14:47 |
bac | sure | 14:47 |
benji | gary_poster: an RM pre-nag about QAing Bug 164196 | 14:53 |
_mup_ | Bug #164196: Quickly-undone actions shouldn't send mail notifications <email> <lp-bugs> <qa-needstesting> <story-better-bug-notification> <Launchpad itself:Fix Committed by yellow> < https://launchpad.net/bugs/164196 > | 14:53 |
gary_poster | thanks benji. Yeah, QA is blocked on the bug I'm working on :-/ | 14:54 |
gary_poster | (was blocked on staging being down for about a week brfore that) | 14:54 |
gmb | bac: http://pastebin.ubuntu.com/575007/ | 14:57 |
gary_poster | hey danilos. ping when you are ready for call? | 15:30 |
gary_poster | https://pastebin.canonical.com/44238/ | 15:44 |
danilos | gary_poster, https://pastebin.canonical.com/44239/ with syntax highlighting :) | 15:44 |
gmb | gary_poster: I've just submitted AWS expenses; rather higher than I'd hoped as there was an EC2 instance that got stuck. | 17:14 |
gary_poster | gmb :-/ ack, approving. | 17:14 |
gmb | gary_poster: Thanks. | 17:15 |
gary_poster | approved. np | 17:16 |
gmb | Ta | 17:16 |
gmb | bac: I've added a rudimentary test suite (i.e. all the setup, doesn't do anything helpful yet) to the ~yellow branch. I'll check back later, so if you've any questions, ping me. | 17:51 |
* gmb -> dinner | 17:51 | |
bac | ok | 17:52 |
bac | gmb: another annoyance -- if you simply click away from the overlay somewhere else on the page, the overlay disappears, leaving the cruft in place. is it supposed to make the overlay disappear? seems wrong. | 18:50 |
gmb | bac: Yes, it's supposed to hide the overlay. I wonder if the overlay does fire a 'hide' event that we can tie our cleanup to. It would seem odd not to. | 18:51 |
gary_poster | bac, I still haven't had lunch and am feeling a bit lightheaded. I should get some food. I might be 10 min late for our call. Is that alright? | 19:20 |
bac | gary_poster: that's fine. would you like to just say 3:00? | 19:21 |
gary_poster | sure, thank you bac | 19:21 |
bac | great. i'm going to take a walk now, then. | 19:21 |
gary_poster | cool | 19:21 |
bac | see you in a bit | 19:21 |
gary_poster | bye | 19:21 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!