=== chihchun_afk is now known as chihchun === chihchun is now known as chihchun_afk === chihchun_afk is now known as chihchun [06:26] good morning [06:33] seb128: ping [06:43] dholbach: Morning :) [06:43] hi mihir [06:43] dholbach: hey , how are you doing ? how was your weekend ? [06:44] good good - how are you? [06:46] dholbach: it was long weekend in India , it was good here :) [06:52] seems something changed in webview component of recent builds. my two apps based on webview can't show the content of web page but just show busy icon. [06:52] mihir, very nice! :) [07:09] good morning [08:39] oSoMoN: you have BQ phone already, right ? I just got one. Anything I should know before I try to flash it ? [08:58] ybon: https://bugs.launchpad.net/ubuntu-app-launch/+bug/1276658 [08:58] Ubuntu bug 1276658 in Ubuntu Application Launcher "Add offline cache" [Undecided,Confirmed] [08:58] see my comments [08:58] now we need a ual developer [09:15] Good morning all; happy Monday and happy Bad Poetry Day! :-D [09:16] I could write poetry, and I could do well, but I didn't do that, when you read this you can see [09:16] something like that JamesTait ? [09:17] justCarakas, good effort! [09:17] kalikiana: thanks [09:18] :p I actually do write poetry :p [09:18] kalikiana: do you think there is any workaround? Can I set env var from my .desktop file for example? [09:18] justCarakas, nothing involving green putty, I hope. ;) [09:21] nope [09:21] JamesTait: http://carakas.be/gedichten/2013/What+do+you+see [09:22] most of my poems are dutch [09:23] I can relate to the sentiment of that poem. [09:23] :) [09:36] ybon: mmm I don't think you can. unless you had C++ code [09:42] justCarakas: first line, youR [09:43] I know exactly the feeling described there [09:43] nicely written [09:44] hehe thx kalikiana [09:44] * kalikiana will read the Dutch ones later as they're harder to process in my head :-P [09:44] :) [09:47] fixed it, it should be updated when the serverside cache is updated [09:48] beuno, did you see the bug report about myapps not allowing some apps to be published? [09:50] bug 1358102 [09:50] bug 1358102 in Developer registration portal "Some apps can't be approved" [Undecided,Confirmed] https://launchpad.net/bugs/1358102 [10:03] dholbach: thanks for filing that bug [10:03] popey, no worries [10:33] popey: Can you help find someone to review some qml MPs of the clock app? [10:33] popey: I will meanwhile review and fix some calendar app stuff [10:34] nik90, I'm available right now [10:34] rpadovani: Hi, can you review the MPs proposed by me at https://code.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/utopic-3.0/+activereviews [10:35] rpadovani: in particular https://code.launchpad.net/~nik90/ubuntu-clock-app/add-custom-sound-backend/+merge/225812 which has been around for 41 days nearly [10:35] nik90, wow, Lot of work :-) I'm on it! [10:36] rpadovani: 4 MPs.. although the 4th https://code.launchpad.net/~nik90/ubuntu-clock-app/improve-timezone-error-handling/+merge/231070 has been reviewed and approved already. So 3 Mps then :) [10:36] thanks rpadovani ☻ [10:37] mzanetti: can you approve and top approve https://code.launchpad.net/~nik90/ubuntu-clock-app/improve-timezone-error-handling/+merge/231070. You had a first look at it in the weekend. It is also approved by DanChapman. [10:37] mzanetti: actually don't top approve since its prerequisite branch needs to go in first. [10:38] nik90: seems we wont have a promoted image today, so I'd recommend pushing a new version to the store as and when you're happy with what's landed [10:38] we'll look to switch the apps as soon as we're confident we have a promotable image coming [10:38] popey: I have already pushed the 3 MPs above to the store since friday :D [10:39] popey: but yeah I will push out an update if we land more fixes [10:39] nik90, nameFilters: [ "*.ogg", "*.mp3" ]. Why only these two? :-) [10:40] rpadovani: ah that's because those two file types are the only ones in the ringtone folder. [10:40] rpadovani: we don't support custom ringtones yet [10:40] nik90, ahhhh, ok, thanks! [10:40] code looks good, time to try on phone! [10:40] Good Morning all o/ [10:41] Good Morning :D [10:45] mihir: do you know when kunal usually comes online on IRC? [10:54] There is someone on rev 197? [10:55] rpadovani: me. why? [10:56] nik90, does the datetime indicator works? On my phone the clock in up right angle is blocked always on the same minute [10:56] rpadovani: yup, known bug [10:56] rpadovani: also the datetime indicator is empty [10:56] nik90, yap, ok, thanks [10:56] rpadovani: it is one of the promotion blockers which started at #189 [10:56] nik90, btw, approved and top approved https://code.launchpad.net/~nik90/ubuntu-clock-app/add-custom-sound-backend/+merge/225812 [10:57] rpadovani: thnx a lot [10:57] * rpadovani needs to read blocker mail, lol [10:57] nik90, next one? https://code.launchpad.net/~nik90/ubuntu-clock-app/json-search-ui/+merge/230339 ? [10:57] rpadovani: yes [11:04] nik90: nope , he usually not on IRC, you can send an email to him [11:04] popey, will take care of it [11:04] mihir: :/ thnx anyway [11:04] nik90: np :) [11:05] nik90: i might need your help on customizing AbstractButton [11:06] mihir: I am working on improving https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/CanderMan-List/+merge/231122, but just ping away if you are stuck [11:10] nik90: i will be doing after my office hours not now..:) [11:10] mihir: ack [11:11] popey: it seems like none of the calendar app files have copyright license headers [11:11] hah. doh [11:11] let me see... [11:13] nik90: popey you meant this , http://paste.ubuntu.com/8079157/ ? [11:14] yes [11:14] mihir: yes [11:14] mihir: although that copyright header was improved in the new clock app [11:14] its in the debian/copyright, but not the individual files [11:14] nik90: i'll update this..i'll take reference from Clock app. [11:15] popey: it has to be in all the .qml / .js files ? [11:15] nik90: ^^^ [11:15] mihir: take it from the new clock app. https://bazaar.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/utopic-3.0/view/head:/app/ubuntu-clock-app.qml [11:15] anything that consititudes code.. [11:15] nik90: sure , thanks for this. [11:15] mihir: np [11:16] mihir: Btw I will be updating all import statement to Ubuntu.components 1.1 [11:17] nik90: I was doing that, but i wanted to make sure it doesn't break any functionality, that's why i was little worried to change. [11:17] mihir: well easiest way to check is updating them and run the ap tests, no? [11:18] nik90: hmm yup that's the way, we're still left with writing AP's for all the functionalities. [11:18] mihir: ok [11:19] nik90, I don't know if you know, but the city online search is a bit strange. If I want to add "Samarcanda", if I wirite "Samarcan" nothing appears, and I have to write all the name. I think should appears all the names that partially match the textfield, if 4 or more letters have been written [11:19] popey: does this bug still exists , https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1341697 ? [11:19] Ubuntu bug 1341697 in Ubuntu Calendar App "Future instances of repeat events not showing after sync" [Critical,Triaged] [11:21] mihir, I'm not able to reproduce it with my google calendar [11:21] rpadovani: so it shows all the recurrence for you ? [11:22] mihir, yap [11:22] mihir, I have all the ubuntu meeting until the eternity :-) [11:23] hmm thanks for confirming rpadovani :) [11:23] mhall119: ping !! [11:27] dholbach: can you review https://myapps.developer.ubuntu.com/dev/click-apps/989/ ? seems to have been sat around a while? [11:27] popey, bizarre - I didn't see it in the review list [11:28] yeah, odd [11:28] I still can't see it there [11:28] beuno, ^ [11:28] popey, let me review the others first [11:28] k [11:28] popey, maybe beuno can let us know why it's not turning up on the review list right now [11:28] popey, dholbach, ack, also addde to my list [11:29] beuno, thanks [11:29] dholbach, the short answer is [11:29] because it's in "needs information" from the deeloper [11:29] beuno, but since then he uploaded two new versions [11:29] but i updated it [11:29] oh, i have a "resubmit application" button [11:30] let me press that :D [11:30] you guys just want to make me look bad [11:30] Your application has been submitted for review. [11:30] yes, thats all i live for [11:30] beuno, are there other ways to upload a new version? [11:30] other than "resubmit"? [11:30] yeah, I think the UI around some of these flows can be improved a bit [11:30] dholbach: does it show in the list now? [11:30] dholbach, I think in the initial submission stage, no [11:30] popey, yes [11:30] but I'll fix that to feel more natural and predictable [11:31] sweet [11:31] it's obscure for historical reasons [11:31] beuno, I'm just wondering if other apps were caught in the same trap [11:33] maybe [11:33] hum [11:33] popey, all your apps are approved now [11:33] lets call it "vacation" instead of "trap" [11:33] ta [11:33] [Send app to happy place] [11:33] need a button which says that [11:34] popey, I await your greasmonkey extension. [11:34] hah [11:45] :-) [12:09] popey, there is an open ticket on canonical portal about predictive search for http://geoname-lookup.ubuntu.com/ ? [12:09] popey, I mean, I expect that 'Samarcan' gives me 'Samarcanda', but if you query Samarcan result is empty [12:10] http://geoname-lookup.ubuntu.com/?query=Samarcan [12:10] http://geoname-lookup.ubuntu.com/?query=Samarcanda [12:11] no bug that I know of [12:11] popey, could you please open one? I think is quite important [12:12] did it used to work? [12:12] dunno [12:12] maybe is a feature request [12:12] let me check in the old clock app [12:12] ah, in the old clock app yes [12:12] http://api.geonames.org/searchJSON?style=full&username=uweatherdev&name_startsWith=Farnbor&maxRows=25&featureClass=P [12:13] rpadovani: https://bugs.launchpad.net/ubuntu-geonames is the bug tracker for it [12:13] thanks sir [12:13] rpadovani, popey: It doesn't work in the old clock app. So we are not regressing, [12:13] its a feature request [12:13] https://bugs.launchpad.net/ubuntu-geonames/+bug/1045224 [12:13] Ubuntu bug 1045224 in Ubuntu Geonames "search string 'Pari' doesn't return 'Paris' " [Low,Confirmed] [12:14] Do you think we can up the Importance? i mean, maybe it's me, but I'm a bit annoyed about this [12:15] rpadovani: ok, but I don't think we can get it in before RTM. But let's see what happens [12:17] rpadovani: thnx a lot [12:18] nik90, yw, anything else? [12:19] rpadovani: how familiar are you with QML unit tests? [12:19] nik90, ~0 [12:19] I use QTests as recommended by mzanetti [12:19] rpadovani: ah ok. np [12:20] rpadovani: I recommend keeping an eye on https://code.launchpad.net/~nik90/ubuntu-clock-app/add-first-qmltest/+merge/230702 since you might need to add unit tests to reminders app as well [12:20] nik90, thanks! [12:24] nik90: how do get the correct time out of the timezone? since geoname-lookup has no "dst" field for timezones [12:25] nik90: is there or will be a sdk component for this? [12:25] m-b-o: you have to use QTimeZone class where you can set the timezoneID and get the local time there [12:26] nik90: clock app has thisalready, I guess? [12:26] m-b-o: QTimezone will automatically take care of dst and all the other rules [12:26] m-b-o: yes, we wrote a custom c++ plugin class [12:27] nik90: ok, thanks! will borrow that one :) [12:27] m-b-o: :) You can't borrow it just like that though since we integrate and calculate the data in a list which was required for the world clock city list [12:27] m-b-o: https://bazaar.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/utopic-3.0/files/head:/backend/modules/Timezone/ [12:28] m-b-o: however you will be able to see what functions we use. So it shouldn't be hard to create one quickly for your usecase [12:28] nik90: ok, I will have a look :) [12:33] rpadovani: I'll take a look at it [12:33] thanks === MacSlow is now known as MacSlow|lunch [12:52] dbarth: if you can confirm it builds with that changed in debian/control at lp:~mardy/ubuntu-system-settings-online-accounts/master , you could probably just quickly rebuild it. [12:52] mardy: changing ^ to this channel since you're here, libmirclient-dev dependency would need to be libmirclient-dev [!arm64 !powerpc !ppc64 !ppc64el] (and online-accounts would need to accept it can build without it) [12:54] mardy: or simply !powerpc !ppc64el would be enough for Ubuntu, arm64 has Mir at the moment and arm64 build even succeeded in the PPA [12:55] mardy: of course, you may need to change some build files to not build the trusted session / Mir support if it's not installed [12:56] Mirv: ah, I didn't know that mir was not available in all platforms [12:56] Mirv: thanks, I'll fix that [12:58] mardy: thanks! hopefully it's not a big thing, but at least it should have zero effect on other platforms where things continue to be built normally [13:09] kalikiana: zsombi: Hi guys! Is there a chance we can do anything to get this landed? [1] It's been a while now since this branch has been proposed, and I heard from Christian it's not being landed because of unrelated unstable tests. This bug is pain to users testing our app. https://code.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/organizationDomain/+merge/230085 [13:09] Your insight appreciated, thanks! [13:10] karni: once the app_launcher gets fixed... sorry, yet all our stuff - since last Wednesday!!! are pending there [13:12] zsombi: I see. Do you know if it's actively being worked on (app_launcher)? [13:12] karni: yes, it is [13:12] zsombi: thanks man [13:12] karni: welcome === _salem is now known as salem_ [13:32] nik90: hey, here's a quiz for you: how many bugs can you spot? http://i.imgur.com/yLOs1Dl.png [13:34] haha [13:35] mzanetti: the times are off...that's indicator-datetime's fault :P [13:35] translation... [13:35] nik90: no.. the app froze [13:35] wat [13:35] oof [13:35] nik90: yes, the translation too [13:35] and the Next alarm in is too small, doesn't say when [13:36] mzanetti: ah the Next Alarm string is not yet implemented.. I am working on it atm. [13:36] mzanetti: so it shows "next alarm in..." always :D [13:36] nik90: I guess it shouldn't be in that label though [13:37] mzanetti: what do you mean? When the feature is implemented it will "next alarm in 3h20m" according to design atleast [13:37] nik90: but when the app is not frozen it looks like this: http://i.imgur.com/lJRCPBv.png [13:37] doesn't really help [13:38] mzanetti: https://bugs.launchpad.net/bugs/1357945 [13:38] Ubuntu bug 1357945 in Ubuntu UX "[bottom edge] Current bottom edge hint timeout makes it difficult to read the bottom edge action" [Low,Confirmed] [13:38] mzanetti: I have requested that it behaves like the unity8 dash bottom edge hint [13:38] nik90: we (as in unity-team) are really unhappy with that label tbh [13:38] and it's probably going away [13:39] mzanetti: oh :P [13:39] mzanetti: well dialer, contacts etc uses it and I don't think we will change that before RTM. it is design call I am afraid [13:39] yeah... [13:39] well, just saying [13:40] btw how did you get the clock app to freeze? Did it run for a really long time or something? [13:40] it could be an indicative of a memory leak some where perhaps? [13:40] dunno... I woke up at 8 today. decided I can't get up yet, set an alarm to 9 and went back to sleep. now I just noticed that the app is still open, but not updating time any more [13:40] the time it froze is when I woke up and changed the alarm... [13:41] is there someone of online accounts I can ask a question ? [13:41] justCarakas: you can try mardy [13:41] * mardy hides ;-) [13:41] found you :p [13:42] mzanetti: hmm strange...I will try to keep the app open for a long time and see if I can reproduce it. [13:42] I figured popey might have hit this issue long time ago with his usual experience to hit strange bugs :P [13:42] ☻ [13:42] nik90: btw... the app was crashing constantly when setting the alarm (might be related to the bug I fixed which is not released yet) [13:43] mzanetti: yup..oh you are on devel! [13:43] nik90: so I had old and new clock open and tried getting that alarm set to 9 with both... eventually I succeeded and left both open [13:43] mardy: this bug was filed a while ago to the calendar team but they say it is for online accounts but after that it looks like an abandoned place, can you tell me if and when there could be a fix for this https://bugs.launchpad.net/sync-monitor/+bug/1309042/ ? it is keeping me from switching to U touch [13:43] Ubuntu bug 1309042 in Ubuntu Calendar App "Calendar sync only syncs the default calendar" [Wishlist,Confirmed] [13:43] nik90: no... on the promoted one. I installed the new clock app from the store [13:43] mzanetti: yeah that's what I meant..I am rocking the devel-proposed one :) [13:44] nik90: yeah, it's my dogfooding phone [13:44] mzanetti: :D [13:44] mzanetti: oh btw go ahead and add some world cities that you helped implemented [13:44] implement* [13:44] so the thing is, I usually don't care much for the time in other places :D [13:45] lol [13:46] london might be an exception... but I mostly manage to subtract 1 :) [13:46] ok. lemme add my work mates timezones, just to test the feature [13:48] looks nice :) [13:48] :D [13:48] nik90: wondering if the local clock shouldn't stick around and only the list of other places should scroll [13:48] it helps me know when to ping charles, mhall119, popey etc.. [13:49] mzanetti: but then you need to scroll the local clock down to see the add city button [13:49] yeah... fair point [13:50] although I keep triggering that "add city" button accidentally when I try to scroll down after seeing the world city times. [13:50] I might ask the designer to see if we can change that [13:51] justCarakas: no, online accounts doesn't know anything about calendars [13:51] nik90: +1 [13:52] nik90: pong? [13:52] any idea what I should do to get it fixed mardy [13:53] charles: oh no..I didnt ping you [13:53] :D [13:54] nik90: my suggestion: use the normal page header, move the add city into the settings button and make only the city list scroll [13:54] justCarakas: you'd better talk with bfiller, I don't know enough about calendar-app to help you [13:55] nik90: actually, I guess alarms are more important than world clocks... so maybe swap the alarms pull-up thing with the world city list even [13:55] bfiller: this bug was filed a while ago to the calendar team but they say it is for online accounts but after that it looks like an abandoned place, can you tell me if and when there could be a fix for this https://bugs.launchpad.net/sync-monitor/+bug/1309042/ ? it is keeping me from switching to U touch [13:55] Ubuntu bug 1309042 in Ubuntu Calendar App "Calendar sync only syncs the default calendar" [Wishlist,Confirmed] [14:00] mzanetti: I will check with design === zbenjamin_ is now known as zbenjamin [14:07] nik90: btw, I'm still getting https://bugs.launchpad.net/indicator-datetime/+bug/1358309 [14:07] Ubuntu bug 1358309 in Indicator Date and Time "Repeating alarms don't repeat" [Undecided,New] [14:08] popey: are you able to reproduce ^^ [14:09] nope [14:09] mhall119: I have been waking up to my daily alarms for several weeks now..I am not sure why I cannot reproduce it. [14:09] i set two alarms on my phone, one for 12:00 lunch and one for friday at 17:00 and they both go off all the time [14:09] exactly [14:09] mhall119: old or new clock app? [14:09] do you both have calendar events synced to those phones? [14:09] yes [14:09] nik90: doesn't seem to matter [14:09] mhall119: I do have calendar events synced [14:09] nik90: I'm pretty sure it's in the datetime indicator [14:09] not the clock app [14:11] its really weird === MacSlow|lunch is now known as MacSlow [14:37] dholbach, I'm in for the frameworks air thingie, btw [14:37] rock and roll [14:37] bzoltan, ^ you as well? [14:38] dholbach: beuno: I am all here boys, shaved and fresh :) [14:39] rock and roll - see you in a bit more than an hour then :) [14:39] if I have to shave, I'm out. [14:43] beuno: you can disable the video :P [14:44] but then you wouldn't see my awesome beard, and then we all loose [14:44] lol [14:46] popey: design bugs incoming :D...I got my work cut out for the week [14:47] nik90: "yay" [14:47] :P [14:53] zsombi: Would you have time to work on the alarm enable/disable switch issue with me tomorrow? It is getting more attention from the designers. [14:53] nik90: tomorrow yes, not today pls [14:53] dholbach: I'll bring kalikiana too [14:54] zsombi: me busy today too [14:55] bzoltan, the more the merrier [15:55] dholbach: will you paste the link to the hangout? [15:55] if you're interested in a discussion about ubuntu app frameworks, we are going to have a hangout in 5m on http://ubuntuonair.com [15:56] let mhall119 or myself know if you want to join in === chihchun is now known as chihchun_afk [16:08] seb128: ping [16:09] zsombi, you already did that this morning, I usually don't reply to contentless pings... [16:10] seb128: ah, ok :D [16:10] zsombi, if you have a question please state it so I can decide if I've the slots to start the discussion [16:10] seb128: Slider issue in System Settings [16:10] what sort of issue? [16:10] seb128: ai, sire! [16:10] seb128: lemme show the bug [16:11] seb128: bug #1353966 [16:11] bug 1353966 in Ubuntu UI Toolkit "[SDK] Slider does not work in a List View" [Medium,In progress] https://launchpad.net/bugs/1353966 [16:12] balloons, hey [16:13] seb128: so, Ive tried to alter the Flickab;e's interactive property with a Binding, but that doesn't return the original value if there was no property bound to the interactive :/ [16:14] brendand, howdy [16:15] balloons, in http://bazaar.launchpad.net/~ubuntu-filemanager-dev/ubuntu-filemanager-app/trunk/revision/253#manifest.json.in were you thinking that the x-test:autopilot:depends property would work? [16:16] seb128: what I was thinking is that you can eventually provide a temporary binding in your Flickable - which is as ugly as it can be - so the Binding can return the proper value, or we simply alter the Flickable's interactive property by assignment, which in your case migth work until you do a binding on that [16:16] brendand, ? what do you mean? if I remember correctly I ran using that setup [16:16] seb128: in which case the binding will be broken [16:16] balloons, it seems ci didn't like it [16:17] balloons, which is using phablet-click-test-setup obviously [16:17] zsombi, not sure I've an opinion on that, the different solutions seems like workaround [16:17] brendand, it's for autopkgtest, but shouldn't break anything [16:17] brendand, reminders has the same in trunk :-) [16:17] balloons, no - but it doesn't fix it either :) [16:17] zsombi, kenvandine did the change to add the slider to the sound panel btw, maybe he has an opinion [16:17] could you try asking on #ubuntu-touch? [16:18] brendand, "fix"? fix what/ [16:18] oh [16:18] he's not online [16:18] balloons: ping [16:18] I feel like I'm missing some context here [16:18] seb128: ok, I'll try to catch him [16:18] balloons, filemanager broke on the dashboard [16:18] zsombi, thanks [16:18] seb128: both are workarounds and the upstream bug is not progressing :( and there's no trivial solution on that [16:19] balloons, ci uses phablet-click-test-setup and that doesn't cause python3-lxml to be installed, so it was failing with the missing dependency [16:19] seb128: I thank you, finally I know to whom to talk :D [16:19] zsombi, I would suggest you just pick the workaround you find best and merge propose the change? [16:20] brendand, ahh sure. Adding x-test had nothing to do with ci, just making sure autopkgtest continues to work [16:20] seb128: yeah... the toolkit's workaround sounds better, at least we can control it more, but in case it overrides the property binding, it won't work for anyone :( that's why I thought maybe a small hack on app would be wiser... [16:20] seb128: but I'll catch up with Ken, thx!! [16:20] yw! [16:22] brendand, for python3-lxml it will need to be an MP in CI tools.. outside of autopkgtest, there's not way to fix it [16:22] so we can push something to lp"ubuntu-test-cases [16:23] balloons, that's cool - i'm just wondering if it was detected in CI before landing [16:23] balloons, or not [16:27] brendand, I assume it would fail unless python3-lxml is installed on the image by something else [16:32] nik90: ping [16:35] nik90: i have pushed MR for copyright headers , https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/CopyrightHeaderComments/+merge/231227 [16:35] mihir: go ahead...brb in 15 secs [16:39] mihir: will look at it. Can you review https://code.launchpad.net/~nik90/ubuntu-calendar-app/revamp-calendarchoicepage/+merge/231176 for me pls [16:39] * mihir reviews MR [16:55] popey, ping === dholbach_ is now known as dholbach [16:58] nik90: reviewed your MR :) [17:00] mihir: I approved and top approved your mr [17:00] nik90: thanks :) [17:00] mihir: u 2 :) [17:00] balloons: hi [17:01] mihir, gello [17:01] balloons: do you have couple of mins ? [17:01] mihir, what's up? [17:02] balloons: i am doing great, how was your weekend ? [17:02] mihir, excellent, thanks [17:02] balloons: i was wondering if we can fix the Ap issues on Calendar. === chihchun_afk is now known as chihchun [17:03] mihir, sure, I seem to remember a new-event issue.. is that what you mean? [17:03] balloons: yup , you showed that is breaking because of delay bug , and you said my machine is fasters so it passes all the Tests heehe [17:04] mihir, ahh right indeed. So mihir do you have a thought on how to wait for things to be loaded? [17:04] balloons: i guess , i should add sleep to break AP on my system right ? [17:04] mihir: btw you guys referring to https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/1188819/+merge/230554 ? [17:05] I believe so [17:05] ok [17:05] nik90: yup [17:08] balloons: let me try that. [17:09] mihir, we want to avoid a sleep, and instead use a wait_for().. we just need something to wait for, heh. Meaning, we need a way to know when the page has fully loaded. Is there a qml property we can use for that page? [17:09] you have the spinner animation, for instance, we could wait till that object is destroyed [17:10] make sense ^^ [17:10] ? [17:10] balloons: yup it does [17:22] nik90: i wounder how did Jenkins passed your MR..:-| [17:22] wonder* [17:22] balloons: ^^ [17:23] mihir, ? who's mr and why? I'm confused? [17:23] balloons: i just took latest from trunk , made changes and pushed it Jenkins failed it ttps://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/CopyrightHeaderComments/+merge/231227 [17:23] balloons: same has been done by nik90 and it Jenkis approves the MR..https://code.launchpad.net/~nik90/ubuntu-calendar-app/revamp-calendarchoicepage/+merge/231176 [17:24] mihir, the issue you are fighting occurs in trunk too [17:24] mihir, look at the very end of http://91.189.93.70:8080/job/generic-mediumtests-utopic/1596/artifact/calendar_app.tests.test_new_event.NewEventTestCase.test_delete_event_must_remove_it_from_day_view%20%28with%20mouse%29.ogv, from that mp [17:25] balloons: i see that , i wonder how did nik90 MR got approved [17:25] balloons: it is just about time , so it passes ? [17:26] mihir: no the failure happens occasionally [17:26] mihir: I noticed my MP fail the first time due to the delete event AP test failing [17:27] nik90: yup , that's what I am confirming.. [17:28] mihir, nik90 yep yep [17:34] so mihir can you make the fix? [17:35] balloons: i am trying with , ensure_visible i am not sure this would be correct method or not looking into APIs [17:35]