| === chriadam|away is now known as chriadam | ||
| === chihchun is now known as chihchun_afk | ||
| === _salem is now known as salem_ | ||
| === chihchun_afk is now known as chihchun | ||
| === salem_ is now known as _salem | ||
| === chihchun is now known as chihchun_afk | ||
| === chihchun_afk is now known as chihchun | ||
| === sergiusens_ is now known as sergiusens | ||
| dholbach | good morning | 07:01 |
|---|---|---|
| JoeyChan | morning :) | 07:06 |
| mihir | hi dpm | 07:37 |
| mihir | dpm: I am working on bug#1342275 for navigate user to account screen, | 07:40 |
| mihir | dpm: are you aware about the provider id for Google Calendar ? | 07:41 |
| dpm | hi mihir | 07:45 |
| dpm | bug 1342275 | 07:45 |
| ubot5 | bug 1342275 in Ubuntu Calendar App "Provide a quick way to setup a sync account" [Medium,Confirmed] https://launchpad.net/bugs/1342275 | 07:46 |
| mihir | and here is MR , https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/1342275/+merge/230386 | 07:46 |
| mihir | dpm: when i use evernote as providerId ,it does work but couldn't find documentation for list of providerIds. | 07:46 |
| justCarakas | is there any progress on https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1309042 | 07:48 |
| ubot5 | Ubuntu bug 1309042 in Ubuntu Calendar App "Calendar sync only syncs the default calendar" [Wishlist,Confirmed] | 07:48 |
| dpm | mihir, the list of provider ids is in /usr/share/accounts/providers/ | 07:48 |
| dpm | from the .provider file, the provider id is "google" | 07:49 |
| mihir | dpm: yes, I have already kept it as google, but it didn't work on desktop but evernote did work so i was wondering. | 07:50 |
| dpm | mihir, have you gone to system settings and enabled the google account? | 07:57 |
| === chriadam is now known as chriadam|away | ||
| nik90 | Good morning | 08:59 |
| nik90 | hmm is launchpad down? | 09:01 |
| DanChapman | nik90, morning :-) i believe so I've been unable to access it for last hour or so | 09:05 |
| justCarakas | I already had a lot of timeout errors yesterday | 09:06 |
| nik90 | DanChapman: ah..:/ | 09:06 |
| JamesTait | Good morning all; happy Vinyl Record Day! :-D | 09:08 |
| mihir | Good morning nik90 DanChapman :) | 09:19 |
| justCarakas | anybody has an idea whats wrong with http://www.reddit.com/r/Ubuntu/comments/2dbhcz/ubuntu_archive_repos_down/ | 09:20 |
| nik90 | mihir: Good Morning | 09:20 |
| dpm | justCarakas, some networking issues, I think they've been fixed by now | 09:21 |
| justCarakas | dpm: is it possible that my request time outs on launchpad yesterday where alo related ? | 09:23 |
| nik90 | justCarakas: I cannot access launchpad still | 09:23 |
| dpm | justCarakas, I'm not sure, but they could be. You might want to ask on the #canonical-sysadmin channel | 09:24 |
| nik90 | ok scratch that I can access now | 09:24 |
| === chrisccoulson_ is now known as chrisccoulson | ||
| nik90 | popey or dpm: Review -> https://code.launchpad.net/~nik90/ubuntu-clock-app/fix-alarm-label-behavior/+merge/230387 (https://www.dropbox.com/s/bih4zlrun7f7yyc/com.ubuntu.clock.devel_0.1_all.click) | 09:33 |
| popey | ack | 09:33 |
| m-b-o | hey is there a way to get PageHeadConfiguration component on trusty? | 09:33 |
| popey | nik90: it wont let me download, says I need an account | 09:34 |
| popey | nik90: i do have an account, but this is probably not what you want. feel free to share a folder with me, alan@popey.com and stick stuff in it for testing | 09:34 |
| nik90 | popey: will do | 09:34 |
| popey | email also works ☻ | 09:44 |
| dholbach | nik90, popey: I pinged jibel who might be able to sort out the test/autopilot issues some time later today | 09:46 |
| nik90 | popey: :D | 09:47 |
| nik90 | dholbach: ok | 09:47 |
| nik90 | dholbach: btw I reported the bug about the click package issue | 09:47 |
| nik90 | https://bugs.launchpad.net/ubuntu-clock-app/+bug/1355295 | 09:47 |
| ubot5 | Ubuntu bug 1355295 in Ubuntu Clock App "Unable to run clock app reboot on device with reboot-packaging branch" [Critical,New] | 09:47 |
| dholbach | great, thanks | 09:48 |
| popey | nik90: in the alarm name, press space a few times ☻ | 09:48 |
| popey | you get full stops, as if it's the end of a sentence | 09:49 |
| dholbach | nik90, how can I reproduce this? | 09:49 |
| nik90 | dholbach: have you tried creating a click package and installing it on device? | 09:49 |
| nik90 | dholbach: I reproduced this using qtcreator | 09:50 |
| nik90 | dholbach: not sure about the terminal commands | 09:50 |
| dholbach | ok | 09:50 |
| nik90 | popey: why does it show fullstops when you press space? | 09:50 |
| popey | two spaces = ". " - end of sentence, gonna file a bug against osk | 09:51 |
| nik90 | popey: either way on the desktop, it seems that pressing space multiple times counts as a not empty string. So it allows that. I will fix that. | 09:53 |
| popey | k | 09:53 |
| m-b-o | popey nik90: is there a way to get the latest sdk version on trusty? | 10:09 |
| m-b-o | with PageHeadConfiguration | 10:09 |
| popey | sadly not m-b-o | 10:09 |
| nik90 | m-b-o: no the latest SDK requires 5.3 | 10:09 |
| popey | time to upgrade | 10:09 |
| nik90 | m-b-o: or use emualtor | 10:09 |
| popey | or use a vm/emu | 10:09 |
| nik90 | emulator* | 10:09 |
| m-b-o | popey, nik90: ok thanks. utopic isn't an option stm, unfortunately... will try the emulator | 10:10 |
| t1mp | bzr branch lp:ubuntu-ui-toolkit && cd ubuntu-ui-toolkit && qmake && make && source export_modules_dir.sh might work in the meantime, but it is not recommended and will break immediately when we use qt53 specific features (maybe already broke) | 10:11 |
| m-b-o | t1mp: hmm, will try the emulator ;) | 10:12 |
| t1mp | m-b-o: yeah that's the supported option :) | 10:12 |
| === daker_ is now known as daker | ||
| dpm | mihir, sorry, I got disconnected earlier. Did you manage to get the online account provider for google working? | 10:30 |
| mihir | dpm: i haven't tried yet, in office right now | 10:33 |
| mihir | dpm: will do that soon and will ping you back. | 10:33 |
| dpm | ok | 10:33 |
| mihir | dpm: as of now i have kept it on Google only, if that is the case it should work on device , could you help to test on device ? | 10:33 |
| mihir | dpm: MR, https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/1342275/+merge/230386 | 10:34 |
| popey | nik90: sorry, i didnt mean you had to email me clicks, but if that's easier, that's fine ☻ | 11:10 |
| nik90 | popey: well I can't find an easier means of sharing stuff. hmm may be I should use people.ubuntu.com for that | 11:11 |
| popey | you use dropbox? share a folder with me and just stick it in there, I'll automatically get it | 11:12 |
| nik90 | popey: yeah the backlog makes sense :P | 11:12 |
| nik90 | I will do that | 11:12 |
| popey | ☻ | 11:12 |
| popey | the bottom tab is unreadable | 11:12 |
| popey | i know it has text on it, but i cant see it | 11:12 |
| popey | as soon as i tap it, it disappears | 11:13 |
| nik90 | yeah it seems that is the default behaviour of the upstream component in dialer app as well | 11:13 |
| nik90 | I favour the older version myself | 11:13 |
| nik90 | I will check with renatu about that | 11:13 |
| nik90 | bzoltan: can you trigger a jenkins rebuild on https://code.launchpad.net/~mzanetti/ubuntu-ui-toolkit/parent-alarms/+merge/230233 as well | 11:19 |
| bzoltan | nik90: sure | 11:21 |
| bzoltan | nik90: do not expect quick result.. jenkins is super slow, but actually it started to move today after some fight | 11:21 |
| nik90 | bzoltan: thnx :) yeah I saw | 11:22 |
| nik90 | zsombi: hey If you remember we talked about ways to fix https://bugs.launchpad.net/ubuntu/+source/qtorganizer5-eds/+bug/1272337 at Malta? Unfortunately I did not note down our discussion. Do you remember how to fix it still? | 12:20 |
| ubot5 | Ubuntu bug 1272337 in Ubuntu Clock App "EDS reverts alarm status immediately" [High,Triaged] | 12:20 |
| zsombi | nik90: "triangulation" | 12:21 |
| nik90 | zsombi: what was the 3rd point other than the switch and the alarm status? | 12:21 |
| === MacSlow is now known as MacSlow|lunch | ||
| zsombi | nik90: there should be an intermediate property that you set with an assignment so it won't occlude with the binding. You can also have an idle timer which you kick so next event handling will be setting the switch | 12:23 |
| nik90 | hmm I will give it a shot | 12:23 |
| zsombi | nik90: the idea is to set one of the values async | 12:24 |
| nik90 | zsombi: ok, so I will connect the switch to this intermediate property. The intermediate property will only be updated async by the sdk | 12:25 |
| Mirv | nik90: wow, I got some location indoors now with the newest image in clock | 12:31 |
| nik90 | Mirv: sry, but the old clock uses geoIP and not the GPS. | 12:31 |
| Mirv | nik90: ah, that explains it! | 12:32 |
| zsombi | nik90: probably... depending on what you do :D | 12:32 |
| nik90 | Mirv: I am fixing that for the new clock app right from the start | 12:32 |
| nik90 | zsombi: I will give my best shot and see how it goes, If I am stuck I will let you know :D | 12:33 |
| === _salem is now known as salem_ | ||
| === MacSlow|lunch is now known as MacSlow | ||
| dholbach | nik90, trying to just run the app after I set up the sdk and everything again, I get: http://paste.ubuntu.com/8026729/ | 13:24 |
| dholbach | so... no idea | 13:24 |
| nik90 | dholbach: :/ I cannot figure out why anyone gets that error..its definitely EDS that messes up things there | 13:28 |
| dholbach | sorry :/ | 13:28 |
| nik90 | nothing much we both can do | 13:28 |
| dholbach | maybe it's time to post to ubuntu-phone@ about this branch soon | 13:28 |
| dholbach | we've been struggling for a few days now and need some expert help | 13:29 |
| nik90 | yeah I will do that by the end of the day | 13:29 |
| nik90 | this has become a critical blocker to replacing the old clock app | 13:29 |
| dholbach | yep | 13:29 |
| dholbach | I mean there's a few things which need to be figured out | 13:30 |
| dholbach | I can imagine that a fresh set of eyes will help | 13:30 |
| === chihchun is now known as chihchun_afk | ||
| nik90 | yes | 13:32 |
| nik90 | zsombi: How would I know when the alarm save operation is complete before changing the switch value? When I do var tempAlarm = alarmModel.get(index), I am not exactly sure how to wait for it. | 13:47 |
| nik90 | zsombi: if it was a new alarm, then I do Alarm{} and there I follow the onStatusChanged signal | 13:47 |
| nik90 | zsombi: I tried http://paste.ubuntu.com/8027116/ but that doesn't work | 14:14 |
| mzanetti | popey: I hoped at least for one good review of the WhereTheIssAt app :D | 14:30 |
| nik90 | mzanetti: popey tells me that you can help us with the clock app packaging. Ping me whenever you are free and we can discuss about that | 14:30 |
| mzanetti | nik90: yeah... I can try at least | 14:30 |
| popey | mzanetti: you do ☻ | 14:32 |
| nik90 | mzanetti: dholbach has put together all fixes for debian and click packaging at https://code.launchpad.net/~dholbach/ubuntu-clock-app/reboot-packaging/+merge/229173 | 14:33 |
| nik90 | mzanetti: I believe sergiusen also looked at it and contributed fixes. We just are stuck at the end | 14:33 |
| mzanetti | popey: lol | 14:33 |
| mzanetti | thanks | 14:33 |
| nik90 | I think the current issue with the store is discovering new apps. There is no way to tell what new apps landed to give them a shot and review. | 14:34 |
| === gatox is now known as gatox_lunch | ||
| mzanetti | nik90: so what's actually the issue with that branch? | 15:23 |
| nik90 | mzanetti: I can no run that branch on the phone | 15:25 |
| nik90 | mzanetti:https://bugs.launchpad.net/ubuntu-clock-app/+bug/1355295 | 15:26 |
| ubot5 | Ubuntu bug 1355295 in Ubuntu Clock App "Unable to run clock app reboot on device with reboot-packaging branch" [Critical,New] | 15:26 |
| nik90 | mzanetti: yeah branch essentially adds debian packaging. However while doing the debian packaging we noticed that the cmake instructions for the debian packaging were incorrect. As a result, files didn't install at the correct location or were missing altogether. Sergiusen did some fixes, but now the click packaging is broken as a result. | 15:30 |
| nik90 | mzanetti: so the end goal here is to fix debian and click packaging by fixing the cmake stuff. I guess that's where your expertise is needed. | 15:31 |
| mzanetti | mhm... | 15:31 |
| mzanetti | you guys are aware that all of my apps a) use qmake, b) have a custom script to create the click package and c) don't use debian packaging | 15:32 |
| mzanetti | :D | 15:32 |
| nik90 | lol | 15:32 |
| nik90 | blame popey :P | 15:32 |
| mzanetti | nik90: PYTHON_PACKAGE_DIR in tests/autopilot/CMakeLists.txt doesn't get resolved | 15:37 |
| * nik90 looks | 15:37 | |
| absk007 | how to install emmet in Geany editor? | 15:38 |
| === gatox_lunch is now known as gatox | ||
| mhall119 | absk007: try asking in #geany | 16:12 |
| absk007 | mhall119, noone has responded yet. Inactive channel. | 16:13 |
| mhall119 | ah, too bad, I use geany but don't mess too much with plugins and such | 16:13 |
| mzanetti | popey: nik90: this at least makes it run on the phone with qtcreator again for me: https://code.launchpad.net/~mzanetti/ubuntu-clock-app/reboot-packaging/+merge/230513 | 16:34 |
| mzanetti | not sure if there's some other place that requires python3 though | 16:34 |
| nik90 | mzanetti: oh | 16:35 |
| nik90 | I think dholbach EOD. I will ask him to merge it to the reboot-packaging branch | 16:36 |
| nik90 | balloons, fginther: Has python3 being enabled for clock app? | 16:43 |
| balloons | nik90, not quite yet | 16:47 |
| === absk007 is now known as Guest16400 | ||
| nik90 | balloons: ok quick question. You were discussing about the reboot-packaging with daniel, is everything ready from a testing perspective other than the python3 issue? As in can jenkins find and run the tests correctly? | 16:56 |
| balloons | nik90, should be although I'm testing some changes I'd like to make to how the tests are setup | 16:57 |
| nik90 | balloons: I am just trying to find out what else might be left. We got a cmake fix for the click packaging from mzanetti. So we are good in that area. | 16:57 |
| balloons | from my end, it can be merged | 16:57 |
| nik90 | balloons: wasn't that in your ap-setup branch? or more new stuf? | 16:57 |
| balloons | nik90, more new stuff | 16:58 |
| nik90 | ack | 16:58 |
| balloons | daniel's fix for the path issues is ok, but I'm looking at a better fix | 16:58 |
| nik90 | balloons: okay then first thing tomorrrow morning, me and daniel will try to merge it. | 16:58 |
| nik90 | balloons: does this mean jenkins can in theory automerge reboot-packaging branch if we top-approve? | 16:58 |
| nik90 | after running the tests | 16:59 |
| balloons | once francis has it passing I will rebuild it | 16:59 |
| balloons | and yes, it should automerge then | 16:59 |
| balloons | I don't see why it wouldn't.. | 16:59 |
| nik90 | awesome | 16:59 |
| nik90 | I have already sent a mail to daniel requesting him to confirm if everything in the debian package side of things are good to go. | 16:59 |
| nik90 | balloons: can you ensure that the python3 transition happens by tonite pls so that tomorrow morning is hassle free | 17:00 |
| balloons | nik90, francis said he would get to it today, so we should be good | 17:01 |
| balloons | I can definitely push the button once it's ready :-) | 17:02 |
| nik90 | :-) | 17:02 |
| nik90 | bzoltan: yay both those critical sdk fixes for alarms has been merged into staging :) :) | 17:10 |
| fginther | nik90, balloons, I made a change to force using python3 for the new clock app branch. The tests failed on my first attempt with an import error: http://91.189.93.70:8080/job/generic-mediumtests-utopic-python3/2/console | 17:30 |
| fginther | is that expected? | 17:31 |
| mhall119 | popey: DanChapman: I didn't know Dekko got multi-account support! that's awesome! | 17:31 |
| nik90 | fginther: which clock app branch? | 17:31 |
| nik90 | fginther: clock reboot or the old clock app? | 17:32 |
| fginther | nik90, lp:~ubuntu-clock-dev/ubuntu-clock-app/utopic-3.0 | 17:32 |
| nik90 | fginther: ok that's the right one..let me look through the console log | 17:32 |
| nik90 | balloons: It looks like it is failing at find_local_path() function in the __init__.py file. I remember daniel telling me that he merged your branch which should over write that? | 17:35 |
| * balloons is looking | 17:35 | |
| balloons | nik90, I remember checking the deb file and indeed it looks correct. autopilot should be finiding everything it needs in /usr/lib/python3/dist-packages/ubuntu_clock_app/ | 18:02 |
| === absk007_ is now known as absk007 | ||
| balloons | nik90, daniel's local path stuff doesn't matter for jenkins | 18:03 |
| balloons | ohh, I should bzr pull | 18:03 |
| nik90 | balloons: oh yeah since jenkins installs it as deb to run tests | 18:03 |
| balloons | ok, so he did change some things | 18:03 |
| balloons | but anyways since francis has it running on py3, I can tweak it and fix | 18:04 |
| balloons | he changed the installed_location_backend.. kind of odd | 18:05 |
| nik90 | hmm | 18:06 |
| balloons | guess I should recreate using the deb locally' | 18:07 |
| nik90 | balloons: the deb can be built by "debuild -S -sa"? | 18:08 |
| nik90 | balloons: I can try on my end as well | 18:08 |
| balloons | k, sure enough easy to recreate locally | 18:10 |
| nik90 | DanChapman: hey can dekko be built the usual way by just opening the cmakelist.txt in qtcreator and then run? | 18:13 |
| bzoltan | nik90: we know our priorities :) I will initiate a landing to the image/archive tomorrow. | 18:13 |
| nik90 | bzoltan: thnx a lot :) Owe you one | 18:14 |
| bzoltan | nik90: I wish to see this one landing https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/focus-handling-on-touch/+merge/230296 that will make the calendar app prettier | 18:15 |
| nik90 | bzoltan: it is top approved, so it just requires jenkins merge? Or do you want help with testing on calendar app? | 18:15 |
| bzoltan | nik90: I have tested that in all forms and I have run half a dozen app autopilot tests... so all it needs is a hug from Jenkins | 18:16 |
| nik90 | bzoltan: damn seems like a lot of work for you before branches can land in trunk | 18:17 |
| bzoltan | nik90: Sometimes I am thinking that adding up all these work would match with fixing couple of regressions :) | 18:17 |
| bzoltan | nik90: but at least we can be fairly sure that we do not mess up anything :) | 18:18 |
| nik90 | true | 18:18 |
| nik90 | mzanetti: for the QTests, each component must be independent for easier testing right? For instance if I want to test the AlarmPage.qml which depends on the main ubuntu-clock-app.qmml file for the AlarmModel, I will need to ensure that AlarmPage.qml has top level properties that correctly provide these data easily? | 18:20 |
| nik90 | mzanetti: this way I can import AlarmPage.qml into the test and just say "model: alarmModel" instead of having to also import ubuntu-clock-app.qml | 18:21 |
| popey | mhall119: seen https://launchpad.net/downow ? | 18:24 |
| DanChapman | nik90, no not yet. So to run on desktop cmake flags are "-DWITH_UBUNTU=on" and for run on device it's "-DWITH_UBUNTU=on -DUBUNTU_CLICK_MODE=on -CMAKE_INSTALL_PREFIX=/" I had planned to simplify it but that's low priority atm :-) | 18:25 |
| nik90 | DanChapman: so where do I add these cmake flags? As build parameters in the projects tab for a respective kit? | 18:27 |
| DanChapman | nik90, yeah in the "Cmake arguments" text field for each kit | 18:28 |
| nik90 | DanChapman: ah thnx :) | 18:29 |
| * nik90 notes this down | 18:29 | |
| mhall119 | popey: cool, but wouldn't it stop downloading when it's not focused? | 18:29 |
| DanChapman | nik90, yw :-D | 18:29 |
| popey | i guess | 18:30 |
| popey | mhall119: there's a click submitted to the store.. | 18:30 |
| mzanetti | nik90: yeah well, it depends a bit what you want to test | 18:30 |
| mzanetti | nik90: if you want test the alarmpage only, yeah, you just use that one an mock the stuff you need | 18:30 |
| nik90 | mzanetti: ok. I haven't had much luck with mocking the Alarms EDS server..so I might just link the real stuff like we do in AP tests. | 18:31 |
| nik90 | mzanetti: but I think I get the idea | 18:31 |
| mzanetti | nik90: you can do that too, yes. | 18:32 |
| mzanetti | nik90: if you think mocking would make sense but just can't figure how to do it, feel free to push a branch and I'll have a look | 18:32 |
| nik90 | mzanetti: hey btw your tutorial video was cut at the begining by a few minutes. So I missed the explanation as to why we should include tests inside Item{}. Can you explain briefly? | 18:32 |
| nik90 | ok | 18:32 |
| mzanetti | nik90: well, if you wrap an Item {} around you can open the test file in qmlscene | 18:32 |
| mzanetti | nik90: so you can debug the test more easily | 18:33 |
| nik90 | mzanetti: ah ok..In Unity8 I saw add buttons in your video, I am guessing that was due to using Item{} and other test UI elements | 18:33 |
| mzanetti | nik90: yep | 18:34 |
| mzanetti | let me try to find an easy example for that | 18:34 |
| mzanetti | nik90: for example here: http://bazaar.launchpad.net/~unity-team/unity8/trunk/view/head:/tests/qmltests/Stages/tst_PhoneStage.qml | 18:36 |
| mzanetti | nik90: it just loads the component to be tested which is PhoneStage | 18:37 |
| mzanetti | then adds 2 buttons, for adding removing apps | 18:37 |
| mzanetti | and then below all the test functions that test the phone right edge | 18:37 |
| nik90 | what's the empty rectangle{} for? | 18:37 |
| mzanetti | nik90: nothing... I just realized that's leftover from before the dash-as-app stuff | 18:40 |
| nik90 | mzanetti: ah ok. | 18:40 |
| mzanetti | nik90: same as the Binding {} below... old cruft | 18:40 |
| * mzanetti cleans up | 18:40 | |
| nik90 | mzanetti: yeah I need to think about how I structure my qml code so that I can use it in my test cases | 18:40 |
| mzanetti | nik90: one thing you should avoid is to reach out of context | 18:42 |
| mzanetti | nik90: for example, don't use variables etc which aren't defined in the same code file | 18:43 |
| mzanetti | if you need something from outside, make a property and set it where you use that component | 18:43 |
| nik90 | mzanetti: ok, let me provide an example of a test and how I go by it. | 18:43 |
| nik90 | mzanetti: I want to test creating a new alarm and so indirectly check if the alarm label, time, sound can be set. Each of these properties are in their own page. | 18:44 |
| nik90 | mzanetti: so in my test case, I will import the EditPage.qml, place it in a pagestack so that when clicking on the alarm label, it gets pushed to that page, do the test and return back to the editalarm page. | 18:45 |
| balloons | nik90, so I guess I'll just reupload dholbachs branch to a common branch | 18:45 |
| nik90 | balloons: sure, I should have suggested that to him at the start | 18:46 |
| balloons | his python code has an error and it causes the failures | 18:46 |
| nik90 | ack | 18:46 |
| nik90 | small fix? | 18:47 |
| balloons | I'm just going to put back the hardcoded paths. I'm working on the better solution, but that will be another mp for sanity | 18:47 |
| nik90 | ok | 18:47 |
| nik90 | anything to get jenkins to pass for now | 18:48 |
| balloons | they assume a build inside buildir, etc | 18:48 |
| balloons | basically I reverted to trunk, which has my mp merge in it | 18:48 |
| nik90 | ah ok.makes sense | 18:48 |
| balloons | k, jenkins runtime | 18:50 |
| * nik90 holds his breath | 18:51 | |
| nik90 | balloons: commit message :) | 18:56 |
| nik90 | balloons: nvr mind I added it | 18:58 |
| balloons | argh | 19:00 |
| nik90 | https://www.irccloud.com/pastebin/W40m9EC3 | 19:05 |
| nik90 | balloons: ^^ | 19:05 |
| balloons | ahh right-o | 19:06 |
| balloons | I fatfingered an extra line | 19:07 |
| balloons | I actually noticed it, but forgot pep8 would care | 19:07 |
| nik90 | hehe :P | 19:07 |
| balloons | ok, third time is the charm | 19:09 |
| nik90 | :D | 19:09 |
| xcyclist | Can someone tell me if the latest Ubuntu apt-get install of juju is golang? | 19:10 |
| xcyclist | Sorry. I guess I got my answer. | 19:11 |
| nik90 | balloons: it PASSEDDDDDDDDDDD! | 19:15 |
| balloons | LOL | 19:16 |
| balloons | you ready to land/ | 19:16 |
| nik90 | balloons: Old clock -> Sry bro you are out :P | 19:16 |
| balloons | you get top approval honors even | 19:17 |
| nik90 | balloons: most likely tomorrow. I want to make sure we don't get regressed on the click package side of things. I will do some testing on device and desktop. But first Dinner :) | 19:17 |
| nik90 | balloons: frankly I think dholbach should top approve. He went through all the hoops to get it done | 19:17 |
| balloons | nik90, I'm sure he like the rest of us just want to see it land ;-) | 19:18 |
| nik90 | :-) | 19:18 |
| nik90 | balloons: what remaining steps are required before the old clock app can be replaced? Anything we need to change to include in default image? Or you basically upload the new clock app click package when all this is done? | 19:22 |
| nik90 | I mean infrastructure wise btw | 19:22 |
| balloons | nik90, yes, we would simply need to rename this to clock and upload it as such | 19:22 |
| mihir | balloons: ping | 19:23 |
| balloons | mihir, pong | 19:23 |
| mihir | balloons: do you have couple of mins to test MR on device , https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/1342275 | 19:23 |
| nik90 | balloons: ack. I will have to discuss some things with popey tomorrow about what happens to the old clock app and stuff. But hopefully we can get this into the image by the end of the week or even earlier. | 19:24 |
| mihir | balloons: is there any specific pacakges needs to be installed on utopic for Ubuntu.Components | 19:33 |
| popey | pindonga hope you weren't expecting that demo app to pass ☻ | 19:33 |
| popey | nik90: ping whenever you fancy a chat about it. | 19:38 |
| balloons | mihir, so with this merge, when should I see the dialog for no account? | 19:38 |
| mihir | balloons: yup | 19:38 |
| nik90 | popey: let's do it then. My food only arrives in another 25-30 mins | 19:38 |
| mihir | balloons: is it navigating you too Google Account ? | 19:38 |
| balloons | mihir, lol.. when should it appear? I don't see anything | 19:38 |
| mihir | When you don't have any accounts and you click on Sync from right top | 19:39 |
| mihir | balloons: it should navigate you to Google account , for singing in. | 19:41 |
| mihir | balloons: actually we aren't sure regarding providerId for Setup connection , we have assumed it. | 19:42 |
| balloons | mihir, sync seems to do nothing atm | 19:58 |
| balloons | i don't see any response | 19:58 |
| mihir | balloons: okay , i'll work on that, thanks for your feedback. | 19:58 |
| balloons | mihir, do you normally display something while syncing? | 20:16 |
| mihir | balloons: it does change the Sync to Syncing if it is actually Syncing | 20:16 |
| mihir | but this code might not be correct for open System Setting directly | 20:16 |
| balloons | mihir, as far as I can tell visually I get no response clicking the sync button | 20:18 |
| balloons | it would be good if something visually occurred regardles | 20:18 |
| balloons | anyways, let me know if you need help | 20:18 |
| balloons | with running in on the device | 20:18 |
| mihir | balloons: do you have google account added on device , if yes then it'll start syncing and change the text of Menu to Syncing instead of Sync | 20:19 |
| mihir | balloons: if not, then it'll popup Add new Account , and it should navigate you to add new google Account | 20:19 |
| daker | yo popey, can you please take a look https://myapps.developer.ubuntu.com/dev/click-apps/1008/feedback/ ? | 20:33 |
| popey | done | 20:35 |
| balloons | mihir, no google account on device | 20:35 |
| mihir | balloons: okay, i'll look into it.thanks for the feedback. | 20:36 |
| daker | popey: can you look again please ? | 20:39 |
| popey | beuno: "Application status could not be updated. Please try again later." with https://myapps.developer.ubuntu.com/dev/click-apps/1008/review/ | 20:39 |
| beuno | popey, we are working on a deployment to sign packages on upload | 20:55 |
| beuno | and it's a bit wonky | 20:56 |
| popey | heh | 20:59 |
| popey | ok | 20:59 |
| === salem_ is now known as _salem | ||
| dpm | balloons, ok, so we just had the file manager meeting. As a summary, and related to AP tests, we're going to with the current branch, with just a small tweak. Just a heads up, as before that I wasn't sure if we'd need drastic changes to the branch | 21:24 |
| popey | daker: approved | 21:37 |
| daker | popey: thanks! | 21:37 |
| popey | np | 21:38 |
| mihir | balloons: ping ! | 22:19 |
| mihir | balloons: i have tried to write an AP for editing an event , though it is in WIP. | 22:29 |
| mihir | balloons: for somehow it is not letting me add guest while creating new event , giving following log http://paste.ubuntu.com/8030498/ | 22:29 |
| mihir | balloons: but if i ommit Guest, then AP does work flawlessly and fine. | 22:30 |
| ahayzen | balloons, about to start the ap refactor... what would you recommend i call this new file i dump the new tests into from test_music.py? something like test_music_app.py test_music2.py or something? | 22:40 |
| mihir | balloons: here is MR for edit AP , https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/1188819/+merge/230554 | 22:52 |
| zyga | hey | 23:30 |
| zyga | I'd like to change my app namespace | 23:30 |
| zyga | mhall119: ^^ perhaps you know the person to talk to? | 23:30 |
| zyga | mhall119: I'd like to be able to upload com.canonical.certification namespaced app | 23:31 |
| zyga | mhall119: just leave me a message please, I'm off (it's 1:30AM) | 23:33 |
| zyga | thanks! | 23:33 |
| ahayzen | balloons, when u get a moment could you rerun the tests for https://code.launchpad.net/~vrruiz/music-app/i18n-strings/+merge/230512 ... that failure looks a bit random? | 23:49 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!