[00:10] Can anyone tell me what channel should nexus 4 user should be using? I'm currently on utopic but I haven't had an update since October and I keep seeing images from the rtm channel. [00:38] Is it possible to have a normal desktop on Ubuntu Touch? Instead of the homescreen, like Ubuntu Desktop for Nexus 7 that was dropped. [01:12] why is ubuntu touch armel? [01:13] the boot image is armhf [01:13] but the preinstalled system says armel [01:20] I can't flash the boot image because it's too large [01:20] correct size is 8192 KB [01:21] but it's 8368 KB [02:01] I just installed Ubuntu Touch using ubuntu-device-flash, but now fastboot isn't working and the device won't boot to Ubuntu, only recovery works === duflu_ is now known as duflu === vrruiz_ is now known as rvr_ === rvr_ is now known as rvr [03:02] scratch that [03:02] but it still fails to boot === IdleOne is now known as PopsNClicks === PopsNClicks is now known as IdleOne [09:34] Good morning all; happy World Architecture Day! :-D [09:46] rpadovani, hey, I’ve just reviewed https://code.launchpad.net/~rpadovani/webbrowser-app/addressBarFullWidth/+merge/239039 , the change looks good but it breaks a number of autopilot tests (see my comment in the MR) [09:47] kenvandine, jgdxx, Laney, could one of you review/approve https://code.launchpad.net/~seb128/ubuntu-system-settings/build-translations-template/+merge/240696 ? [09:48] daker, hey, I don’t know if you ever saw my comments on https://code.launchpad.net/~daker/webbrowser-app/fix.airbnb-smartbanner/+merge/230855 ? [09:49] oSoMoN, ok, thanks, I'll try to take a look and fix it [09:50] rpadovani, cheers! don’t hesitate to ping me for help, fixing autopilot tests can be arduous sometimes :/ [09:51] seb128: I didn't follow the reasoning, what good does that do? [09:51] oSoMoN, thanks! [09:51] Laney, it means the template is updated on package uploads and that we don't need to play catchup with manual updates commits in trunk [09:52] so we use ubuntu translations? what happens to the one in trunk? [10:00] Laney, we stop using it [10:00] yes we use ubuntu translations [10:00] easier [10:00] dpm asked to keep the pot in trunk for a bit because it's used for stats [10:01] if stats are bit wrong it's not a big issue, better than having the template used for translations behind [10:01] does it still commit the updates to trunk? [10:01] Laney, yes [10:01] s/it/lp/ [10:01] okay! [10:03] Laney, regardless of where the translations are done (Ubuntu or upstream), they're committed to trunk. It's only that the Ubuntu template will be up-to-date with every upload, but the upstream one won't be up-to-date unless it's manually committed. This will make the stats to be a bit wrong, but until I can get the stats exported from Ubuntu, "a bit wrong" is still better than "no stats at all" :) [10:08] Laney, dpm, thanks [10:12] oSoMoN: I'll fix it [10:13] daker, thanks! [10:23] Laney, was bug #1384760 a whoopsie issue? did you say you were looking at it or talking to bdmurray about it? [10:23] bug 1384760 in ubuntu-system-settings (Ubuntu) "Reporting of crashes and errors randomly becomes enabled" [High,New] https://launchpad.net/bugs/1384760 [10:23] seb128: never reproduced it, can you? [10:25] Laney, no, but I didn't try hard [10:26] well, let's mark it as Incomplete until we get better steps [10:27] mardy: ping [10:27] marcustomlinson: pong [10:28] mardy: looking at your slides. The slide on scopes OA, there is a rough "a" before ".application and .service files" [10:28] mardy: also the example then is only of a service file. [10:29] mardy: then "unity-shell-api" should be "unity-scopes-api" [10:34] marcustomlinson: I did some changes, does it look better now? [10:39] mardy: so the bold text in the stuff you copied from our API doc is the stuff to be highlighted. [10:40] mardy: you've highlighted odd stuff :p [10:41] marcustomlinson: which slide? [10:42] mardy: slide 13. Example: only "accounts" should be highlighted [10:42] mardy: in the apparmor file [10:42] mardy: the manifest file also has some bold text [10:42] marcustomlinson: right :-) [10:43] mardy: hightlight the bold stuff [10:43] marcustomlinson: those are the OA-specific hooks [10:43] mardy: right, which is what you want to draw attention too [10:46] mardy: and just for some readability, on slide 14, could you insert a blank line before "// Check if our service is authenticated..." === alan_g is now known as alan_g|afk === alan_g|afk is now known as alan_g === cwayne-afk is now known as cwayne [12:43] seb128: hi there! how busy are you today? i've got a not-at-all-urgent review for system settings that needs doing :) === dandrader is now known as dandrader|afk [12:43] yes, a person [12:44] I just installed Ubuntu Touch on my grouper, following the instructions exactly, but when it's done, it drops to a boot screen in which 'start' does nothing. Selecting 'reboot to bootloader' and then 'start' puts 'Booting failed' in the top left corner. [12:45] To install I put it in fastboot and used 'ubuntu-device-flash --channel=ubuntu-touch/trusty --bootstrap' [12:45] Riku-san_, groupper has not been supported in about 1 year ... [12:47] Chipaca, hey, reasonably busy but I can probably find a slot for a review, is that the push helper logging one? [12:47] seb128: yep [12:47] Chipaca, on my list ;-) [12:47] seb128: excellent, thanks [12:47] yw! === MacSlow is now known as MacSlow|lunch [12:52] ogra_, there's an image for it of 14.04 [12:53] therefore it should be supported, no? [12:53] not really, no, that was just fo convenience [12:53] I've had enough 'not supported' crap for one day [12:54] just please help me fix my tablet [12:54] install android ... the ubuntu image you would install there would be massively outdated anyway [12:55] and most likelyonly work half way [12:55] grah [12:55] I'll just go back to the even less supported 13.04 desktop image [12:56] grouper support was dropped before trusty release, not sure the driver would even work or anything [12:56] seb128, can you add [1] to the list? :) [1] https://code.launchpad.net/~jonas-drange/ubuntu-system-settings/cellular-manual-carrier-persist-fix-1388044/+merge/241541 [12:56] jgdxx, hey, looking [12:57] oh of course, the tablet I just bought for $100 isn't supported [12:57] seb128, hi, thanks. [12:57] I'll go back to the desktop image where xorg breaks alot === jgdxx is now known as jgdx [12:57] jgdx, oh, that's not a small one :/ [12:57] not sure I'm going to have slots for that one [12:58] seb128, no, not really. But if you'd like, a quick install + testing of the new functionality would help! [12:58] k [12:58] I have only forbidden carriers, so it's pretty useless. [12:59] gtg === alan_g is now known as alan_g|lunch === dandrader|afk is now known as dandrader [13:15] Laney, hi, how do I land [1]? I assume it's not automatically landed by jenkins, or else it probably would have landed by now.. [1] https://code.launchpad.net/~jonas-drange/gsettings-ubuntu-touch-schemas/other-vibrations/+merge/239734 [13:15] ci train [13:15] I just commented on that too after I saw your mail. :-) [13:15] ah :) [13:16] jgdx, you ask the lander of your team :) [13:16] he/she will then push it onto the train [13:17] ogra_, got it === MacSlow is now known as MacSlow|lunch [13:21] Laney, pushed r51 which adds the comment. [13:21] ty [13:21] you can get it in the same silo as whatever uses it === dpm_ is now known as dpm === alan_g|lunch is now known as alan_g [13:59] the ubuntu online summit is starting; come join the foray http://summit.ubuntu.com/uos-1411/meeting/22327/intro-by-michael-hall-keynote-by-rick-spencer/ [14:01] mardy: ping [14:09] marcustomlinson: pong [14:09] mardy: hey man, sorry to bug you about this again. its about the soundcloud scope I'm working on. [14:10] mardy: so (on the desktop) when I add the account from settings, the account is successfully and I get this log: http://pastebin.ubuntu.com/8964802/ [14:10] mardy: but, when I do the log-in from a scope via OnlineAccountsClient::Setup, the account doesn't get added. I get this log: http://pastebin.ubuntu.com/8964817/ [14:13] marcustomlinson: I can see that the account plugin is deleting the identity, for some reason [14:13] mardy: yeah, it seem like everything goes alright, then it just gets deleted :/ [14:14] marcustomlinson: please set UOA_LOGGING_LEVEL=2 and UOA_DAEMON_TIMEOUT=9999 and restart online-accounts-service [14:14] mardy: k [14:19] mardy: ok, where do I get the online-accounts-service log? [14:19] marcustomlinson: on the terminal where you run the online-accounts-service command [14:19] mardy: ah [14:19] :p [14:24] mardy: http://pastebin.ubuntu.com/8965121/ [14:25] marcustomlinson: that "error: " line, is it from your Main.qml? [14:26] mardy: "qml: error: 401"? no [14:27] marcustomlinson: not the whole string, just "error:". Maybe when getting the username [14:28] mardy: well I can't tell you that its not coming from unity8, but not from me [14:29] marcustomlinson: can I see the account plugin source code? [14:30] mardy: oh! right. now I get what you're asking [14:30] marcustomlinson: :-)) [14:30] mardy: sorry, I inherited this project [14:31] mardy: http://pastebin.ubuntu.com/8965274/ [14:35] I've now updated all packages' bzr:s that I rebuilt for Qt 5.3.2, so your next landings should work as normal [14:36] btw Qt 5.3.2 is in the newest vivid-proposed image now, just published [14:38] mardy: I may need to set the user agent in the http header [14:39] marcustomlinson: no, I think that the access token should not be passed like that [14:40] marcustomlinson: unfortunately the SoundCloud API doc only shows how to use their client libraries, but it's likely that you need to write the token in the Authorization header [14:40] mardy: k, let me try something === oSoMoN is now known as oSoMoN|afk [14:46] marcustomlinson: instead of "?access_token=", try "?oauth_token=" [14:46] mardy: k [14:46] marcustomlinson: at least, this is what I can guess looking at https://github.com/soundcloud/soundcloud-python/blob/master/soundcloud/client.py#L115 [14:47] marcustomlinson: ah, you also need "client_id" [14:48] marcustomlinson: or maybe not; try without it, first [14:49] dednick, i just finished thoroughly testing my cherry-pick of the check.sync branch in rtm and found some things that failed which work in vivid [14:49] kenvandine: which things? [14:49] orientation-lock, wifi switch under battery, and data roaming under cellular [14:50] the orientation-lock and wifi under battery both fail to stay in sync with the indicators [14:50] mardy: right, it didn't error 401 this time :) [14:50] mardy: but... It looks like Main.qml is trying to parse xml into JSON :p [14:50] the data roaming under cellular isn't really changing the ofono setting, no indicator for that [14:51] dednick, so are you sure there aren't indicator/shell changes needed to keep those in sync with the indicator? [14:51] kenvandine: is the orientation lock gsetting changing, or is that broken? [14:51] let me confirm [14:51] mardy: but that was the right fix THANKS! [14:52] dednick, ok, that one is working it's the indicator is broken [14:52] i forgot that didn't work in rtm yet [14:52] kenvandine: ya. thought so [14:52] kenvandine: what's up with the wifi? [14:52] marcustomlinson: yw :-) [14:52] the wifi switch under battery doesn't stay in sync with the network indicator [14:53] dednick, the wifi switch under wifi does stay in sync [14:53] dednick, that's weird though, because it works in vivid [14:53] and that code merged cleanly [14:53] dednick, it's in rtm silo 3 [14:54] kenvandine: depends if the battery & wifi use same backend [14:54] kenvandine: i'm having trouble with my devices at the mo. they aren't co-operating :/ testing is a bit of a trial [14:54] dednick, yeah, but that plugin hasn't diverged in vivid [14:54] so odd... let me compare the code [14:56] kenvandine: right, the wifi uses the indicator, and the battery uses libnm. [14:56] mardy: ping, is it possible for a service to pull from 2 providers? [14:56] dednick, oh... that's annoying! [14:56] kenvandine: slightly! [14:56] dednick, but... why is that regressing? [14:56] Wellark, hi, where is bug 1374082 in the pipeline? :) [14:56] bug 1374082 in indicator-network (Ubuntu RTM) "no API to unlock a specific sim" [High,Triaged] https://launchpad.net/bugs/1374082 [14:56] kenvandine: no idea there [14:56] or... maybe it's already broken in rtm [14:57] jgdx, do you have an rtm device handy? [14:57] jgdx, can you test to see if the wifi switch under battery works? [14:57] kenvandine: i would think that the indicator uses libnm anyway [14:57] kenvandine, sorry no [14:58] dednick, the wifi switch there does stay in sync with the indicator on vivid [14:58] but not on rtm [14:58] pitti, renatu, so https://translations.launchpad.net/ubuntu-rtm/14.09/+source/mediaplayer-app/+pots/mediaplayer-app/es/+translate?batch=10&show=all&search=player shows no .desktop file string to translate [14:58] pmcgowan, can you test something on rtm quickly? [14:58] kenvandine: i'll try rtm one of mine. [14:58] dednick, thanks [14:58] - SettingsItemTitle { [14:58] + ListItem.Standard { [14:58] pitti, renatu,looks like for RTM the template was not updated on build, or that the package was not uploaded recently? [14:59] cwayne: I don't think so... what's your use case? [14:59] that's the diff between vivid and this branch for rtm [14:59] dpm: right, I just found that out with renatu; https://code.launchpad.net/~renatofilho/mediaplayer-app/fix-1382317/+merge/240875 needs to land to fix that [15:02] kenvandine, what do you need [15:02] cyphermox: please let me know if you need some logs or anything regarding this: https://bugs.launchpad.net/ubuntu/+source/bluez/+bug/1391912 [15:02] pmcgowan, can you see if the wifi switch in the battery panel works for you? [15:02] Ubuntu bug 1391912 in bluez (Ubuntu) "signal strength indication doesn't work" [Undecided,New] [15:02] oSoMoN|afk: i updated the bug, but yes, child processes do get stopped as well [15:02] pmcgowan, specifically does it seem to stay in sync with the network indicator? [15:03] kenvandine, seems to yes [15:03] wtf! [15:03] kenvandine, I have monday build [15:03] pmcgowan, i'm testing the backport of the fix for keeping the indicators in sync [15:03] kenvandine, turned off wifi in the panel, ok, then turned it on in the indicator, and ok [15:03] and that doesn't work on rtm [15:04] but does on vivid [15:04] kenvandine, want me to test on rtm157? [15:04] seb128, please do [15:04] the diff between vivid and rtm is one line... and shouldn't affect it [15:04] but it doesn't work for me with silo 3 [15:05] kenvandine, doesn't work on rtm 157 [15:05] pmcgowan, and why did bug 1336715 drop off the topblocker list? [15:05] bug 1336715 in ubuntu-system-settings (Ubuntu RTM) "[TOPBLOCKER] switch-items in indicators sometimes get out of sync with system-settings" [Critical,In progress] https://launchpad.net/bugs/1336715 [15:05] I toggled it off in battery's panel [15:05] seb128, interesting... ok [15:05] and the indicator didn't change [15:05] well the icon did [15:05] mardy: sorted out. Thanks again! [15:05] but not the item in the indicator menu [15:05] wifi panel still shows it on as well [15:06] seb128, so that isn't a regression caused by this branch then [15:06] still odd [15:06] but it doesn't list any aps [15:06] seb128, can you test silo 3? [15:06] kenvandine, yes, but after that UOS session [15:07] http://pastebin.ubuntu.com/8965818/ [15:07] seb128, that's the list of stuff to test [15:08] 2 failures for me, one expected [15:08] the rotation lock indicator fix hasn't landed yet [15:08] kenvandine, ok [15:08] oh, data roaming also failed for me [15:09] pmcgowan, i think that bug fell off the list because the unity8 task was marked fixed [15:09] but the settings fix didn't land in rtm yet [15:09] kenvandine, it should be showing up, let me check on t [15:10] seb128, kenvandine, so for a u-s-s fix that I expect to land on rtm, which branch should it be targeted for? [15:10] we land in trunk(vivid) and cherry-pick for rtm [15:11] to speed up you can also propose a branch for our rtm branch [15:11] kenvandine, Is the rtm branch getting seb128's header fix? I merged that already, but it doesn't seem to be in rtm. [15:11] nope [15:11] Uhg, okay. [15:11] sorry... i wish :) [15:12] convince pmcgowan it's important :) [15:12] * kenvandine passes the buck [15:12] pmcgowan, kenvandine says this is important ^ [15:12] haha [15:12] it's nice for sure.. [15:12] which fix? [15:12] the header/labels stuff [15:12] kenvandine, my mako is soon at latest rtm. Will test. [15:12] cosmetic [15:13] sounds like ota? got a bug # [15:14] i'll find it [15:14] Probably makes sense when we sync all of vivid. [15:16] tedg, do you have the bug number handy? [15:16] oSoMoN|afk: oh, but you should have like 3 seconds before life cycle kicks in, so you should have time to pull back the keepDisplayOn [15:17] seb128, was there a bug filed for that? [15:17] seb128, the section headers/titles branch [15:17] kenvandine,Doesn't seem like one was attached: https://code.launchpad.net/~seb128/ubuntu-system-settings/titles-uis-tweaks [15:17] yeah [15:18] kenvandine, tedg, you can use bug #1389767 [15:18] bug 1389767 in ubuntu-system-settings (Ubuntu RTM) "Differentiate titles from normal text" [Undecided,New] https://launchpad.net/bugs/1389767 [15:18] it was opened after the fix landed but that's good enough [15:18] pmcgowan, ^^ [15:21] kenvandine, I can propose it we'll see, as you said nice to have [15:21] yeah [15:22] don't forget the great sequel: ~jonas-drange/ubuntu-system-settings/titles-uis-tweaks-2 [15:23] jgdx, can you add that to the bug [15:23] pmcgowan, done [15:33] kenvandine: looks like the wifi setting in batter doesnt work in rtm either. [15:33] *battery [15:33] dednick, yeah seb128 saw that too [15:34] pmcgowan, what image version did you test that in ? [15:34] i'm trying to see what's changed [15:34] dednick, kenvandine: the battery toggle seems to work to me [15:34] the icon in the panel change to a not connected one [15:34] it's the indicator switch which is buggy it seems [15:35] oh, maybe i didn't look close enough :) [15:35] kenvandine, r7 keeps indicator and uss wifi toggle in sync [15:35] kenvandine, seb128: works indicator->battery wifi, but when changing battery indicator does not update [15:35] seb128, oh... you're right [15:35] seb128: in rtm? [15:35] dednick, rtm 157 [15:36] on krillin [15:36] same here [15:36] with silo 3 [15:36] it is working, just the switch in the indicator isn't changing [15:36] but it is disconnecting from wifi [15:36] and going to edge [15:36] kenvandine: ah. ok [15:36] whew [15:36] so an indicator bug :) [15:36] that works in vivid though [15:36] kenvandine: how do you tell that? just ifconfig? [15:37] dednick, the icon in the indicator [15:37] changes to wifi disconnected, then to edge [15:37] but the wifi switch in the network indicator stays checked [15:38] dednick, however in vivid the switch in the indicator toggles properly [15:39] kenvandine: hm. my icon isnt changing :/ [15:39] so looks like bug 1336715 is still valid for indicator-network in rtm [15:39] bug 1336715 in ubuntu-system-settings (Ubuntu RTM) "[TOPBLOCKER] switch-items in indicators sometimes get out of sync with system-settings" [Critical,In progress] https://launchpad.net/bugs/1336715 [15:39] kenvandine, https://launchpad.net/ubuntu/+source/indicator-network/0.5.1+14.10.20141007.2-0ubuntu1 ? [15:39] dednick, weird... mine is [15:39] is that in rtm? [15:39] 0.5.1+14.10.20141031.3~rtm-0ubuntu1 [15:39] that's the version i have === oSoMoN|afk is now known as oSoMoN [15:40] dunno then [15:40] check with Wellark [15:40] yeah, dunno [15:40] Wellark, ^^ === dandrader is now known as dandrader|lunch [15:40] hm. ok, it seems to be changing when i turn it off, but not back on again [15:41] or just sometimes [15:41] kenvandine, does your silo address this as well https://bugs.launchpad.net/ubuntu/+source/ubuntu-system-settings/+bug/1388804 [15:41] Ubuntu bug 1388804 in ubuntu-system-settings (Ubuntu) "Silent mode status and message out of sync" [Critical,In progress] [15:41] pmcgowan, i think so [15:42] seb128, according to the changes mail, that did land in rtm [15:42] kenvandine, k [15:42] pmcgowan, confirmed, that bug is fixed in silo 3 [15:43] kenvandine, seb128: ok, it's actually an indicator-network issue. the backend switch value isn't updating. it's not a UI sync issue at least. [15:43] dednick, yeah [15:43] kenvandine, great [15:43] tedg, you proposed a branch for that [15:44] kenvandine, titles fix is approved to land, seems everyone likes that [15:44] kenvandine, Yeah, updating it now. [15:44] tedg, thanks [15:44] kenvandine, (quite literally, it was pushing as I was typing) :-) [15:44] that should already be fixed in vivid and now in this branch for rtm [15:44] pmcgowan, woot [15:45] seb128, would you mind preparing a backport of that for rtm? [15:45] kenvandine, I can do [15:45] seb128, thanks! [15:45] yw! [15:45] maybe we can included jgdx's too [15:45] which i haven't looked at yet :) [15:46] it's fine *waves hand* [15:46] dednick, so now i'm down to figuring out why data roaming isn't working [15:46] dednick, can you file a bug about the indicator-network issue? [15:46] kenvandine, did that land in trunk yet? [15:46] seb128, did what? [15:47] kenvandine, maybe we can included jgdx's too [15:47] that hasn't [15:47] so we should do that first [15:47] i don't think it's been reviewed yet [15:47] before backporting [15:47] yeah [15:47] it has [15:47] it's approved [15:47] seb128, just looked at it [15:47] cool [15:48] pmcgowan, did the titles fixed get approved for rtm like for landing today? [15:48] or for ota? [15:48] we should probably ota that one [15:49] kenvandine, we can land it now if you want [15:49] it's non trivial and cosmetic [15:49] if for today, we should just take what's already in vivid [15:49] i'd like too [15:49] if you think its too risky we can defer [15:49] it's been in vivid for a while now [15:49] seb128, what do you think? [15:49] i think it's awesome :) [15:49] kenvandine, wfm [15:50] seb128, ok, can you prepare just the backport then? [15:50] it's a bit of suboptimal to have it incomplete, but better that nothing [15:50] k [15:50] we'll handle jgdx's later [15:50] k [15:52] kenvandine, So I'm not sure how to update this silent mode autopilot test. Is there an example of someone using an action in the autopilot tests? [15:52] kenvandine: ok [15:53] kenvandine, btw I tested in 127 mako [15:53] tedg, there is an existing test for that [15:54] dednick, ah... found the cellular data roaming problem [15:54] in merging that the Binding got dropped [15:54] but i think we still need that [15:55] kenvandine, Do you know which one? :-) [15:55] the silent mode test :) [15:55] tedg, i really don't remember though... i just know there was a test [15:56] and that uses an action [15:56] dednick, so in the Binding, i want the property to be serverChecked right? [15:56] not checked [15:57] kenvandine: line? === chihchun is now known as chihchun_afk [16:08] kenvandine, https://code.launchpad.net/~seb128/ubuntu-system-settings/titles-uis-tweaks-rtm/+merge/241578 [16:09] dednick, sorry... http://bazaar.launchpad.net/~ken-vandine/ubuntu-system-settings/rtm-lp1336715.check.sync/revision/937 [16:09] dednick, what do you think? [16:12] kenvandine: eh? whats the binding for? [16:12] seb128: have you ever had the duplicated indicator messages in the wild again? [16:13] Chipaca, no [16:13] seb128: ok [16:14] kenvandine, So, I really can't find it :-/ Any ideas of what to look for? [16:16] dednick, oh... i was wrong [16:16] doesn't need the Binding, just fix for poweredSim [16:17] kenvandine: hm. might also need a check for a change in what "getOnlineSim()" returns. [16:17] dednick, ok, dropped the Binding and pushed again [16:18] larsu: did you do anything about the weird delete index error in the messaging indicator? [16:18] if there isn't already [16:24] dednick, got a bug number for the indicator bug? i want to reference it in the test plan for the silo for QA [16:24] pitti, pmcgowan mentioned that dpm and you discussed reviewing the set of font and lang packs in the touch seed, is bug #https://launchpad.net/bugs/1378184 on your radar? [16:24] Ubuntu bug 1378184 in ubuntu-touch-meta (Ubuntu RTM) "RTM r3 why is Nepali Unicode texts not displaying in browser app" [Undecided,Confirmed] [16:28] tedg, sorry, let me look now [16:28] mpt: when can i have 10 minutes of an mpt to go over system update notification pitfalls with you? [16:28] Chipaca, ask Magdalena or Benjamin [16:29] Chipaca: weird index error? [16:29] tedg, ok... i was wrong we don't directly mess with the actions, we mock accountsservice [16:29] mpt: ok [16:29] larsu: the one on delete [16:29] or remove? [16:29] tedg, now i remember what jgdx was complaining about... the actions being hard to mock/test :/ [16:30] larsu: let me dig up the pastebin for you again [16:30] kenvandine, Hmm, okay. They're not hard as much as it's easier when someone's done the foundation for you :-) [16:31] larsu: “g_menu_remove: assertion '0 <= position && position < menu->items->len' failed” [16:31] oSoMoN: (in meeting -- but no, not on my radar) [16:31] tedg, or that :) [16:31] tedg, so do the foundation for jgdx :) [16:31]