/srv/irclogs.ubuntu.com/2014/08/14/#ubuntu-app-devel.txt

=== _salem is now known as salem_
=== salem_ is now known as _salem
Mirvnik90: hey, sure! my hours just are a bit like this :)03:56
=== chihchun_afk is now known as chihchun
dholbachgood morning06:22
mihirGood Morning :)06:54
dpmmorning mardy - if I include an authentication plugin inside a click package, what will the user experience be for authentication? Will the account appear in system settings to choose from?07:09
mardydpm: hi! Yes, it will07:09
dpmah, cool07:10
justCarakasGoeie morgen07:10
=== chriadam is now known as chriadam|away
=== dkessel_ is now known as dkessel
nerochiarodpm: sorry, moving conversation here07:44
nerochiarodpm: more appropriate07:44
dpmnp :)07:44
dpmnerochiaro, that looks good to me. What is is you're mentioned you did differently as originally suggested?07:45
nerochiarodpm: essentially the problem was that i was looking for .in.in.h in / instead of /po, so i fixed it as in here: http://bazaar.launchpad.net/~phablet-team/camera-app/camera-app-desktop-translation/revision/28807:47
nerochiarodpm: the rest was all ok07:47
ybonpopey: Your application SensorsStatus has completed the approval process. It is now available to the world from Ubuntu Software Center. \o/07:48
dpmnerochiaro, ah, that's what I was mentioning that "RELATIVE ${CMAKE_CURRENT_SOURCE_DIR}" on the line above should be "RELATIVE ${CMAKE_CURRENT_SOURCE_DIR}/po"07:48
nerochiarodpm: right, different ways to fix the same problem i guess07:48
nerochiarodpm: i can use your solution if you want, but i think this is more clear (at least to me). more explicit07:49
=== Guest75668 is now known as wgrant
=== wgrant is now known as Guest89446
dpmnerochiaro, the only reason I suggested that was because those paths are put in the .pot file and shown to translators, and by convention they are relative to the po/ folder07:51
nerochiarodpm: ok, i can change it that way then, if you prefer07:51
dpmnerochiaro, if that's ok with you, I would still suggest to make it relative to po/ to be consistent with all other core apps and for translators07:52
dpmnerochiaro, also, I'm not sure if the .pot file in the MP is up-to-date, but it does not contain the .desktop file translations yet07:52
popeyybon: sweet!07:53
nerochiarodpm: ok, let's do those07:53
dpmcool, thanks07:53
zsombi_mihir: ping07:53
dpmybon, just tried your app, really cool :)07:53
ybonthanks :)07:53
ybonMade that initially as a helper for me, then thought it could be a helper for everyone :)07:54
=== Guest89446 is now known as wgrant
mihirzsombi_: pong07:55
zsombi_mihir: you're with Calendar app, right?07:55
ybonbtw, I wonder why when you put your phone on a table without moving it the accelerometer values are not equal to 0, and they keep changing07:55
mihirzsombi_: yes07:55
zsombi_mihir: I've some suggestion for you for the DatePicker07:55
ybonMaybe because Earth is moving? :p07:55
zsombi_mihir: like on how to use it07:56
mihirzsombi_: sure , suggestions are always welcome :)07:56
zsombi_mihir: we are implementing the focus grabbing on teh components and I saw something in the NewEvent.qml that causes some problems07:56
zsombi_mihir: I saw you were using TextField for the date and time fields07:57
mihirzsombi_: yup, we are using it for creating new event , and time07:57
zsombi_mihir: it's not the best idea, especially that you are not making those textfields read-only...07:57
mihirzsombi_: they are readonly ,07:57
zsombi_mihir: I'd rather do some custom component instead of using that07:57
mihirzsombi_: it pop up Date & TIme picker at bottom of screen07:58
zsombi_mihir: I don't see any readonly setting of it...07:58
zsombi_mihir: startDateInput is set to RO where?07:59
mihirzsombi_: I see , we have used MouseArea, http://bazaar.launchpad.net/~ubuntu-calendar-dev/ubuntu-calendar-app/trunk/view/head:/NewEvent.qml#L39608:00
zsombi_mihir: but that's the smallest issue. The bigger one is the MouseArea that you place over it08:00
mihirzsombi_: hmmm , i see that now, zsombi_ we were using custom component previously developed by Kunal , but after we got this into SDK we start using this.08:01
nerochiarodpm: i don't get it. if i use relative to po then it will find the .in.in.h but it won't find the QML files anymore08:01
nerochiarodpm: i don't understand how it can find them in the core apps08:01
zsombi_mihir: but the thing is that teh component is too heavy for the purpos you need it08:01
mihirzsombi_: to match the design guidelines08:02
zsombi_mihir: beside, highlighted property will be soon deprecated, teh new design does not have that functionality anymore08:02
mihirzsombi_: oops, :|08:02
dpmnerochiaro, in the core apps, in here we add a line to look for in the build dir too -> http://bazaar.launchpad.net/~phablet-team/camera-app/camera-app-desktop-translation/view/head:/po/CMakeLists.txt#L1508:02
zsombi_mihir: I do get it, but the problem is that you shoudl always challenge design, especially if the page loading seems slow, or the content seems crowded ;)08:03
mihirzsombi_: what would you recommended then , custom component would be too heavy task to match our design .08:03
dpmnerochiaro, i.e.08:03
dpm        -D ${CMAKE_CURRENT_SOURCE_DIR}08:03
dpm        -D ${CMAKE_CURRENT_BINARY_DIR}08:03
zsombi_mihir: perhaps revisit the design with teh designer08:03
mihirzsombi_: yup true.08:03
dpmnerochiaro, and in here: http://bazaar.launchpad.net/~phablet-team/camera-app/camera-app-desktop-translation/view/head:/po/CMakeLists.txt#L18 - you'll need to add --keyword=N_ too08:04
mihirzsombi_: there is no way we can utilize SDK datePicker or Datepicker panel ?08:04
zsombi_mihir: because this text field would be useful if the OSK would have the ability to open the DatePicker, or, more if the user would be able to enter the date manually08:04
zsombi_in this way it cannot08:04
zsombi_mihir: of course you can!08:04
zsombi_mihir: you can invoke PickerPanel.openDatePicker() from any omponent, AbsttactButton, Button, whatever08:05
zsombi_*component08:05
nerochiarodpm: are you sure you are linking to the right file ? i don't see -D ${CMAKE_CURRENT_BINARY_DIR} anywhere in the file you just linked08:06
dpmnerochiaro, that's the camera file, it needs to be added there08:06
mihirzsombi_: hmmm i see.. we have calendar meeting today with popey will have discussion of this08:07
zsombi_mihir: if you need same design, you can do that with a custom component derived from UbuntuShape + SharedSource to be able to render the content08:07
zsombi_mihir: please do08:07
mihirzsombi_: if we can make it work using readonlying textbox ?08:07
zsombi_mihir: also remember that lots of UbuntuShapes on a UI cause performance decrease08:07
mihirpopey: ^^^08:07
zsombi_mihir: no, you cannot :/08:08
nerochiarodpm: why keyword "N_" ? where is it used ? I thought the desktop file used just "_"08:08
zsombi_mihir: the problem is that even if you make a textfield RO, it won't open the PickerPanel08:08
dpmnerochiaro, look at the generated .desktop.in.in.h file, it uses N_08:08
nik90t1mp: Hey Good Morning, I am in the process of writing some unit tests for the clock app and I am wondering how to press the clear button in a textfield. At the moment I press on the right edge corner of the textfield.08:08
mihirzsombi_: hmmm , what about putting clickable label with some shadow and highlights ?08:08
mihirzsombi_: if user clicks , it opens same way and update label values , just a random thought08:09
zsombi_mihir: that's fine as well08:10
mihirzsombi_: thanks for this ,i'll bring up in today's meeting and will file bug for this too.08:10
zsombi_mihir: you can use a Stroke button for instance08:10
dpmnerochiaro, in summary, you'll need this: http://pastebin.ubuntu.com/8043322/ (in core apps we use CMAKE_CURRENT_* for the build and source dirs, not sure if it makes a difference)08:11
zsombi_mihir: that will kinda look nice and you will be able to set its label based on the DatePicker's response08:11
nerochiarodpm: check if it works for you now, just pushed a new commmit08:11
zsombi_mihir: thx dude :)08:11
mihirzsombi_: yeah, we can try stroke button.08:12
zsombi_mihir: it even looks better than a text field ;)08:12
dpmnerochiaro, looks good to me, thanks!08:13
zsombi_t1mp: I have some changes in the header on the focus handling, please check it https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/focus-handling-on-touch/+merge/23029608:14
mihirzsombi_: for using button stroke , we just have to provide property for that ? i couldn't find in docs http://developer.ubuntu.com/api/qml/current/Ubuntu.Components.Button/08:20
zsombi_mihir: wrong URL :)08:21
zsombi_mihir: http://developer.ubuntu.com/api/qml/sdk-14.10/Ubuntu.Components.Button/08:23
zsombi_mihir: there supposed to be a property called "strokeColor", but for some reason the doc doesn't have it...08:23
zsombi_mihir: perhaps the doc on the web is outdated a bit08:24
zsombi_mihir: do you use QtCreator?08:24
mihirzsombi_: yes i do i'll try that :)08:25
zsombi_mihir: then you should have context help, so if you open the help whil on Button, you will see the doc there08:26
nerochiarodpm: can you please work with oSoMoN to get that MR landed and approved ?08:36
nerochiarodpm: also, anything else I can help with in terms of desktop file translations ?08:36
oSoMoNdpm, nerochiaro: which MR?08:36
dpmnerochiaro, yes, the dialer-app and messaging-app don't seem to load translations. I mentioned the details and findings on my e-mail08:37
nerochiarodpm: ok, i'll look into these later today08:38
dpmcool, thanks08:40
dpmoSoMoN, https://code.launchpad.net/~phablet-team/camera-app/camera-app-desktop-translation/+merge/223229 - nerochiaro, I think I'll have to leave it to you guys, I'm not sure I'm the best person to coordinate landings in system apps08:41
nerochiarodpm: can you please top approve the MR if you haven't done it already ?08:42
dpmnerochiaro, I can't, I'm not otherwise involved in system apps development and I'm not a member of the phablet-team08:42
dpmso no permissions to top-approve08:43
oSoMoNnerochiaro, I’ll take a look in a moment08:43
dpmmardy, I'm setting up reminders to ship the authentication plugin in the .click. Does the hook for the account in the manifest need to have a specific name, or match any provider id, etc? I just named it "evernote-account-plugin" -> http://pastebin.ubuntu.com/8043589/08:59
nik90mzanetti: hey, have you had the issue where a QTest passed when you run that specific testcase but fails when you run ctest -v ? I think I may have timing issues since adding a wait(2000) seems to do the trick.09:29
nik90mzanetti: I thought about using signalSpy but then it is a group of listitems that are clicked. So I don't know what the signal spy should target09:30
mzanettinik90: use "tryCompare()"09:30
mzanettinik90: that's the equivalent of AssertThat(Eventually())09:30
mzanettinik90: or well, let me see the code09:31
nik90mzanetti: but it is already outside the loop that presses on the listitem.09:31
mzanettinik90: yeah... not sure exactly what the issue is atm. would need to see code09:31
nik90mzanetti: https://bazaar.launchpad.net/~nik90/ubuntu-clock-app/add-first-qmltest/revision/6009:31
nik90mzanetti: line 80  // #FIXME: Without this wait, it fails when running ctest -V09:32
mzanettiI see09:34
mzanettinik90: remove the wait(2000)09:34
mzanettinik90: and add this as the first line of this test:09:34
mzanettiwaitForRendering(alarmRepeatPage);09:34
mzanettinik90: that once waits until the alarm page has finished rendering and then starts the test09:35
mzanettipasses fine here09:35
nik90mzanetti: ok that passes09:35
mzanettinik90: in gernal waitForRendering(item) should be a good friend of yours when writing qmltests :)09:35
nik90mzanetti: do I add the waitRendering before each testcase?09:35
mardydpm: it can have any name, because it will be renamed by the hook program09:35
mzanettinik90: well, if its not needed, I wouldn't add it09:35
mzanettinik90: but if you have some timing issues, waitForRendering should be the first thing you try09:36
nik90mzanetti: ok, this is kinda cool. I think I am getting comfortable with this :)09:36
dpmthanks mardy09:39
popeynik90: i have updated my merge proposal for admin2, filed an RT, need to find someone to approve the merge.09:39
nik90popey: this is the same MP as the timezone one?09:39
mzanettinik90: *\o/*09:40
dpmmardy, with this in the manifest, I'm getting an error (I believe it comes from the click review tools) "The apparmor path is missing in the manifest file"-> http://pastebin.ubuntu.com/8043589/09:44
dpmmardy, does the account plugin hook need an apparmor entry?09:45
mardydpm: I don't know, but it looks like it does :-)09:47
mardydpm: just ask for network and accounts09:47
popeynik90: yes09:47
popeynik90: updated it.09:47
mzanettinik90: thanks for cleaning up my typos :D09:48
nik90mzanetti: lol09:49
mzanettihow did you find that bug report anyways?09:49
nik90its in the main sdk page as the first bug report09:49
nik90mzanetti: is this the UI toolkit on the latest phone image as well that you are having issues with? or staging?09:50
mzanettinik90: image. I don't use staging here09:51
mzanettinik90: http://i.imgur.com/GDQUlxF.png :)09:51
popeyybon: you might want to switch your sensors app to the new header..09:52
* nik90 tests09:52
ybonpopey: damn, I though it was automatic09:53
ybonOH09:53
ybonI know09:53
ybonI need to switch useDeprecatedAction to false or something like that right?09:53
nik90ybon: affirmative09:53
ybon:)09:53
* ybon prepares a 0.0.2 with that change09:54
nik90mzanetti: https://imgur.com/rKN4JE709:54
nik90mzanetti: why do you have the overflow button09:54
mzanettinik90: because I wanted that. But because of the ugly popover I decided to publish it as is for now09:55
mzanettinik90: I think the way it is atm is not really good for 2 reasons:09:55
mzanettia) you don't need the add account buttons often enough to have them visible all the time09:55
mzanettib) if they're in the popover you get the text explaining what the camera icon actually does09:55
mzanettiso I'd prefer it to have like in my screenshot09:55
mzanettibut obviously not as the popover is atm09:56
nik90hmm I am trying to test against other apps which have a overflow09:56
mzanettinik90: well, the old clock app looks the same09:56
mzanettihttp://i.imgur.com/lz4oQHC.png09:56
nik90mzanetti: that's because its background is purple09:57
mzanettinik90: ^09:57
mzanettinik90: no09:57
mzanettinik90: that's because the background is "dark"09:57
nik90mzanetti: check out the music app, its overflow is black09:57
nik90mzanetti: even in my app Flashback https://github.com/krnekhelesh/flashback/blob/master/Flashback.qml, I set the background color to UbuntuColors.coolGrey and the tab overflow menu looks black.10:00
mzanettinik90: indeed... still broken though... text color should be white10:00
nik90mzanetti: yeah there is a bug for that10:00
mzanettinik90: so why is the authenticator's one purple?10:00
nik90mzanetti: I am not sure, I suspect it could be because of the gradient10:01
mzanettinik90: also the clock app's background purple and the overflow purple do not match10:01
DanChapmanGood Morning all o/10:01
nik90mzanetti: try removing that and setting only the background color10:01
nik90mzanetti: https://code.launchpad.net/~fboucault/ubuntu-ui-toolkit/better_visual_feedback/+merge/23012110:01
mzanettinik90: you're right10:07
mzanettinik90: it only turns purple when there's a gradient10:08
mzanettinik90: which explains why its the same purple in clock app and authenticator10:08
nik90mzanetti: lol10:08
mzanettiSaviq: fyi: ^10:08
nik90mzanetti: I heard the SDK devs are deprecating gradient backgrounds10:08
nik90mzanetti: afaik all core apps were asked to move towards solid backgrounds instead10:08
ybonpopey: you whould now have an ugrade :)10:08
ybonwhat about the name, btw? "Sensors Status" is what I was able to put out from my globish, but it's not that good10:09
Saviqmzanetti, right, so it's a $random_color when gradient is used10:10
mzanettiSaviq: rather $purple, but yeah10:10
mzanettiSaviq: removing the gradient and only setting background color makes the popover black10:11
mzanettiblackish10:11
Saviqwith grey text I assume ;)10:11
mzanettiofc10:11
mzanettihttp://i.imgur.com/xGWiv98.png10:11
nik90mzanetti: both of these are fixed in the MP that I linked above10:12
mzanettinik90: both?10:12
mzanettiwhat's the other issue?10:12
mzanettian the gradient one10:12
mzanetti?10:12
nik90mzanetti: the black box in your overflow10:12
popeyybon: name is fine to me10:12
mzanettinik90: oh really? I just wanted to report a bug10:13
dpmmardy, would you mind reviewing the branch for the migration of the Evernote account plugin into the click package? It's not a big change, but it'd be good check if the layout and the rest is as expected for the account to work inside the click -> https://code.launchpad.net/~dpm/reminders-app/account-click/+merge/23076810:13
mzanettinik90: that seems to be that iconSource is ignored in Action10:13
nik90mzanetti: yup, even reminders app had that issue10:13
mzanettioh did it :D10:13
ybonpopey: okey :)10:13
* mzanetti starts losing track of all his apps10:13
popeymaybe the myapps bit for developers should have a thing to let you view by category/department10:26
dpmpopey, have you come across any message similar to "Unable to exec 'reminders' in '/opt/click.ubuntu.com/.click/users/phablet/com.ubuntu.reminders': No such file or directory" when launching an app from the dash, after having installed the click package manually with pkcon?10:27
dpmThe installation path seems wrong10:27
rpadovanidpm, hey :-) I was able to add bottomEdge to remiders app,  only on phone mode, and it's awesome! I implemented the solution I think is 'cleaner' as possible, but I had to modify the BottomEdge component, and create a 'fake' bottomEdge. I'll explain all in today meeting, cause I want to hear mzanetti's opinion as well.10:29
rpadovanidpm, meanwhile, if you want to try it: https://code.launchpad.net/~rpadovani/reminders-app/bottomEdge10:29
dpmrpadovani, oh wow! nice work!10:29
rpadovanidpm: now I start to work on edit notebooks name10:30
mzanettirpadovani: nice!10:32
mzanettirpadovani: I also found a bug with the pulldown thingie10:33
mzanettirpadovani: pull it down to make the text change to "release to refresh"10:33
mzanettirpadovani: then drag it back up to the beginning and release10:33
mzanettiit still refreshes10:33
mzanettishould only if you release it below the threshold10:33
popeydpm: nope10:33
rpadovanimzanetti, interesting, I'll take a look10:34
dpmmzanetti, rpadovani, also -> https://code.launchpad.net/~dpm/reminders-app/account-click/+merge/23076810:35
mzanettidpm: oh wow! didn't know that's possible now10:36
mzanettigreat news10:36
mzanettidpm: For some reason I don't understand, the binary fails to start with this error, where the layout is clearly wrong: "Unable to exec 'reminders' in '/opt/click.ubuntu.com/.click/users/phablet/com.ubuntu.reminders': No such file or directory"10:37
dpmmzanetti, indeed. However, with a caveat: the 'accounts' policy is still only limited to trusted apps, if I'm not mistaken, so it's not yet available for general use. I think there need to be some security improvements added before all app devs can use it, but mardy can probably expand on that10:37
mzanettidpm: I guess that's because the reminders binary is in lib/armhf../bin/reminders10:38
mzanettiinstead of the top level dir10:38
dpmmzanetti, yeah, but that's always been the case10:38
mzanettidpm: you can either change the install dir of the binary, or update the .desktop file to launch it from where it is now10:38
mzanettireally?10:38
dpmmzanetti, yeah, I've just checked that on the Reminders click that comes preinstalled with the images. I'm a bit puzzled as well that the desktop file finds the binary at all10:39
popeynik90: admin2 merged by ev! :D10:39
nik90popey: wow10:39
zsombi_popey: dpm: guys, who's with teh Settings app?10:40
dpmmzanetti, http://pastebin.ubuntu.com/8044258/10:40
dpmzsombi_, I generally ping seb128 or Laney when I've got questions about u-s-s10:40
mzanettidpm: /usr/armhf/bin vs /usr/x86/bin/10:41
zsombi_seb128: pingus10:41
zsombi_Laney: ping U210:42
dpmmzanetti, where?10:42
mzanettidpm: http://pastebin.ubuntu.com/8044162/  vs http://pastebin.ubuntu.com/8044258/10:42
mzanettidpm: stupid me... I meant /lib/...10:43
mzanettinot /usr10:43
dpmmzanetti, ah, got it, thanks. No idea how that got replaced by the wrong arch, well spotted, will fix!10:43
=== MacSlow is now known as MacSlow|lunch
mardydpm, mzanetti: the accounts policy is limited as of now, but it won't be limited in the RTM11:07
mzanettiI see. thanks mardy11:07
rpadovanimzanetti, https://code.launchpad.net/~rpadovani/reminders-app/fixForReload/+merge/23077511:08
dpmok, thanks mardy. Do you have any ideas about why we're getting this errors when trying to migrate to the account in the click package? -> http://pastebin.ubuntu.com/8044372/11:09
dpm*these11:09
mardydpm: yes, the click hook should run that query for you; it's in my to-do list11:11
dpmmardy, oh, so this is not supposed to work yet?11:12
mardydpm: meanwhile, I think that you can workaround it locally by running "account-console list"11:12
mardydpm: try to close and re-open online accounts, that might fix it11:12
dpmmardy, it doesn't seem to: http://pastebin.ubuntu.com/8044459/11:14
dpmi.e. it doesn't list the evernote account and when I open the app, it takes me to an empty Accounts page11:14
nerochiarooSoMoN: Elleo: can you guys please have a look at this merge request when you have time ? https://code.launchpad.net/~phablet-team/gallery-app/gallery-app-restrict-pick-content-type/+merge/230776 it is missing tests, but if you could start verifying that the code is good and the functionality works that would help speed things up considerably11:17
mardydpm: it's weird that evernote is not there11:18
Elleonerochiaro: sure11:18
mardydpm: what's in ~/.local/share/accounts/providers/ ?11:19
dpmmardy, $ ls .local/share/accounts/providers/11:20
dpmcom.ubuntu.reminders_evernote-account-plugin.provider11:20
oSoMoNnerochiaro, I’ll put it on my list for today11:20
nerochiarooSoMoN: Elleo: and one unrelated question. do you guys usually build inside the same tree as your branch or in a separate directory ?11:21
oSoMoNnerochiaro, that depends on the project, some support both, some support only out-of-source builds11:21
Elleonerochiaro: I typically do out-of-tree builds for anything that supports it11:22
mardydpm: silly me, of course evernote is not listed by account-console until you create an account11:22
karnizbenjamin_: zsombi_: hi folks, I need someone from the sdk team to help me understand just a couple things. first, I wanted to ask if qtc can 'Run' the app from a folder different than build-telegram-app-Desktop-Default/.ubuntu-sdk-deploy, as well as which step/component is actually responsible for creating that directory (which seems to be missing in my casE)11:23
dpmmardy, so we're in a chicken and egg situation?11:23
nerochiaroElleo: oSoMoN: ok, and in case of out of tree builds how do you normally run tests ?11:24
mardydpm: no no, the problem is that an unconfined process needs to make a certain libaccounts call, in order to update the service list11:24
karnikalikiana: hey buddy o/ any news on that ~/.config/qt-project s/qt-project/actual.app.domain fix?11:24
nerochiaro(just trying to realign my workflow to current best practices)11:24
mardydpm: online-accounts-ui does that, but I cannot remember in which scenario -- I'm checking the source11:24
oSoMoNnerochiaro, what tests? unit or autopilot?11:25
nerochiarooSoMoN: both11:25
zbenjamin_karni: that directoy is created by the UbuntuSDK Click build step in the deploy configuration. Which is only present on a non desktop target11:25
oSoMoNnerochiaro, usually for unit tests, just run ctest or "make test" from the build dir11:25
zbenjamin_karni: the app is not running from there, what happens is a real click package is created and isntalled on the phone, then we execute the installed app and remove it when its closed11:25
zbenjamin_karni: there is no other way to run the app on  a device11:26
oSoMoNnerochiaro, for AP tests, running them out of source might not always work, that depends on how clever the setUp() function is, for webbrowser-app for example it works only either in source, or installed system-wide11:26
karnizbenjamin_: I see. I'm trying to run this on the desktop, and I get: QProcessPrivate::execChild() failed to chdir to /home/karni/src/canonical/telegram/build-telegram-app-Desktop-Default/.ubuntu-sdk-deploy11:26
karnizbenjamin_: is it possible to make it clear to qtc that I'm really trying to run on a desktop and not a device (thus, no need for .ubuntu-sdk-deply directory? )11:26
zbenjamin_karni: weird11:27
nerochiarooSoMoN: and to run these tests on the device what's the current recommended method ?11:27
nerochiaroAP11:27
mardydpm: try creating the evernote account from the system settings11:27
zbenjamin_karni: can i test that myself ?11:27
karnizbenjamin_: this might be due to modified CMakeLists.txt ? or qtc solely bases it's "setup" on project configuration? or are these two bound?11:27
zbenjamin_karni: basically the configuration is based on the Kit type you use. If its a Ubuntu Kit the deploy will happen11:28
oSoMoNnerochiaro, if it’s a click app look up an e-mail in your inbox sent by bfiller on May 28 titled "running autopilot tests for click"11:28
oSoMoNnerochiaro, for a pure debian package, just install the corresponding -autopilot debs, and use phablet-test-run11:28
nerochiarooSoMoN: cheers11:28
dpmmardy, I think I tried it earlier and it did not work, but I cannot confirm, as now the online accounts app won't load at all from ubuntu-system-settings :/11:30
kalikianakarni: failing on false negatives in ci I'm afraid (https://code.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/organizationDomain/+merge/230085)11:31
nerochiarooSoMoN: what's the process for getting this into trunk ? https://code.launchpad.net/~phablet-team/gallery-app/gallery-app-remove-qmlproject/+merge/230665 (or to check that it's in the pipeline to get into trunk at some point)11:34
rpadovanihey mzanetti, there is a simpe way to use void setName(const QString &name); (notebook.h) from QML?11:38
* mzanetti looks11:39
dpmmardy, so from the system settings I get evernote as an available account, but when I try to create it, it also lands me on a blank page11:39
nik90popey: do you happen to know when the geoname-lookup server gets updated?11:40
popeynik90: i filed an RT today, and now the merge landed, I can push it, which I will with IS11:40
popeyno ETA yet11:40
nik90ack11:41
nik90mzanetti: hey, can you approve https://code.launchpad.net/~nik90/ubuntu-clock-app/json-search-backend/+merge/230338. You already did a code review of it.11:42
nik90mzanetti: I will tackle the issue of the QNetworkAcessManager in another MP.11:42
mzanettinik90: yeah... as I said.... I wouldn't do much for the QNAM at the moment11:43
mzanettinik90: it's probably not worth spending the efforts as it doesn't make a difference on the Ubuntu platform for now11:43
mzanettinik90: and I'm not sure it ever will11:43
nik90mzanetti: yes but I will need your help to tackle that in the near future since I will need to do another json request in another plugin for the user location11:43
nik90mzanetti: which will end up with two QNAM in the same app.11:44
mzanettinik90: yeah... just create a new QNAM in that plugin too for now11:44
nik90mzanetti: ok11:44
mzanettinik90: again, it doesn't matter on Ubuntu (would matter on MeeGo or Sailfish though)11:44
nik90got it11:44
oSoMoNnerochiaro, as usual, a landing request needs to be filed with this MR. Given the nature of this MR, it would be worth grouping it with other non-conflicting small MRs11:45
mzanettinik90: approved11:45
nerochiarooSoMoN: ok, i've never filed one so far, that's why I am asking about the process11:45
nik90mzanetti: thnx11:46
jdstranddpm: fyi, the accounts policy group should be a 'common' policy group very soon. dbarth and mardy have been working to make that happen and I think code for it may hit a silo soon12:02
=== MacSlow|lunch is now known as MacSlow
=== chihchun is now known as chihchun_afk
=== _salem is now known as salem_
nerochiarorenatu: artmello: oSoMoN: Elleo: do we know of any app that is already doing AP integration tests with the content hub ? I would like to add some to the gallery but if I can start with some examples that would make things a lot quicker12:34
renatunerochiaro, I do not know, I am sure that address-book does not have that yet12:34
Elleonerochiaro: have a chat with ken when he gets on, he's just created a test peer that can be used for autopilot tests12:35
Elleonerochiaro: as at the moment autopilot doesn't handle cases where you need to switch between apps very well, so the test peer can be configured to just instantly return various types of content or to accept content without any user interaction12:35
nerochiaroElleo: do you have an example of that ?12:36
nik90dpm: I reviewed your MP and left some comments. It is all good just for some few corrections that need to be made.12:36
nerochiaroElleo: what i'd like to do is to have content hub request gallery some stuff, and verify that gallery displays the right UI and filters the data properly12:37
Elleonerochiaro: not yet; it only got merged this week, so there's nothing using it yet; kenvandine should be able to tell you how it works though12:37
dpmnik90, yeah, I saw it thanks, I've been looking at reminders today, and I've not yet had the chance to address those, but I will12:37
nik90dpm: np12:37
nerochiaroElleo: ok, I'll ask ken when he comes in later12:37
nerochiaroElleo: thanks12:37
Elleonerochiaro: also, rhuddie is planning on using the new test peer to write some AP tests for various apps, so you might want to co-ordinate with him as it might already be something he's working on12:37
dholbachdpm, can you land https://code.launchpad.net/~dholbach/ubuntu-sdk-tutorials/updates/+merge/230791 maybe?12:44
dholbachor mhall11912:45
dholbachlooks like I don't have the necessary powers12:45
dholbachand maybe add me to https://launchpad.net/~ubuntu-sdk-tutorials-dev as well :)12:45
popeyballoons: I have rebuilt the clicks for _all_ the core apps, could you please upload en-masse?13:05
balloonspopey, sounds like fun :-)13:06
mhall119dholbach: want me to top-approve that MP?13:07
mihirballoons: Good Morning :)13:09
balloonsmihir, morning13:09
dpmdholbach, I've added you to the team13:09
mihirballoons: because of new change in Yearview we have break in AP , so we13:09
mihirhave to fix that.13:09
dholbachmhall119, yeah, that'd be nice13:10
balloonsI finished the tweaks I wanted to https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/1188819/+merge/23055413:10
dholbachh no, dpm did13:10
mhall119dholbach: yeah, I just saw that13:11
dholbachthanks guys!13:11
mihirballoons: yes i saw that balloons i am yet to understand that and then i'll fix in office right now will do that by tonight, my long weekend starts from today :D13:12
dholbachI just went through the ubuntu-developerportal bugs and that was one of them13:12
=== chihchun_afk is now known as chihchun
mihirzsombi_: ping13:14
zsombi_mihir: pong13:14
mihirzsombi_: could you join #ubuntu-touch-meeting for a while ?13:14
dpmballoons, when you've got a minute, would you mind looking at https://code.launchpad.net/~ubuntu-filemanager-dev/ubuntu-filemanager-app/require-screenlock-password/+merge/230058 again? It seems there are some Jenkins failures and the jobs need to be retriggered13:15
balloonshmm.. I don't see my push from yesterday.. interesting13:16
dpmballoons, perhaps you pushed to another branch?13:17
balloonswho knows13:17
balloonspopey, I think everything should have a new version uploaded now13:24
popeythanks balloons13:30
dpmmzanetti, joining the reminders hangout?13:30
=== gatox is now known as gatox_lunch
=== charles_ is now known as charles
=== chihchun is now known as chihchun_afk
nik90bzoltan: I think you might like this -> https://www.youtube.com/watch?v=PBpCykbmqs8&feature=youtu.be14:32
dpmballoons, so the tests passed for file manager \o/ - are you happy with them, and comfortable with landing it as it is?14:54
nik90balloons: can you join the clock app meeting in another hour and a half? I would like to discuss together what it would take to make the replacement.15:02
nik90and then we can take some workitems and get them done15:03
balloonsnik90, sure can do15:04
balloonshangout?15:04
nik90balloons: not sure yet, normally I ask popey at the start of the meeting. Either way I will keep you informed15:04
balloonsdpm, I'm wanting to potentially change some things, but I would not hold the mp for it, as I know it's important.15:05
balloonsdpm, and I'd much prefer doing an mp with the changes against trunk, instead of making this one longer15:05
=== gatox_lunch is now known as gatox
popeydaker: you might want to update your airbnb app with their new logo16:11
popeydaker: just installed it on my #189 mako and it refuses to start here.. just get the loading screen16:11
dakerpopey: ok16:12
bzoltannik90:  beautiful background you have :)16:17
nik90bzoltan: :)16:17
bzoltannik90:  I am at 2 minutes, but I love that video already! You rock!16:19
nik90bzoltan: I wanted to do it earlier but was waiting for the UI workflow to stabilize. Also some devs kept asking how to run their apps after the recent update.16:19
dakerpopey: https://myapps.developer.ubuntu.com/dev/click-apps/246/feedback/16:36
rpadovanidpm, i18n.tr doesn't work with single quote? I mean, i18n.tr('Translate this') doesn't work?16:40
popeydaker: looks like dholbach beat me to it ☻16:40
brendandrpadovani, i ran into that one16:40
dpmrpadovani, I've no idea, I would have thought it should. What's not working?16:40
dholbachpopey, sorry16:40
popey16:40
dholbach:)16:40
rpadovanidpm, I'm looking at this MR... it's odd16:41
rpadovanihttps://code.launchpad.net/~vrruiz/reminders-app/i18n-strings/+merge/23083816:41
rpadovaniI mean, makes sense, but why?16:41
dakerpopey: let's if it works, i know it will show the android ads16:41
dpmrpadovani, ah, it might be that the string is not extracted by xgettext because it interprets the .qml file as a .c or .cpp file and expects strings to be "16:43
dpmxgettext has no direct support for qml16:43
rpadovanidpm, thanks for the explanation!16:43
dpmnp :)16:46
dakerpopey: the "Travel" section on the store still shows the old one16:59
ahayzenDoes the apparmor_parser process take *forever* for anyone else when 'Allowing autopilot to play well with apparmor' or on a first boot after upgrading an image? or is it just my device?17:05
balloonsahayzen, a minute yes17:05
ahayzen...or two or three...17:06
balloonsahayzen, that sounds a bit long17:06
balloonsbut :-) it has to recompile everything17:06
ahayzenoh god :/ guess it is proportional to num of apps installed as well then?17:06
balloonsahayzen, lolololololol..17:07
balloonsdid you go crazy with apps?17:07
ahayzendone!17:07
ahayzengod probably took like 4-5 mins17:07
ahayzenballoons, no not many probably +10 to standard17:07
ahayzenballoons, also do u have time today to help me with the music-app ap refactor? ...17:08
popeydaker: cached maybe?17:08
ahayzenballoons, i would really appreciate it if u could help me do the first 1/2 test transitions to make sure i set it up correctly etc...then i can do the rest17:09
balloonsahayzen, I will in a bit.. trying to finish something up17:09
popeyahayzen: takes 20 mins on my nexus 717:09
ahayzenpopey, jesus!17:09
popeyahayzen: but I have 380 clicks installed17:09
popeyindeed!17:09
ahayzenpopey, are there any plans to improve this?17:09
balloonsahayzen, I can certainly answer questions as you go.. feel free to ping17:09
balloonsahayzen, they did some work to improve the speed, lol17:09
ahayzenballoons, thanks i'll probably start after dinner so in like 1-2hrs17:09
balloonsbut honestly, it's a dev thing, so17:09
ahayzenballoons, but it runs after u upgrade?17:09
popeyahayzen: it's known ☻17:09
ahayzenballoons, and it sits at 'Google' for *ages* i have been thinking the upgrades have been failing recently because of it17:10
ahayzenif the shell in at that time u can see like ~4 of them all using 100% cpu17:10
ahayzenpopey, is there a bug to track?17:11
nik90davmor2: so to fill you in, we will be doing the replacement once we have the next promoted image since the clock reboot needs the latest SDK which is not available in the previous promotoed image (179)17:12
popeyahayzen: the understated bug 135059817:12
nik90davmor2: so definitely by monday17:12
ubot5bug 1350598 in apparmor (Ubuntu) "apparmor_parser takes a long time" [Undecided,Confirmed] https://launchpad.net/bugs/135059817:12
ahayzenpopey, that title is perfect :)17:12
ahayzenah 'This will not normally affect consumers because neither the kernel nor the policy will change during the system image update process.'17:13
balloons^^ as  I said, 'dev' thing :-)17:13
davmor2nik90: hahahaha you think they'll be a promoted image Monday ah bless ;)17:13
balloonsdavmor2,  . . .17:14
ahayzenyeah but i guess if someone moved their phone from 14.10 -> 15.04 then they would experience it17:14
ahayzenbut only once so not to bad :)17:14
nik90davmor2: you are not instilling much confidence in me :P17:14
davmor2nik90: blame popey he keeps finding bugs17:14
* ahayzen wants the new media-hub in a promoted image17:14
nik90davmor2: well me and popey are in the same page, so he will understand what needs to be done :D17:16
popeyyeah, don't listen to him ☻17:19
dakerpopey: i think yes17:21
nik90:P17:22
nik90popey: in image 189 can you see if the indicator datetime shows you any thing? For me it blank..no alarms, no calendar events..really strange17:25
davmor2nik90: see now you're finding bugs how do we stand a chance17:25
nik90davmor2: well this bug is so alarming that I cannot ignore it :P17:26
nik90davmor2: how does it look for you?17:26
davmor2nik90: well quite obviously it isn't alarming and that's the problem :P17:26
nik90davmor2: lol.17:26
zsombinik90: howdy sir17:27
nik90I think for me the datetime service crashes because the time it shows is also incorrect17:27
davmor2nik90: I see the alram notification, but I see nothing in the indicator17:27
nik90zsombi: hello :)17:27
nik90davmor2: yup me too17:27
nik90davmor2: time will go out of sync as well17:27
zsombinik90: will you have time tomorrow to check the alarm-switch thing?17:27
nik90zsombi: I did and I got stuck :D17:27
nik90zsombi: tell me you will swoop in and save the day again17:28
zsombinik90: I know, I saw your post, just had no time to check it17:28
zsombinik90: ;)17:28
davmor2I blame that nik90 for this demanding changes for the alarms ;)17:28
zsombinik90: do you have some sample?17:28
nik90zsombi: yeah 1 sec17:28
zsombinik90: I know you had, just lost the pastebin17:28
nik90zsombi: http://paste.ubuntu.com/8027116/17:28
zsombinik90: was this pastebing a delegate of a listview?17:31
nik90zsombi: yup17:31
zsombinik90: do you eventually have a broader code pastebin?17:31
nik90zsombi: I had a local branch I was working on. I could push that for you17:31
zsombinik90: pls, txh17:32
zsombithx17:32
nik90zsombi: lp:~nik90/ubuntu-clock-app/implement-alarm-status17:35
zsombinik90: thx! I'll try to see whether we can get somewhere with this17:35
nik90zsombi: https://bazaar.launchpad.net/~nik90/ubuntu-clock-app/implement-alarm-status/view/head:/app/alarm/AlarmDelegate.qml17:35
zsombinik90: have you tested with memory manager as well?17:36
nik90zsombi: I will be online tomorrow early. Ping me and I will join you with it.17:36
nik90zsombi: no I haven't tested with memory manager.,.only EDS17:36
zsombinik90: ok, let's see17:36
zsombinik90: I think I did back few months ago, and it had the same issue17:36
nik90zsombi: the code I wrote was taken from the sdk sample code.17:37
zsombinik90: even worst, as that is much faster, so it took the switch back almost instantly17:37
zsombinik90: I know... sorry for that :/ it doesn't work properly, and it's because of th eswitch17:37
zsombinik90: we had a branch with teh CheckBox - having teh same issue, I have to check that, perhaps we can do some API change to come over this issue17:38
nik90zsombi: no worries, I have a potential other issue related to this, so hopefully fixing this will help with that17:38
zsombinik90: ok, let's hope17:38
nik90zsombi: how do you read an alarm, update it and wait for the alarm operation to be completed before doing anything.17:39
nik90zsombi: because when creating an alarm I create the Alarm{} object. Where i monitor the status signal.17:39
nik90zsombi: but while reading alarms I do, var tempAlarm = alarmModel.get(index)17:39
nik90zsombi: hence that confuses things for me17:39
zsombinik90: yeah, agree, it's a but confusing, perhaps the model should only return roles, not objects...17:40
nik90yeah I think so17:40
zsombinik90: the Alarm{} is a helper to create alarms, you can use it to create several alarms if you clear the alarm once got saved17:40
zsombinik90: then alarmModel.get(index) returns an alarm from teh model's given index. The index has no relation with the Alarm itself, so that's why is confusing17:41
zsombinik90: there's a ccokie in teh Alarm internal which could be used as ID, however that is platform (adaptation) specific17:42
nik90yeah17:42
zsombinik90: and that cookie is really bad one..17:43
nik90zsombi: I am hoping something can be done within the clock app side of things since the SDK takes too long to update, but let's see what happens17:43
zsombinik90: there is some gles hassle in the CI than needs to be sorted so we can land...17:44
nik90zsombi: yes I was tracking the sdk update for the past 2-3 days since I desperately needed those 2 critical bug fixes for alarms.17:44
zsombinik90: like the two bugs around the alarm sound and the clear() crash are fixed, landed in staging, but the silo is dying all the time17:45
nik90yup17:45
nik90zsombi: hey atleast I can now make install any SDK branch without blockage due to EDS :)17:46
zsombinik90: ;)17:46
=== nerochiaro is now known as nerochiaro_dinne
popeynik90: yes, i have stuff in mine17:59
nik90popey: does the time in the panel update for you?18:00
nik90popey: and are you runnign 189?18:00
nik90popey: clean wipe or just update?18:00
nik90sry too many questions :D18:00
popeynik90: i lie, wrong phone, no i see no notifications in upcoming18:00
popeynik90: and no, time is 17:08 (it's 19:00)18:01
popeyso indicator-datetime needs a bug i think?18:01
nik90popey: yup...I am discussing with charles atm.18:01
popeyso i see, lemme know if you need anything from me18:01
nik90well I wanted to ask you to test my custom-alarm-sound branch tonite. but we can't do it anymore18:02
nik90since I doubt alarms will ring18:02
mhall119nik90: this is a really great video18:58
mhall119nik90: can we embed it on developer.u.c?19:00
nik90mhall119: do you want me to edit it a bit? like adding titles and so on?19:10
mhall119nik90: that would be great if you have time, but I'm happy to add it as-is, it's a huge help19:12
popeywhat video is this?19:13
nik90popey: https://www.youtube.com/watch?v=PBpCykbmqs8&feature=youtu.be19:13
mhall119popey: it's brilliant19:14
popeyooh19:16
mhall119popey: nik90: http://developer.ubuntu.com/start/ubuntu-sdk/using-device-kits/19:17
nik90mhall119: awesome thnx19:18
mhall119no man, thank you, this is great stuff19:19
nik90:)19:21
nik90popey: hey I pushed to test click for https://code.launchpad.net/~nik90/ubuntu-clock-app/add-custom-sound-backend/+merge/225812 in dropbox19:30
nik90popey: can you test on 189, it works for me. What you need to test is check when editing an alarm if you see the ringtone you chose.19:30
nik90popey: and ofc that the alarm rings with the ringtone you chose19:31
mihirballoons: i have merged the branch , thanks for your help19:39
balloonsmihir, ohh excellent. you arre welcome19:39
mihirballoons: here is MR link , https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/1188819/+merge/23055419:41
nik90balloons: btw the MR with the qml tests is at https://code.launchpad.net/~nik90/ubuntu-clock-app/add-first-qmltest/+merge/230702.19:43
nik90balloons: I will update the readme files with the necessary info to run the qml test as elopio suggested.19:44
popeynik90: sure19:45
nik90popey: btw don't look at the datetime indicator time before setting the alarm :)19:46
popeyuh, okay19:46
balloonsnik90, awesome19:46
mihirnik90: do we need any PPA for UbuntuComponents on utopic ?19:50
popeynik90: it doesn't play the sound when I choose one, I mean, when selecting the alarm sound to be used  - i would expect it to19:51
nik90mihir: no19:51
popeyotherwise I dont know what noise will come out when the alarm goes off19:51
nik90popey: +1. I have a bug about that. Unfortunately I need design spec for that.19:51
mihirnik90: i am just trying to run Gallery app  , and getting this in console , file:///home/mihir/Documents/apps/gallery/ubuntu-ui-toolkit/examples/ubuntu-ui-toolkit-gallery/ubuntu-ui-toolkit-gallery.qml:18 module "Ubuntu.Components" plugin "UbuntuComponents" not found19:51
nik90popey: for instance where to show the stop button and so on19:51
nik90mihir: how did you run it? Through Qtcreator?19:52
mihirnik90: yup19:52
popeyi wouldn't show a stop button19:52
popeyother phones dont19:52
nik90popey: our system settings app does19:52
popeyjust press the button, noise comes out until you choose a different one19:52
nik90that can be implemented19:55
popeynik90: tested...20:00
popeynot sure it used the tone I chose20:00
popeybut it wasnt those dammed harps20:00
nik90ah ok. Yeah I had to use the sytem settings app to hear the ringtone and then chose it in the clock app :)20:00
nik90But yes we have to add previewing system sounds in the clock app20:01
mihirnik90: by saying gallery,  I meant ubuntu-ui-toolkit application20:02
nik90mihir: yeah I know..I can't understand why since it works for me on my utopic vm. I choose gallery from the Tools->Ubuntu->ShowCase Gallery20:03
nik90mihir: on my utopic vm I have no PPAs installed20:04
mihirnik90: it did work now , thanks for help20:19
mihirballoons: ping20:19
mihirballoons:  i still see some errors in AP , is it because of some assert issues , http://91.189.93.70:8080/job/generic-mediumtests-utopic/1531/testReport/junit/calendar_app.tests.test_new_event/NewEventTestCase/test_delete_event_must_remove_it_from_day_view_with_mouse_/ ?20:20
nik90mihir: ah, awesome20:20
mihiri did run test cases on my machine and it did passed flawlessley20:20
popeynik90: yes, set a few alarms with different sounds and they work20:21
nik90popey: do you want me to add support for previewing sounds in this MP or a new one?20:21
popeyseparately20:21
nik90ok20:22
nik90on it20:22
nik90balloons: mind doing a code review of  https://code.launchpad.net/~nik90/ubuntu-clock-app/add-custom-sound-backend/+merge/225812 when you are free. popey has tested green on image 189.20:23
balloonsnik90, do you underscore in qml for a specific reason? aka _soundModel20:25
nik90balloons: nope, I just use it similar to in python indicating it is a internal variable20:26
nik90balloons: I am going to add a bit of comments to the code in a bit20:27
=== salem_ is now known as _salem
mihirballoons: it seems a same problem , it is not able to find the event after deleation20:37
balloonsnik90, kk20:40
* balloons watches video20:41
balloonsI think you are seeing an issue in trunk though mihir20:42
balloonsahh yes indeed20:42
mihirballoons: didn't get you :|20:42
balloonsmihir, https://bugs.launchpad.net/ubuntu-calendar-app/+bug/135060520:43
ubot5Ubuntu bug 1350605 in Ubuntu Calendar App "Existing Event can be "Hidden" when adding new events" [High,Triaged]20:43
balloonsbasically there is a display issue going on with events, and that's what we are partially seeing. The other issue is timing.. the page hasn't loaded yet and it tries to grab the object but it doesn't exist20:44
balloonsyour desktop is fast, so no weird issues and it runs fine20:44
mihirballoons: ohhh I see, I was trying to fail that test case on machine and couldn't20:45
mihirballoons: as we have added bug workaround in AP it should refresh the view righ t?20:46
balloonsmihir, yes and you can see it dies.. but right after refreshing it checks for the object and fails.. you can just see the spinner on the dayview pag20:48
balloonswhen the video ends. the page is still loading20:48
mihirballoons: hmm I see that.20:49
ahayzenballoons, let me know when ur free21:09
nik90balloons: it seems there is a flaw in our autopilot test for alarms. It doesn't test the value of the alarm sound. As a result the test doesn't fail if the alarm sound cannot be chosen22:07
balloonsnik90, the sound selection is new thought correct?22:10
nik90balloons: no it was there before this MP22:10
balloonsahayzen, sorry I've gotten stuck on this mp for some time22:10
balloonsnik90, ahh, we should have included it then..22:10
ahayzenballoons, no worries i'm just playing about attempting things ;)22:10
nik90balloons: the issue there is that the alarm sound chosen is not shown in the UI in the Alarm List page22:10
nik90balloons: so how do we tests its value if we cannot retrieve it via the UI?22:10
nik90balloons: unit tests?22:10
nik90balloons: are you stuck in my MP?22:11
balloonsnik90, lp wouldn't let me comment on your mp, lol22:11
nik90weird22:11
balloonsnik90, if the property is set we can read the value if needed22:17
nik90balloons: the property seems to be set in the EditAlarmPage where we don't do any verfication yet. We do the verification once the alarm is saved in the main Alarm Page where the alarm sound property cannot be read.22:18
nik90balloons: I will add a verify function in the EditAlarmPage for this22:18
nik90balloons: ah looks like carla did add a verify function in the EditAlarmPage...22:20
* nik90 investigates why that didnt trigger the failure22:20
nik90balloons: can you look at https://bazaar.launchpad.net/~nik90/ubuntu-clock-app/add-custom-sound-backend/view/head:/tests/autopilot/ubuntu_clock_app/emulators.py#L22522:23
nik90balloons: that's the verify function which wait for the alarmSound text to match the given sound name22:23
nik90balloons: why doesn't it fail?22:23
nik90balloons: in the emulator.py file, how do you check if two strings are the same and if not it fails the test22:28
nik90balloons: I can't use assert statements in the emulator.py file22:28
balloonsnik90, need just a bit longer22:35
nik90sure22:35
nik90balloons: I fixed it22:45
nik90balloons: AP now fails if incorrect alarm sound is chosen22:46
ahayzenballoons, nik90, any reason/preference why the reminders app has a BaseTestCaseWithTempHome and then a RemindersAppTestCase but the clock app just has ClockAppTestCase ?22:47
nik90ahayzen: you got a link to BaseTestCaseWithTempHome?22:48
ahayzennik90, http://bazaar.launchpad.net/~reminders-app-dev/reminders-app/trunk/view/head:/tests/autopilot/reminders/tests/__init__.py22:48
ahayzennik90, note the RemindersAppTestCase at the bottom which inherits22:48
nik90no idea22:50
ahayzenhah22:50
ahayzennik90, so should i just do MusicAppTestCase ?22:50
Akiva-Thinkpadsoooo; before ubuntu ships the rtm22:50
nik90well frankly I never wrote the __init__.py fiel for either of the clock apps. So not the right person to ask22:50
Akiva-Thinkpadis there any apps that need shoring up?22:51
Akiva-ThinkpadI have some free time i can throw their way22:51
ahayzennik90, hah no worries then i'll continue playing about :)22:51
nik90Akiva-Thinkpad: do you have a device to test apps?22:51
ahayzenAkiva-Thinkpad, are you interested in any particular area? coding/design/testing?22:52
Akiva-Thinkpadnik90, unfortunately not.22:52
nik90and that ^^22:52
Akiva-Thinkpadahayzen, core apps I have experience with, but I would be happy doing something new too.22:53
* Akiva-Thinkpad still needs to finish my own app :S -- First though he needs a custom keyboard layout22:53
nik90Akiva-Thinkpad: your best bet would be core apps since the other system apps get enough attention with full time devs working on them :)22:54
Akiva-Thinkpadnik90, any one in particular that needs more work than the others?22:54
Akiva-Thinkpadnik90, ahayzen also; has the design for the tablet mode of Unity8 been planned yet?22:55
nik90Akiva-Thinkpad: I personally think the order is calendar app -> music app -> clock app.22:55
ahayzenclock and music ;) hehe ... no vested interest at all there right nik90 ?22:55
nik90Akiva-Thinkpad: calendar requires the most22:55
ahayzenyeah agreed calendar22:55
Akiva-Thinkpadthanks22:56
Akiva-Thinkpadwhats the lp?22:56
Akiva-Thinkpadcalender-app?22:56
nik90Akiva-Thinkpad: https://launchpad.net/ubuntu-calculator-app22:56
nik90Akiva-Thinkpad: lp:ubuntu-calculator-app22:56
Akiva-Thinkpado_O22:57
Akiva-Thinkpadcalculator app?22:57
Akiva-Thinkpadfor the calender?22:57
Akiva-Thinkpadyou are pulling my leg, right?22:57
nik90Akiva-Thinkpad: crap, sry22:57
Akiva-Thinkpadha ha22:57
nik90Akiva-Thinkpad: https://launchpad.net/ubuntu-calendar-app22:57
Akiva-Thinkpadokay thanks22:57
* Akiva-Thinkpad branches22:58
ahayzento many apps starting with C lol22:58
nik90Akiva-Thinkpad: its 01:00 AM here, a bit sleepy :P22:58
Akiva-Thinkpadheh22:58
nik90ahayzen: yup22:58
nik90Clock, Calendar, Calculator....CMusic...anything else :D22:58
ahayzenCMusic \o/22:58
Akiva-Thinkpadha22:59
ahayzennik90, how are you doing with the clock-reboot?23:00
nik90ahayzen: good..hoping for a replacement after the next promoted image since we need the latest sdk bits.23:00
nik90ahayzen: we are actually a position to do the replacement but have to wait for ^^23:00
ahayzenOMG \o/ well done :)23:01
nik90thnx :) awfully tired but finally finally there ;)23:01
ahayzenhah ...i've been playing with the click from the store and love the transitions and animations :)23:01
nik90ahayzen: how's your bottom edge implementation going?23:02
nik90ahayzen: ETA on its landing?23:02
ahayzennik90, ETA = when victor says it is ready23:02
nik90ahayzen: I think https://launchpad.net/ubuntu-clock-app/+milestone/rtm says part of the story23:02
nik90ahayzen: lol23:02
ahayzennik90, he has done a first review and i've fixed that so will hopefully get another tonight :) https://code.launchpad.net/~andrew-hayzen/music-app/bottom-edge-implementation-001/+merge/23030323:03
nik90ahayzen: oh cool23:03
ahayzennik90, thats nearly as big as ur list at ... no thats getting old ;)23:03
nik90ahayzen: i am gonna kill you :P23:04
ahayzenhehe23:04
ahayzennik90, but seriously awesome work :)23:04
nik90thnx mate23:05
ahayzennik90, just wondering...as me and victor were discussing the other day....how are u going to handle rtm...are u gonna have a separate branch for rtm then a devel branch? or have u not decided yet?23:06
nik90ahayzen: separate branches I think23:06
ahayzennik90, so then 2 clicks in the store?23:07
nik90ahayzen: I already have a separate branch for reboot, so it ain't that difficult23:07
nik90ahayzen: hmm clicks..23:07
ahayzennik90, hah but at one point i didn't think u could have two click with the same namespace but with different versions23:07
nik90yeah :/23:08
ahayzennik90, whatever we do i feel the coreapps need to be consistent otherwise it is gonna be confusing going between each of them23:08
nik90I personally think that multiple branches is the way for the following reasons,23:08
nik901. Say you migrate to ms3 and framework 15.04. What happens to the people using framework 14.10 and ms2? We would be abandoning them.23:09
ahayzenthat is the biggest one ^^23:09
nik90this is not acceptable for rtm since it is a public device after all23:09
nik90I remember beuno saying that they would perhaps add support for multiple click versions in the store. But no ETA on that.23:10
nik90if we raise this as a critical bug, then we might be get that feature landed23:10
balloonsnik90, ahayzen ok I'm here.. perhaps briefly, but here23:11
nik90I am sure once 15.04 development starts, they will bump framework version and if we want to use that while at the same time supporting the old framework, we need multiple branches (and multiple clicks)23:11
ahayzennik90, yeah maybe i think we need to all have a discussion with dpm and popey about it23:11
nik90ahayzen: +100023:11
ahayzenballoons, this is a non-working (i know why i think)...diff of the music-app autopilto tests...could u tell me if i'm heading in the right direction or not? http://pastebin.ubuntu.com/8049152/23:12
ahayzenhah RemindersAppTestCase ... how did that get there...23:12
ahayzenballoons, i think i need to move the stuff from MusicApp into MainView?23:13
ahayzenballoons, thats what the clock app has23:13
balloonsahayzen, yes, looks good. Keep methods useful.. no 'get_first_track', instead 'get_track' and pass a number to get23:16
balloonsso try and avoid the get_X mantra and you'll be fine23:16
ahayzenballoons, ah yes thanks23:16
ahayzenballoons, i think i understand how it works...23:17
ahayzenballoons, like with a class for the objects...eg having a class for Page etc23:17
ahayzenballoons, but haven't got it running yet still linking things up :)23:17
nik90ahayzen: also keep your tests as short as possible. All the hard work should be done in the emulators.py (or helpers.py).23:17
nik90ahayzen: look at the old clock app tests for example.23:18
ahayzennik90, thats the plan .... so run_test() ? hehe23:18
nik90lol23:18
ahayzenballoons, should i be transitioning to py3 as well?23:19
ahayzenballoons, or are we py3 already...23:19
nik90balloons: jenkins is cruel...it makes me wait for long before telling me that I have pep8 errors and fails the integration :/23:19
nik90long a.k.a 4 mins23:20
ahayzennik90, run pep8/pyflakes locally ;)23:20
nik90ahayzen: I changed 3 lines of code at most...didnt occur I might make a pep8 error there :P23:21
ahayzennik90, hah u'd think23:21
balloonsnik90, I get burned all the time.. wlecome to the club23:21
balloonsahayzen, yes py3 all the way23:21
ahayzenballoons, cool i'll transition over...am i best just deleting all the old code?23:22
ahayzenballoons, then slowly building it back up23:22
ahayzenballoons, otherwise i have to think of alternate names for 'emulator' and 'test_music'23:22
balloonsahayzen, you can take either approach. For music, I would do a page at a time.. transition a page, and leave the rest23:22
ahayzenballoons, most of our tests use multiple pages though :/23:23
balloonscommits will be easier to read as well.. it won't be a giant replacement23:23
ahayzenballoons, which makes it tricky23:23
ahayzenballoons, but but but i like my 2.5K+ diffs23:23
balloonstry and take small bites :-)23:23
* balloons rolls eyes23:23
ahayzenballoons, ok but i would still end up with one large mp at the end no?23:24
balloonsahayzen, no I would propose as you go.. As I said, convert a page, and then use those new methods in the tests for that page23:24
balloonssplit out the tests into new files corresponding to the page23:24
balloonseach one should be it's own mp23:24
balloonsthat's the theory..23:24
ahayzenballoons, and then merge that into trunk....hmm ok so i guess i would need a separate emulators.py ? and __init__.py ?23:25
balloonsreality might be harsher, but see if you can work along those lines23:25
balloonsahayzen, nope you shouldn't need a seperate23:25
ahayzenballoons, hmmm ok i'll try23:25
balloonsahayzen, yep, don't fret so much on what the mp will be.. try and bite off small chunks as I said.. it should come together23:26
ahayzenballoons, ok i'll  try and convert 1-3 tests over the next day and mp that ...then let u have a look :)23:26
balloonsahayzen, sound sgreat23:27
ahayzen:)23:27
ahayzenballoons, thanks for ur help :)23:28
Akiva-Thinkpadahayzen, sorry to bug you; but calender app uses a qml project file; is this a new type of project file? I am used to seeing cmakelists and qt .pro files23:29
Akiva-Thinkpadjust curious23:29
ahayzenAkiva-Thinkpad, we use cmakelists in music-app but yeah qml projects can be used i don't know how new they are?23:30
ahayzenAkiva-Thinkpad, we did use qml project before :) nothing wrong with them23:30
Akiva-Thinkpadahayzen, just curious now which of the templates use this...23:31
ahayzenerrrr Simple and Tabbed did at one point IIRC ?23:31
Akiva-Thinkpadthe template I used for my project in the new projects wizard, used cmakelists, and its annoying to configure.23:31
Akiva-Thinkpadahayzen, yep right on the money23:31
ahayzenyey23:32
ahayzenAkiva-Thinkpad, what issues you having with cmake?23:32
* Akiva-Thinkpad finds an excuse not to learn cmake!!23:32
Akiva-Thinkpadhooray!23:32
Akiva-Thinkpadahayzen, icon setting23:32
Akiva-ThinkpadHell I went so far to put an icon in every folder, and it still wouldn't show up.23:32
Akiva-Thinkpadpah23:32
ahayzenAkiva-Thinkpad, the only issue i have is that for ubuntu-sdk to work with autocomplete i have to open a qml project for qtcreator to index it ?!23:32
Akiva-Thinkpadhrm23:33
ahayzenAkiva-Thinkpad, i spot this line in music set(ICON_FILE images/music-app@30.png)23:33
Akiva-Thinkpadthanks I'll take a look at it.23:33
ahayzenAkiva-Thinkpad, this is our cmake http://bazaar.launchpad.net/~music-app-dev/music-app/trunk/view/head:/CMakeLists.txt23:34
ahayzenAkiva-Thinkpad, other than ^^ issue i stated i have no issues with it23:34
ahayzenAkiva-Thinkpad, it just doesn't feel as integrated with qtcreator23:34
Akiva-Thinkpad^23:34
ahayzenAkiva-Thinkpad, eg with qml project u can right click and do 'Add new file'23:34
Akiva-Thinkpadyes!23:35
Akiva-Thinkpadgeez; that is such an annoyance23:35
ahayzenAkiva-Thinkpad, yeah :/23:35
ahayzenAkiva-Thinkpad, wish it was better23:35
Akiva-Thinkpadcould be worse23:35
ahayzenhah23:35
ahayzenAkiva-Thinkpad, it was referred to as cbreak while we were transitioning ;)23:36
Akiva-Thinkpadha ha23:36
Akiva-Thinkpadlol23:36
* Akiva-Thinkpad gets to debugging the calender23:36
ahayzen\o/23:37
nik90balloons: I am of to sleep. Can we talk about enabling the unit tests in jenkins tomorrow. Let me know if you find something about that.23:53
nik90balloons: I finished the custom alarm sound branch. Any further improvements, I will reserve to another MP.23:53
popey\o/ sleep23:54
balloonsnik90, good night. We can chat tomorrow. I pinged fginther about it earlier, I'll track him down tomorrow23:55
nik90nite :)23:56

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