=== Ursinha is now known as Ursinha-afk
gmbbac: Do you know where the "Subscribe to bug mail" link that appears on a Product's +index is defined in devel?13:24
bacgmb: yes, let me look13:26
bacgmb: are you talking about the one in bugs/browser/structrualsubscription.py?13:27
gmbbac: I don't know. Am I? I ask because in the accordionoverlay branch we have a subscribe_to_bug_mail link defined for the Product context menu, but we don't have that in devel and yet the link still appears (albeit with the name "subscribe" rather than "subscribe_to_bug_mail") and I'm rather confused because there seem to be several "subscribe" links defined around the place.13:28
gmbbac: Ah.13:29
gmbBut you're right.13:29
bacgmb: the one you are referring to (i think) is the *old* one that points to the +subscribe page13:29
gmbbac: Why shouldn't the new one point there too?13:29
gmbOur JS will override that behaviour13:29
bacas i update IStructuralSubscriptionTarget pages I am getting rid of the old one13:29
gmbAh, ok.13:30
bacgmb: my understanding was we were not going to support non-JS for this effort13:30
gmbbac: Ah, my understanding was that it wasn't our first concern... I'm not sure that it actually matters though.13:30
gary_posterbenji bac gmb (danilos you are not here right?) mumble/kanban asap.  sorry about that13:30
gmbbac: Thanks for the pointer anyway. That makes life easier.13:31
bacgmb: would you know why distributionsourcepackages, which are IStructuralSubscriptionTargets, don't currently have a 'subscribe' link?14:04
gmbbac: Er. No. That seems odd to me. Deryck might know.14:05
baci could ask tom on fb i guess.  :)14:06
gmbbac: lp:~gmb/launchpad/feature-flag-overlay-bug-73954914:11
gmbDiff here: http://pastebin.ubuntu.com/583807/14:12
bacgmb: at line 29 of your diff you define the 'subscribe' link.  is that necessary or will it come from the one defined in the structural subscription mixin?14:14
gmbbac: Oh. Dur. That would work too (and better). But that menu doesn't use the Mixin ATM.14:15
bacyeah, that would be too easy14:16
gmbbac: No reason it couldn't, though. Let me give it a shot...14:16
bacgary_poster: we have a problem with the overlay styling with narrow browsers (not so narrow, though) and long descriptions:  http://people.canonical.com/~bac/overlay-overrun.png14:24
gary_posterbac, yeah, LP production has similar issues with existing functionality :-/14:24
gary_posterI suspect there is some more attractive CSS to be had14:25
gary_posterSuggestion/thought: could you keep a "Huw punchlist" doc somewhere?  Maybe on the kanban backlog, or the wiki14:25
gary_posterI think this is something he could help with14:26
gary_posterI vaguely recall seeing relatively attractive CSS solutions for this sort of thing in the past14:26
benjifor the time being we could use a constant title (e.g., "Add a mail subscription") and either leave off the target name or put the target name in the smaller, free running text of the form.14:27
benjire. CSS: there is an ellipsis mechanism, but it's not the most standard thing in the world; I do seem to recall that FF does it well, so that may be all we need14:28
gary_posterI'd prefer to stick to the design for now and see what Huw comes up with.  solutions like that one tend to stick, and I'd honestly prefer for it not to unless necessary.14:28
gary_posterellipsis is an option14:28
gary_posterbut breaking with a border underneath the whole thing might also be an option14:29
gary_posterand at least look better than what we have14:29
gmbbac: Relying on the mixin seems to work fine. I've pushed that change up. Do you want me to merge it into ~yellow too?14:30
bacgmb: sure14:30
gmbbac: Merged and pushed.14:32
gmbI'm heading out to run some errands; I'll be back in an hour or thereabouts.14:32
bacgmb: thanks.  i'll merge it as soon as possible14:32
bacgary_poster: i created 'design tweaks' in the backlog14:42
gary_postergreat bac, thanks14:42
bacthe details has a list of items.  it can be decomposed later14:42
gary_postersounds good14:42
bacwas there something else that needs to be added to that list?14:42
* gary_poster looks14:43
gary_posteryes, I can think of one category off-hand.  I'll add what I think of.14:43
=== Ursinha-afk is now known as Ursinha-lunch
gary_posterbac, this is currently part of the server branch that I am about to submit for review.  Is it problematic for merging into devel, or is it alright?16:36
gary_posterbenji, maybe you could review this MP draft?  You are the other person with significant work in the server branch.  http://pastebin.ubuntu.com/583864/plain/16:45
benjigary_poster: sure16:45
=== Ursinha-lunch is now known as Ursinha
gary_posteroh meh.16:51
gary_poster"Add a function to efficiently get a user's structural subscriptions for a bug" should be deleted16:51
gary_posterand I need to describe "Add a function to efficiently get a user's structural subscriptions for a bug target"16:51
gary_posterI'm going to go lunch; hopefully I will feel more energized afterwards16:52
benjigary_poster: I think we can kill expose_user_subscription_status_to_js/userHasBugSubscriptions I can't find any place that we actually use it.  Of course, we need to make sure no one is adding such a place at the moment.16:55
benjire. the "person_is_team_admin should maybe be preceded with an underline" question: I don't think so; since we're strict about importing only things listed in __all__, there's no need to do the semi-private leading underscore thing.16:57
benjithat's the end of my thoughts on the MP16:59
* benji lunches as well.16:59
bacgary_poster: the paste you have above should not be included17:28
bacgary_poster: it needs to be feature flagged17:28
bachi benji, question about feature flags18:06
bacis there an easy way to enable them for lp.dev interactive testing?18:06
benjiyou should be able to edit them, let's see what is the url...18:07
bacoh, yeah, right18:07
bacbenji: so i tried adding:18:13
bacbut it does not appear in the list at https://launchpad.dev/+feature-info18:14
bacand appears inactive18:14
benji+feature-info only lists documented features; just adding a rule won't make an entry in +feature-info18:17
benjihowever, *referencing* a rule that isn't documented will make it show up in the Undocumented flags area of +feature-info18:17
bacbenji: i see that now.18:19
benjiif you haven't seen it, the wiki doc for feature flags is good: https://dev.launchpad.net/FeatureFlags18:20
bacbenji: the problem was i used "default$" as stated in the scope section.  not sure where that errant $ came from18:22
benjibac: I bet you copy/pasted it from the -info page, where the scopes are given as regular expressions18:23
bacbenji: yes, but is it a regex?  if so, wouldn't 'default$' work?18:23
benjinope, the scope is a string matched by a regex, the -info page shows the regex the strings match (which in "default"'s case is just a constant so it's wierd)18:25
bacbenji: ok, another dumbish question18:53
bacthe FF flag page says for "booleans" an empty string indicates false18:53
bac(makes me think "boolean" isn't the proper description)18:54
bacentering the following into the rule editor:18:54
bacmalone.advanced-structural-subscriptions.enableddefault10 ''18:54
benjiwell, it's really up to the calling code to decide what an empty string is; the FF code just returns strings18:55
bacgives the FF a value of u"''", which evalutes to True18:55
gmbbac: s/malone.//18:55
gmbUnless you've changed it since I merged my branc with ~yellow.18:55
bacgmb: i did.  i thought it followed the convention better18:55
bacbenji: so the question is, how do you enter a value in the rules editor to indicate false?18:56
benjiI don't remember adding that "Value domains" section, it looks more deceptive than helpful18:56
benjibac: if you're writing the code that's retrieving and interpreting the value then you get to decide...18:57
bacbenji: that is not consistent with the usage guide18:57
baci mean, you're right if you understand you're going to get a unicode value back and you must test it yourself18:58
gary_postertrying to get things done for the new baby == surprise 2 hour lunch.  oops.18:59
gary_posterthank you for the feedback bac and benji; I'll incorprate to code and MP18:59
benjigary_poster: babys are indeed full of suprises. I may soon be reminded of that myself, Katie is starting to want another.  I'm not quite on the same page yet though ;)19:01
gary_posterbenji, wow :-)19:01
benjibac: yeah, I really don't like the hints on the -info page that there is type conversion going on.  If I added that I hereby repent.19:02
bacbenji: i'm not blaming, i'm 1) trying to understand it, 2) trying to use it properly19:02
bacthe docstring for FF tell you to:19:02
bacif features.getFeatureFlag('soyuz.derived-series-ui.enabled'):19:02
bacbut that will never work if the feature is edited using the UI19:03
benjino worries, I have no emotional attachment to previous versions of myself19:03
benjibac: I /think/ it will work; if you leave it unset you get [some false value] or you can set it using a rule19:04
benjilet me look at the code a little19:04
bacbenji: i think you're right.  but once set it can never be unset, which makes testing a pain as it involves 'make schema' to reset it19:05
benjibac: I believe there is a testing context manager that lets you temporarily set flags19:06
gary_posterbac, I have an up-to-date accordionoverlay and I am visiting https://launchpad.dev/firefox and I don't see the add button UI anymore.  Do I need to have the feature flag set now?19:26
bacfor product, yes19:26
bacgary_poster: gmb did that page and i've now replicated it for others19:27
bacgary_poster: but my work hasn't been pushed19:27
bacgary_poster: if you want to see it you'll have to set a value for the feature flag19:27
gary_postercool bac, thanks.  I was jusst trying to make sure I hadn't broken anything and was disconcerted to not see it.  Right, so I go to https://launchpad.dev/+feature-info...19:27
bacmalone.advanced-structural-subscriptions.enableddefault   10 on19:28
bacgary_poster: add ^^19:28
gary_posterbac, got it.  thank you.  who do I need to be logged in as?  Mark?19:28
bacyeah, you need to be some admin-type, i'm sure19:28
bacmark will work19:29
gary_postermm, foo.bar worked too.  cool I guess19:29
gary_posterbac, I now have "malone.advanced-structural-subscriptions.enableddefault10on" but I don't see it working on https://launchpad.dev/firefox19:31
bacgary_poster: sorry, strip 'malone.' from the front.  i changed the name from what gmb did originally19:32
bacthe instructions for flags indicated an app area prefix was needed.  if you don't think so let me know and i'll remove it19:33
gary_posterbac, what you did sounds fine19:33
gary_posterand it is working now thank you19:34
gary_posterUnleash THE KRAKEN!20:01
gary_posterbenji, if you run out of cards, as seems all too possible, please take a look at the 739141 defect card in Quick Jobs20:04
benjigary_poster: k.  I hope to be done with this little card (description whitespace) in a few minutes; I'll look at 739141 next20:05
benjigary_poster: are we ready for more merges into ~yellow?20:05
gary_posteryes, benji, no problem.  If they are server side and should be reviewed as such maybe let me know, but we're good AFAIK20:06
benjinope, all JS20:06
gary_posterbac, I'm looking at the kanban board and wondering what you will need to have a good client-side branch for review tomorrow.20:59
gary_posterMy take is that you will need the card you are working on, and the card Danilo is working on in Feature Work 2: Coding.  The card to hook up the edit page is a nice-to-have (blocked in feature 1 tasks).  The card Danilo has in fw2: tasks is even less necessary.20:59
gary_posterTherefore, I'm thinking that we are in a pretty good place for getting the branch ready for review tomorrow as we had hoped.  Do you agree?21:00
bacgary_poster: i would but i'm having difficulties with my branch21:00
bacthe tests with feature flags are not working as i expected21:00
gary_posterbac, I see. Anything I or anyone else can do to help?21:00
bacgary_poster: i could talk it over with you if you like.  perhaps you might see something i'm doing silly21:01
gary_posterbac, I'm happy to if you think it would help.  Now or tomorrow morning?  I need to leave to take care of kids in 15, fwiw21:01
bacgary_poster: now is fine...let's make it quick21:02
gary_posterwith feature_flags():21:06

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