/srv/irclogs.ubuntu.com/2014/09/18/#ubuntu-app-devel.txt

=== _salem is now known as salem_
Akiva-Thinkpadnik90, ping03:15
=== salem_ is now known as _salem
Akiva-Thinkpadhey all06:19
dpmmorning Akiva-Thinkpad06:20
Akiva-Thinkpaddpm, how goes the notes app?06:21
Akiva-Thinkpaddpm, need any help with it, or is it coming together nicely?06:21
dpmAkiva-Thinkpad, you mean reminders?06:21
Akiva-Thinkpadyah06:21
dpmsure, help would be really welcome06:21
Akiva-Thinkpadokay cool.06:22
* Akiva-Thinkpad branches06:22
dpmAkiva-Thinkpad, excellent. You might want to have a look at https://bugs.launchpad.net/reminders-app06:23
Akiva-Thinkpadyep already there06:24
dpmAkiva-Thinkpad, cool. So you might want to pick whichever you find interesting. One that would be good to tackle would be bug 131694706:25
ubot5bug 1316947 in Ubuntu Reminders app "Use the same font size for all notes" [High,Triaged] https://launchpad.net/bugs/131694706:25
Akiva-Thinkpadcool; I'm looking at the list now06:25
dpmit probably requires some investigation as to which default font sizes are used in the 3 different scenarios the screenshot attached to the bug represents06:26
dpmand then we should harmonize those 3 font sizes to be 1 only06:26
Akiva-Thinkpad<dpm> it probably requires some investigation as to which default font sizes are used in the 3 different scenarios the screenshot attached to the bug represents06:26
Akiva-Thinkpadopps06:26
Akiva-Thinkpadsec06:26
Akiva-ThinkpadRequestAccess failed: QDBusError("org.freedesktop.DBus.Error.ServiceUnknown", "The name com.ubuntu.OnlineAccountsUi was not provided by any .service files")06:27
dpmI'd need a bit of context as to what you're doing :)06:27
dpmwhat image, what device (phone, emulator?), etc.06:27
Akiva-Thinkpadqmlscene06:28
Akiva-Thinkpadsorry06:28
Akiva-Thinkpadsec06:28
dpmwhat desktop version?06:28
Akiva-Thinkpad14.1006:28
Akiva-Thinkpadsec06:29
Akiva-ThinkpadI need to setup an account06:29
dpmthe app should prompt you to set up the account. Try installing the build dependencies from the .deb package in the core apps ppa06:29
dpmor this should help too: http://bazaar.launchpad.net/~reminders-app-dev/reminders-app/trunk/view/head:/README.md06:30
Akiva-Thinkpadhttp://i.imgur.com/asvn4yf.jpg06:30
Akiva-Thinkpadsure06:30
Akiva-Thinkpaddpm, this won't be an issue on the phone, correct?06:31
dpmAkiva-Thinkpad, I'm still missing a bit of context. It seems the app actually runs. What happens exactly when you press the "Add account" button?06:33
Akiva-Thinkpaddpm, the debug messages at the bottom06:33
Akiva-ThinkpadAPP_ID isn't set, the handler ignored06:33
dpmright, and then the app freezes?06:33
Akiva-Thinkpadno freeze06:33
dpmso what happens in the UI?06:33
Akiva-Thinkpadit just does not escape the dialogue06:33
dpmok, gotcha06:33
dpmAkiva-Thinkpad, I think you might need to install the ubuntu-system-settings package06:34
Akiva-Thinkpadsure06:34
dpmAkiva-Thinkpad, if all fails to install on the desktop, I'd rather recommend to use a device or the emulator. It should work out of the box there06:34
Akiva-Thinkpaddpm, Thats good to hear06:34
dpmmardy, what's the situation with bug 1369941? When that lands, is the account-click branch of reminders supposed to work without any further changes? Did you happen to try the branch to test the bug fix?06:36
ubot5Error: Could not gather data from Launchpad for bug #1369941 (https://launchpad.net/bugs/1369941). The error has been logged06:36
dpmbug 136994106:36
DanChapmanGood Morning all06:42
Akiva-ThinkpadMorning DanChapman06:44
Akiva-Thinkpadgeeze now I remember what I hate about evernote; bloody can't have spaces in your password, and requires so much logging in06:51
Akiva-Thinkpadomg!!! I just changed my password; why isn't it letting me bloody login?06:54
dpmhi DanChapman06:59
mihirAkiva-Thinkpad: hey06:59
dholbachgood morning06:59
* Akiva-Thinkpad is so frustrated with evernote... 06:59
Akiva-Thinkpadbloody piece of rubbish...06:59
Akiva-Thinkpadmihir, hey06:59
=== chihchun_afk is now known as chihchun
dpmAkiva-Thinkpad, what's up with evernote?07:00
Akiva-Thinkpaddpm, it wont let me login to one of my accounts (although Im able to do it in firefox). So i created a new account, and in the settings enabled it, but this did not get rid of the dialogue, and states that I need an account07:01
Akiva-ThinkpadI wonder if this has to do with the sandbox or something07:01
dpmAkiva-Thinkpad, where are you running the app from? Still from the desktop? Try clicking on the button to add the new account, and it might prompt you to authorize the existing one instead of creating a new one07:02
dpmthere are 2 steps to using an account:07:02
dpm- login to evernote07:02
dpm- approve reminders to access the account07:03
dpmGenerally you create accounts using reminders07:03
dpmand the 2 steps happen together07:03
Akiva-Thinkpaddpm, thats the thing; It asked me to login; I tried using my email and username with my password. Didnt' work. I tried changing my password, retried, still would not let me login.07:03
Akiva-ThinkpadSo I created a new account. okay; that worked, now I see my new account in the lists of accounts.07:04
* Akiva-Thinkpad should just screenshot it.07:04
dpmthe thing with your account is weird, as login is handled by the evernote servers07:04
Akiva-Thinkpaddpm, yah it feels as if its the evernote side of things07:04
dpmas per the second account, if login has happened, then it's an issue with the app or Online Accounts07:04
Akiva-Thinkpadi'm contacting support07:04
mihirhye Akiva-Thinkpad  :) how are you ?07:05
dpmAkiva-Thinkpad, do you have the ubuntu-system-settings app installed (i.e the phone app, running on the desktop)? I think unless you install it, the whole authorization process won't work07:06
Akiva-Thinkpaddpm, yep http://imgur.com/wS82ev307:06
Akiva-Thinkpadim getting the debug message on that click  ---  APP_ID isn't set, the handler ignored07:06
dpmah, great. And does the "Allow" button work?07:07
Akiva-Thinkpadmihir, APP_ID isn't set, the handler ignored07:07
Akiva-Thinkpadopps07:07
Akiva-Thinkpaddpm APP_ID isn't set, the handler ignored07:07
Akiva-Thinkpadthat debug posts07:07
Akiva-Thinkpadwhen I press "Allow"07:07
dpmI think all apps get the APP_ID isn't set warning07:07
Akiva-Thinkpadhmmmm07:07
mardydpm: yes, it will work without changes, I tested your click package07:09
mardydpm: but the click package needs to get installed *after* the silo has landed07:09
mihirdpm: any feedback would be nice if you have , https://plus.google.com/116527124812584566245/posts/YpUGud6y7Jy :)07:10
dpmmardy, excellent. Yes, obviously we won't land the branch until the UOA branch has landed - is there a silo assigned to it yet?07:20
dpmmihir, cool, looking now!07:20
mardydpm: silo 13, you are very welcome to test it :-)07:21
dpm\o/07:21
dpmmihir, added some feedback to the week view picture, nice work!07:26
dpmAkiva-Thinkpad, I think I'd recommend using the emulator rather trying to figure out which part doesn't work on the desktop. Although perhaps rpadovani has a working Reminders desktop setup and can help you. I generally test the app on the phone or emulator these days07:28
Akiva-Thinkpaddpm, yah; strange though that I had it working on the desktop way earlier. Oh well; do what you can~07:29
* Akiva-Thinkpad fires up the ol emulator07:29
dpmyeah, me too, but it was way back in trusty07:29
dpmin utopic it should theoretically work better, but it's been a while since I launched the app on the desktop07:30
Akiva-Thinkpadah, okay so you ended up switching too. Interesting; I feel less useless now :P07:30
dpmyeah, my laptop has been in utopic for a while, but I still keep trusty on my desktop in case I need to test anything on the LTS07:35
JamesTaitGood morning all; happy Respect Day! :-D08:07
mihirmorning JamesTait  :)08:07
mzanetti:)08:07
=== Sooty is now known as Guest20885
justCarakasmorning08:11
Guest20885morning, very new here, so not sure what to do08:13
dpmhi Guest20885, just saying 'hi!' is a very good first step :)08:16
dpmare you interested in writing apps for Ubuntu on phones?08:16
Guest20885Oh right, thanks :)08:16
Guest20885No, just for Ubuntu Desktops08:16
dpmok, gotcha08:17
dpmpeople are more into the phone on this channel, but if you've got any questions, feel free to ask, and if someone is around who knows the answer, they will try to help08:17
Guest20885oh, ok, thanks. Is there another channel for dektop dev, then?08:18
Guest20885i guess i should go and learn how to use QT Creator08:19
dpm_zbenjamin, I'm trying to test the scopes template and I'm using an up-to-date QtC and I've just updated my chroots. However, I got this error about missing dependencies in the chroot. Were they not supposed to be installed by default? http://pastebin.ubuntu.com/8370793/08:57
zbenjamindpm_: yeah somehow they did not :(09:04
dpm_zbenjamin, is this a known bug, then?09:04
zbenjamindpm_: yes09:04
dpm_zbenjamin, do you have the bug id?09:04
zbenjamindpm_: https://bugs.launchpad.net/qtcreator-plugin-ubuntu/+bug/137072709:05
ubot5Launchpad bug 1370727 in qtcreator-plugin-ubuntu "click chroots missing dependencies for the scope template" [Undecided,New]09:05
dpm_zbenjamin, thanks. Would it be possible to triage it? I'd suggest to mark it as High09:06
zbenjamindpm_: i just tried that, i get timeout errors :/09:06
zbenjamindpm_: bah launchpad seems to have problems09:07
dpm_ok, let's see if we can escalate this one, as it's a blocker for using the tools out of the box09:07
zbenjamindpm_: is this on trusty or utopic?09:11
dpm_zbenjamin, utopic, afaik scopes development is only supported on utopic09:11
zbenjamindpm_: with chroots it should work on trusty as well, but i just learned that our MPs have landed but click was not released since then for some reason09:12
popeydpm_: are you able to build filemanager in an armhf kit for device? e.g. https://code.launchpad.net/~ubuntu-filemanager-dev/ubuntu-filemanager-app/force-authentication-parameter/+merge/23401509:13
popeyi get this /tmp/force-authentication-parameter/src/plugin/folderlistmodel/dirmodel.cpp:45:41: fatal error: taglib/attachedpictureframe.h: No such file or directory #include <taglib/attachedpictureframe.h>09:13
dpm_popey, you'll need to do this: http://bazaar.launchpad.net/~ubuntu-filemanager-dev/ubuntu-filemanager-app/trunk/view/head:/README#L3109:14
popeyoh good spot!09:16
popeyta09:16
dpm_np09:16
dpm_zbenjamin, also, when I press the play button on an emulator kit, it launches the emulator, which is great. However, the emulator then shows a password prompt which I cannot get past. I've tried 'phablet', 'ubuntu' and no password, but no success. I'm guessing this is related to developer mode - is there any switch in QtC which allows me to get past this authentication dialog? -> http://i.imgur.com/6wVJ1m0.png09:19
zbenjamindpm_: uh09:20
zbenjaminbzoltan: do you know the default password for the emulator?09:20
zbenjamindpm_: try 000009:21
dpm_zbenjamin, that worked, thanks, but developers will have a hard time figuring it out. Where does this password prompt comes from? it seems to be different than the standard locking screen09:22
zbenjamindpm_: thats the new default behaviour if you want to enable developer mode on your phone09:23
zbenjamindpm_: its the same on the phone, you are forced to set a pw or you cannot enable dev mode09:23
zbenjamindpm_: and i don't like it either09:23
dpm_zbenjamin, right, but I've done it on the phone, but it's different09:23
dpm_it uses the regular locking screen09:23
zbenjamindpm_: for me it looks exactly the same on the phone09:24
zbenjamindpm_: probably it looks different if you set up a code instead of a password09:24
dpm_zbenjamin, ah, that might be it, I've got a code. Let me try to switch to password09:24
bzoltanzbenjamin:  dude :) we set it09:30
bzoltanzbenjamin:  000009:31
zbenjaminbzoltan: do we? ;)09:31
bzoltanzbenjamin:  yes09:31
bzoltanzbenjamin:  wait a sec ... no09:32
bzoltanzbenjamin:  I was wrong... we do not set it09:33
zbenjaminbzoltan: duuuude ;)09:33
bzoltanzbenjamin:  I know.. I am stupid, like very very09:34
dpm_bzoltan, is there a way to set the password from QtC and send it to the emulator, so that the prompt effectively goes away?09:54
dpm_I remember there being a discussion on the mailing list and I thought we had reached a conclusion on how to set the password09:55
bzoltandpm_: for the device flasher it is possible09:55
bzoltandpm_: But the `ubuntu-emulator create` does not take password parameter.09:56
bzoltandpm: But I know what you need... a sec09:56
=== chriadam is now known as chriadam|away
bzoltandpm: what we need here is this -> http://pastebin.ubuntu.com/8371134/09:59
bzoltandpm_: zbenjamin: disable the intro, the welcome setup, and move away the lock screen10:00
zbenjaminbzoltan: ugh10:01
bzoltanzbenjamin:  well... that is what we need.10:02
zbenjaminbzoltan: there is a sudo in it10:02
nik90bzoltan: yes that would be nice10:02
bzoltanzbenjamin: echo 0000|sudo -S10:02
zbenjaminah10:02
bzoltanzbenjamin:  we can not do better10:02
bzoltanzbenjamin:  it is ugly like hell... but what options we have?10:03
zbenjaminbzoltan: we could put that into the launcher i guess10:03
bzoltanzbenjamin:  yes10:03
dpmbzoltan, do you think it's worth having a discussion on possible solutions on the phone mailing list, or have you had that conversation already?10:04
bzoltanzbenjamin:  need to be tested properly  maybe the full 4 lines is overkill :)10:04
zbenjaminbzoltan: its ugly that the powerd-cli does not exit10:05
bzoltandpm_: I have talked about this problem with the CI folks but only in the context of auto provisioning real devices. Where I had exactly the same issue. But there I flash with the --password ${PASSWORD}10:05
bzoltanzbenjamin:  maybe we can live without powerd-cli ... does the emulator ever dimm10:05
zbenjamini don't think so10:07
bzoltanzbenjamin:  so we can loose the first two lines10:08
bzoltanzbenjamin:  I am hungry ... I go to hunt10:09
zbenjaminbzoltan: good luck10:09
dpmthanks bzoltan10:16
nik90gventuri_: I removed the "-" selection state. Do you want to test or should I go ahead and get a code review and merge?10:25
gventuri_nik90: let's test10:25
nik90gventuri_: ok, 5 sec. I will update the click in dropbox10:25
gventuri_nik90: cheers10:26
popeyElleo: if you made a an unconfined version of your spotify app any time soon, I would buy you beer the next time I see you.10:27
popeyJust saying ㋛10:27
=== chihchun is now known as chihchun_afk
nik90t1mp: based on your comment https://bugs.launchpad.net/ubuntu-clock-app/+bug/1370146/comments/4, should I wait for it to land in the SDK or merge my MP?10:33
ubot5Launchpad bug 1370146 in Ubuntu Clock App "Adjust header behaviour & UI in multi select mode" [Medium,In progress]10:33
nik90t1mp: it just got design signed off :)10:33
Elleopopey: yeah, sorry, it's been something I've been meaning to do; but I need to fix some issues with it using the latest SDK first too10:34
Elleopopey: something broke in the past year so the listview no longer displays entries properly, but I haven't had time to dig into it10:34
t1mpnik90: better land it. I can use it as an example, and it will be easy later to switch to the uitk version10:36
nik90ok10:36
t1mpincluding the time to run all our tests and land a new uitk it can take weeks10:37
t1mpnik90: ^10:37
nik90ack :)10:37
popeyElleo: bummer.10:41
popeyElleo: no popey pressure10:41
Elleoheh10:44
dpmdavmor2, have you seen this issue? (I'm using N4, ubuntu-rtm/develp-proposed) 1. Open dialer, 2. Click on settings, 3. Click on the back arrow on the header, 4. You get stuck in system settings: neither Launcher or right edge works10:57
davmor2dpm: works fine on krillin I can check on n4 after for you10:59
dpmdavmor2, thanks, nothing urgent or anything, I thought I'd double-check before reporting a bug11:06
mihirnik90: ping, when you get time can you just merge with trunk and let Jenkins runs ?11:23
nik90mihir: sure11:23
mihirnik90: thanks, i am at work otherwise would have done that :) else will do it tonight11:23
nik90ok11:24
mihirnik90: also, can you do the code review whenever you get time :) i'll speak to kunal today in meeting to review this MR11:24
nik90mihir: I plan on attending the meeting as well11:24
mihirnik90: that's great :)11:25
nik90:)11:25
nik90rpadovani or t1mp: If you some time to spare, can you review https://code.launchpad.net/~nik90/ubuntu-clock-app/improved-multiselectmode/+merge/234992. Just a code review would be sufficient since we have tested it extensively already.11:35
* nik90 reviews dpm's branch11:37
nik90zbenjamin: can you help us fix https://code.launchpad.net/~dpm/ubuntu-clock-app/reboot-desktop-i18n/+merge/230582. dpm proposed it to ensure that the desktop file is translatable. The MP modifies the desktop file location and the cmake file. Running it via qtcreator produced "Error: could not find ubuntu-clock-app.desktop"11:43
=== MacSlow is now known as MacSlow|lunch
=== _salem is now known as salem_
mihirnik90: i did comment on MR :)12:00
mihiri meant your MR12:00
nik90mihir: thnx :)12:00
=== gatox_ultra is now known as gatox
nik90mihir: I am making one more nice little UI fix in a separate branch...until I push its a surprise :-)12:33
popeyhah12:35
popeyis it "sync with microsoft outlook"? ☻12:35
nik90popey: UI fix :)12:35
popeynik90: I have to go out early this evening so will miss out 17:30 meeting. Either I can ask mhall119 to cover (if he's around) or we can move to tomorrow or later in the evening when I'm back (~20:30 my time onwards)12:35
nik90popey: I am cool with whatever you propose :)12:36
popeyok, lets catch up later today then, I'll ping you when I'm back12:36
nik90ok12:38
nik90telegram me if required :P12:38
popeythanks!12:38
popeyhaha12:38
popey12:38
zbenjaminnik90: your manifest file says the desktop file can be found at share/applications/ubuntu-clock-app.desktop  but it is installed into /share in the click package12:46
zbenjaminnik90: thats a bug in the buildfiles12:46
nik90zbenjamin: ah12:46
zbenjaminnik90: easy to fix , just put @CMAKE_INSTALL_DATADIR@/ubuntu-clock-app.desktop into the manifest12:48
nik90zbenjamin: but isn't share/applications/ubuntu-clock-app.desktop the correct location?12:48
nik90zbenjamin: I imagine the deb package would like to install it in /usr/share/applications12:49
zbenjaminnik90: the correct location is where the manifest points to12:49
zbenjaminnik90: the point is the path in the manifest file needs to be correct, the click package does n to care where it is installed to12:49
nik90ok12:50
nik90thnx :)12:51
=== MacSlow|lunch is now known as MacSlow
mihirnik90: don't keep secret among us , hehehe :P12:58
mihirpopey: are we postponing meeting?12:59
mihirnik90: ^^12:59
popeymihir: no12:59
popey20 seconds left !12:59
nik90mihir: he was talking about the clock meeting12:59
nik90lol12:59
mihiroops, got confuse , nik90  is everywhere ;) hahaha13:00
popeymihir: you're not in -meeting though ☻13:00
* mihir joins13:00
popeybfiller: is Ugo about? can he update https://code.launchpad.net/~uriboni/ubuntu-calendar-app/fix-translations-with-arg/+merge/232903 when he has a chance?13:16
ahayzenpopey, sorry for not being in the meeting earlier, all i'll say is "freshers week" ;) ... what did i miss? I saw there is a design doc now?14:10
popeyahayzen: no problem, am on a call right now...14:11
balloonsahayzen, ahaha.. freshers week14:16
ahayzenballoons, yey lets drink loads of alcohol until some silly hour in the morning :)14:16
dholbachmhall119, can you help me with bug 1369803?14:27
ubot5bug 1369803 in Ubuntu App Developer site "Clicking "App Development Training" does not do anything" [Undecided,New] https://launchpad.net/bugs/136980314:27
ahayzenballoons, ooo weather py3 just passed :)14:47
balloonsahayzen, all core apps are now python3 :-)14:47
ahayzen\o/14:47
balloonsahayzen, but doc-viewer and sudoku still need converted14:47
balloonsI just approved weather :-)14:48
ahayzenballoons, thanks :)14:48
balloonswant to do sudoku?14:50
* popey hugs ahayzen and bfiller 14:55
popeyand balloons14:55
* balloons checks sudoku again.. I think I'm crazy and it's already done14:57
ahayzenballoons, what is the status of this? https://code.launchpad.net/~fcole90/ubuntu-docviewer-app/ubuntu-docviewer-app/+merge/23443814:58
ahayzenballoons, it looks like the autolanding tried to use py3 and the CI was using py2?14:59
balloonsahayzen, we missed the cutoff to slide in under py2, so fabio will have to re-add py3 again ;-)14:59
ahayzenballoons, oh u just commented :P14:59
balloonsI feel bad14:59
ahayzenballoons, :( well at least then *everything* will be py3 :)15:00
balloonsyes15:00
balloonshuzzah15:00
ahayzen\o/15:00
nik90zbenjamin: for the https://code.launchpad.net/~dpm/ubuntu-clock-app/reboot-desktop-i18n/+merge/230582, does http://paste.ubuntu.com/8372904/ look good?15:06
nik90zbenjamin: this way we don't change much to what was there before15:06
zbenjaminnik90: sure15:07
zbenjaminnik90: if it works with click then its ok anyway, you just need to verify the click package comes out right15:07
nik90zbenjamin: well qtc runs the app properly on the emulator. I just wanted to make sure the debian packaging stays intact as well :)15:07
t1mpzsombi: still around?15:08
nik90zbenjamin: I checked the .sdk-emulator-deploy folder and it is as expected15:08
zsombit1mp: not for long15:08
nik90dpm: I commented on https://code.launchpad.net/~dpm/ubuntu-clock-app/reboot-desktop-i18n/+merge/230582. It is almost good to go15:08
t1mpzsombi: hmm.. long enough for a small review? https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/10-ap-wait_for_animation/+merge/23514615:09
t1mpzsombi: can do it tomorrow also if that suits you better15:09
zsombit1mp: looks pretty simple15:10
t1mpzsombi: yes, doesn't really do anything, but with this we can add header.wait_for_animation() to the apps already15:10
zsombit1mp: looks good, happroving15:11
t1mpzsombi: great, thanks!15:12
nik90davmor2: hey, you around?15:14
davmor2nik90: I am15:16
nik90davmor2: can you help confirming bug 1364088 again pls with rtm 4315:16
ubot5bug 1364088 in Indicator Date and Time "When the alarm ringing length is set to 10 minutes in clock app, it rings for 11 minutes." [Undecided,Confirmed] https://launchpad.net/bugs/136408815:16
nik90davmor2: charles can't reproduce it...and I am testing it again now15:16
renato___dpm, could you help me? I do not know what is happening. I am trying to do a address-book-app release and I getting a tons of changes on "*.po" which is not related with my MR15:16
davmor2nik90: see #ubuntu-touch15:16
nik90davmor2: :)15:17
renato___dpm, none of my changes touch on "po" files15:17
renato___dpm, and the diff is showing that the file "po/az.po" will be removed15:17
charlesdavmor2, nik90, I have an alarm due to dismiss at :25 so I'll report back then15:18
renato___dpm, I do not know how to solve it15:19
=== karni_ is now known as karni
charlesdavmor2, nik90: https://bugs.launchpad.net/indicator-datetime/+bug/1364088/comments/115:27
ubot5Launchpad bug 1364088 in Indicator Date and Time "When the alarm ringing length is set to 10 minutes in clock app, it rings for 11 minutes." [Undecided,Confirmed]15:27
nik90charles: I need another 3 mins to finish testing :)15:27
nik90charles: same here it ended correctly after 10 mins15:30
nik90commented on bug15:31
=== gatox is now known as gatox_lunch
dpmnik90, applied your patch, let me know if that's all that was needed16:06
dpmrenato___, sorry, I was otp, can you give me some more context and perhaps point me to the branch where you're seeing those changes? I might need to understand how you do the address-book-app release before I can be of any help16:07
mihirnik90: ping16:10
nik90mihir: I am going out now...brb in 1 hour. Talk then?16:10
ahayzenpopey, just submitted the multi file support for content-hub \o/ ... meanwhile for the translations fixes is victor correct with his point in 1) or is it fine as it is? https://code.launchpad.net/~andrew-hayzen/music-app/fix-1369050-content-hub-fix-translations/+merge/23457216:10
mihirsure no issues :)16:11
popeyahayzen: nice one!16:13
nik90mihir: just remembered I got irc client on phone.. so feel free to just message me16:13
popeydpm: do you know what we plan to do for Music directory? I assume not translate it?16:13
popey(guessing it's hardwired in MTP etc)16:14
mihirnik90: i commented on your MR16:16
mihirnik90: https://code.launchpad.net/~nik90/ubuntu-calendar-app/tweak-event-color/+merge/235137/comments/57501216:16
dpmpopey, I think it was, let me double-check. I think if it still is, it will be hardwired for a while, but it'll probably be translated when we have a converged desktop16:16
mihirotherwise , it works great :)16:16
dpmif we're still using file managers by then :)16:16
popeylolz16:17
=== gatox_lunch is now known as gatox
dpmbfiller, I've got some other i18n MRs, I think they should be the last ones to close that bug about the app names not being localized:16:23
dpmhttps://code.launchpad.net/~phablet-team/notes-app/notes-app-desktop-translations/+merge/22323316:23
dpmhttps://code.launchpad.net/~phablet-team/gallery-app/gallery-app-desktop-translations/+merge/22284716:23
dpmbfiller, do you think someone in your team could review them?16:23
bfillerdpm: ok16:26
ahayzenpopey, so does that mean we leave it fixed for now?16:26
dpmgreat, thanks bfiller16:26
popeyyes ahayzen16:29
* popey goes afk for some hours16:29
ahayzen\o/16:29
dpmpopey, ahayzen, does this branch mean the content hub is getting multiple file import and the music app only needs to adapt to support that? -> lp:~andrew-hayzen/music-app/content-hub-multi-file16:38
ahayzendpm, content-hub already supports multi file...just we in the music-app would only process the first..now we'll accept as many as you like coming in one bundle16:39
ahayzendpm, should help with bug 1368648 and bug 136599316:40
ubot5bug 1368648 in Ubuntu Music App "Send multiple files via content hub" [High,In progress] https://launchpad.net/bugs/136864816:40
ubot5bug 1365993 in Ubuntu Music App "Support download of albums as zip files from 7Digital" [Critical,In progress] https://launchpad.net/bugs/136599316:40
dpmahayzen, yeah, that's why I was asking, thanks for clarifying!16:41
ahayzendpm, we'll still need content-hub to extract the zip and send the files in a 'bundle' to us...but our end is basically there (assuming it is approved)16:41
dpmexcellent16:42
dpmkenvandine, I've not followed that one up in a while - is there a plan for content-hub to extract .zip files from 7digital, so that they can be fed to the music app?16:42
ahayzendpm, when i last spoke with people this comment was basically going to be the plan https://bugs.launchpad.net/music-app/+bug/1365993/comments/616:43
ubot5Launchpad bug 1365993 in Ubuntu Music App "Support download of albums as zip files from 7Digital" [Critical,In progress]16:43
ahayzendpm, but i'm sure kenvandine knows more about the situation than i do ;)16:43
dpmcool, thanks :)16:44
kenvandinedpm, sergiusens is doing the download manager work16:44
kenvandineshouldn't actually require any changes to content-hub16:44
kenvandineand Elleo is doing the browser work16:44
dpmah, nice16:45
Elleosergiusens: on the topic of that 7digital thing, I was thinking just having a property on SingleDownload "requestExtraction" or something could do the trick as far as any extra API for browser integration goes (unless you'd rather just automatically extract anything that's compressed?) what're your thoughts?16:46
Elleosergiusens: having an explicit property that says whether to extract or not might be more flexible though, as there could be some future app that wants to keep compressed things as zip files or something16:48
nik90mihir: you htere?17:43
nik90there?17:43
nik90Ain't this pretty -> https://i.imgur.com/zlXeX9l.png18:17
davmor2SHIP IT!18:19
nik90davmor2: patience my friend...all good things take time to ship :P18:20
=== karni is now known as Guest45475
nik90davmor2: btw does striking out the labels for past events look good?18:20
nik90atm I made past events lighter, made the font italic and striked it out18:21
davmor2nik90: hahahaha18:21
nik90balloons: I need your help debugging why jenkins is failing https://code.launchpad.net/~dpm/ubuntu-clock-app/reboot-desktop-i18n/+merge/23058218:24
nik90balloons: its a harmless merge which only adds translation support for desktop files18:24
balloonsnik90, sure thing18:24
nik90the log is a bit cryptic18:25
balloonsnik90, mwahaha18:25
balloonsok, I know the issue18:25
balloonslet me resolve it18:26
balloonsnik90, actually I think I can do you one better. I think I can land it without making you wait for me to fix18:26
nik90hehe18:27
nik90so what's the issue though?18:27
nik90and does it apply across all MPs?18:27
balloonsyes, the autolanding job18:27
=== Guest45475 is now known as karni
balloonsjust for you though :-) aka clock18:27
nik90:-)18:29
mihirnik90: hi18:54
nik90mihir: hi18:54
nik90mihir: how's https://imgur.com/zlXeX9l18:54
nik90mihir: I removed the opacity but reduced the brightness of the color, striked out the title and made it italic for past events18:54
mihirnik90: WoW :D18:54
mihirthis looks good :)18:55
nik90mihir: :)18:55
nik90mihir: btw I found a visual regression in our viewdesign branch18:55
mihirnik90: did you mentioned over there?18:55
nik90mihir: in the screenshot I linked, look at the event in the bottom left..it seems cut of for no reason18:55
nik90mihir: no I haven't18:55
nik90mihir: I am looking for a fix18:55
mihirnik90: hmm , i believe it is again that margins18:56
nik90mihir: well this time it is something to do with the bottom margin. let me check18:57
mihirnik90: thre?19:01
nik90yes19:01
mihirnik90: on your branch , i was trying diff cal color , with this it looks lesss readable,  http://i.imgur.com/73aZQSD.png19:01
nik90yeah I am planning to change the colors we provide to fix that19:02
ahoneybunyay a RTM image!19:02
mihirnik90: we can put colors which is more visible with this design  as it is not connected to google calendar as well19:02
nik90mihir: exactly19:02
ahoneybunthat is a really hard to read yello nik9019:03
nik90ahoneybun: yup19:03
nik90mihir: yellow is the only trouble color right?19:04
nik90mihir: all the other colors should work19:04
mihirnik90: yes , yellow is the problem , either we can make darker yellow then it should work19:05
nik90mihir: should be an easy fix19:05
ahoneybunnik90: maybe pick a darker yellow like the yellow they use on REAL notepads?19:05
nik90ahoneybun: you got a hex code we can use19:06
mihiryeah, just noticed so informed :)19:06
mihirnik90: #FFFF0019:06
mihirnik90: http://www.rapidtables.com/web/color/Yellow_Color.htm19:06
nik90mihir: isn't that bright yellow19:07
mihirnik90: how about this , #FF9912 ?19:08
nik90yeah that might be ok19:09
ahoneybunnik90: should I still look for the code?19:13
nik90ahoneybun: ur call19:13
mihirahoneybun: sure , opinion would be great19:13
mihirnik90: just push that code , i'll re-test and approv before i go to sleep19:13
nik90mihir: no hurry, it can go tomorrow19:14
ahoneybunmihir: nik90 let me grab that ss and add my color, e8d13919:14
nik90mihir: I want to make one more fix before that gets merged19:14
mihirnik90: okay no issues,19:14
mihiri am making change in color popup, would you mind if i do saperate branch ?19:14
mihirnik90: it is trivial, or you can add in yours as well :)19:14
nik90mihir: I was about to fix the color popup height19:15
ahoneybunnik90: I need to upload it to a img host give me a min19:15
nik90mihir: go ahead and propose it, we can then merge yours to trunk first if you so wish19:15
nik90ahoneybun: sure, no hurry19:16
ahoneybunnik90: http://s13.postimg.org/nm0ewko3r/73a_ZQSD.png19:16
mihirnik90: how about this , http://i.imgur.com/SlBsl8A.png ?19:16
ahoneybunkinda crap box but still19:16
ahoneybunthat yellow might look great mihir19:17
nik90mihir: looks better. can you also fix the height as well then19:17
nik90mihir: atm it is hardcoded to 100 gu units which is incorrect19:17
ahoneybunmihir: this is the color from your new image http://s1.postimg.org/o7ah4jer3/73a_ZQSD.png19:18
mihirahoneybun: yeah , it looks better19:18
mihirnik90: it has to be parent.height right ?19:18
nik90ahoneybun: looks better19:19
nik90mihir: no19:19
ahoneybunyay!19:19
nik90mihir: it should have no height19:19
nik90mihir: dialog will automatically calculate its height19:19
nik90atleast that's what I think19:19
ahoneybunmihir: nik90 if anyone needs some help with gimp or design ideas, I'm more then happy to help!19:20
mihirnik90: okay , let me propose19:20
nik90ahoneybun: awesome..thnx for your support ahoneybun19:20
nik90ahoneybun: pretty sure we will need help19:20
mihirahoneybun: sure, that would be great..19:21
mihirahoneybun: how about you can come of some design for EventDetails ?19:21
ahoneybunnik90: np thats why I'm here, btw I'm getting a Nexus 4 on saturday so if you need a tester as well19:21
nik90ahoneybun: sweet19:21
nik90ahoneybun: also what mihir said ^^19:21
ahoneybunnik90: for tablet and phone or what?19:21
mihirahoneybun: as of now , tablet19:22
nik90ahoneybun: we are thinking of revamping the event details page19:22
nik90mihir: eh?19:22
nik90for phone19:22
ahoneybunok just shoot me your designs if you have any and I'll take a look19:22
ahoneybunor just a ss of the main calendar19:22
mihirnik90: sorry , for phone hehehe19:23
ahoneybunwait I have 14.10 so I could add a ppa with or what not19:23
nik90ahoneybun: on your desktop?19:23
ahoneybunyes nik9019:23
ahoneybunrocking great so far19:23
mihirnik90: how can i make ubuntushape round ?19:24
* ahoneybun got a nexus 4 just for ubuntu touch19:24
nik90mihir: round as in a circle?19:24
ahoneybunnik90: round corners19:24
mihirnik90: yup , or we have to use Rectange + radius ?19:24
nik90mihir: you can only increases the radius from "small" to "medium"19:24
mihirnik90: yeah i saw that in doc, was wondering if there is anyway...19:24
nik90mihir: if you want circle, you need to use rectangle19:24
nik90ahoneybun: basically we are looking for design ideas. So for instance we want to revamp the event-details page. So it would be great if you can come up with somem mockups19:25
nik90and me and mihir will try to implement it and give our feedback19:25
ahoneybunok nik90 can you throw me some ss of the current way we are handling event-details?19:26
ahoneybun*ss = screenshots19:26
nik90ahoneybun: ah sure19:26
nik90one sec19:26
ahoneybunok19:26
nik90ahoneybun: https://imgur.com/9gS2off19:27
nik90ahoneybun: https://imgur.com/svrw8i519:28
ahoneybunnik90: so when you click the event on the calendar that page comes up?19:29
nik90ahoneybun: yup19:29
ahoneybunok I'll look at some other designs, I remember seeing a svg with a lot of ubuntu ui elements19:30
balloonsnik90, I left a comment on the mp. Trying to get someone to have a look at the job.. The trouble is it's a bug in phablet-tools19:30
nik90balloons: cool. thnx for looking into it.19:30
nik90balloons: would I be able to do a clock release tomorrow?19:31
ahoneybunnik90: there are color codes for different type of events in the calendar right?19:31
nik90ahoneybun: yes19:31
nik90ahoneybun: they look like https://imgur.com/zlXeX9l19:31
balloonsnik90, it's definitely due, been awhile19:32
nik90balloons: but I need to merge 2-3 MPs before I can release though19:32
nik90balloons: design fixes essentially19:32
ahoneybunnik90: working on it :)19:34
nik90ahoneybun: yay..thnx19:34
* mihir push the change19:34
mihirahoneybun: thanks :)19:34
mihirahoneybun: let us know if you need any feedback , inputs or help19:34
ahoneybunwill do19:34
mihirnik90: https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/ColorPopUpModify/+merge/23520519:36
nik90mihir: screenie pls19:37
nik90mihir: I found the issue with the regression I told you19:38
mihirnik90: it is same as previous change , still http://imgur.com/L1c3vtu19:38
mihirnik90: the edge cut off?19:38
nik90mihir: In TimeLineBaseComponent.qml you set the flickable height to parent.height - allDayContainer.height19:39
nik90mihir: however in some days the addDayContainer is not shown...and that was the issue19:39
nik90mihir: how are you hiding the AllDayContainer on some days?19:39
mihirnik90: it is coming from that file only, we check if there are any events then make it visible else hide it19:40
nik90mihir: do you hide it in AllDayContainer.qml or in TimeLineBaseComponent.qml ?19:41
nik90mihir: because when you hide it, the flickable height should be parent.height instead of parent.height - allDayContainer.height19:41
mihirin AllDayEvent , otherwise we have to execute same redudent code here i guess19:41
nik90mihir: rest assured I will fix it19:41
mihirnik90: so either we can use that function to define weather we have all day events , then make it visible and and adjust height accordingly19:43
nik90mihir: yup I am doing it atm19:43
mihirnik90: okay, great :)19:45
mihirnik90: how about this, http://i.imgur.com/d6jmNbG.png19:51
mihir?19:51
ahoneybunnik90: http://s8.postimg.org/i9homkcnp/Event_Details_Design.png19:51
nik90mihir: circle looks better imo19:51
nik90ahoneybun: the guest list can be long sometimes, how would that fit?19:52
ahoneybunnik90: true let me fix19:52
mihirnik90: final one , http://imgur.com/UZenX3h reduce space inbetween19:53
nik90ahoneybun: do we need to specify start: and end:. Cant it be just 13:00-14:00 ?19:53
nik90mihir: its not centered horizontally in the dialog19:53
mihirahoneybun: it looks better :)19:54
nik90mihir: also since we know the exact amount of colors, can't we assign it 2 rows and 4 columns? and reduce the grid height a bit19:54
nik90mihir: too much vertical space imo19:55
nik90of the dialog I mean19:55
ahoneybunhttp://s16.postimg.org/yz9zu6dnp/Event_Details_Design2.png20:02
mihirahoneybun: hmm , looks good nik90 ^^20:04
ahoneybunmihir: sad to say I dont know how to make round boxes in gimp though ;(20:04
nik90ahoneybun: the overview looks good, but you need to specific about which font-weight we should use, the text being left-centered and so on.20:05
nik90ahoneybun: the revamp will only happen next week (most likely)20:05
nik90ahoneybun: you get the next few days time to polish the mockup and specify those details20:06
ahoneybunnik90: I think the text should be centered no?20:06
nik90I am not sure, I will have to implement and check how it looks on device20:07
mihirahoneybun: take few days to brainstrome the design20:07
ahoneybunmihir: will do20:07
ahoneybunI'm doing research on current designs20:07
mihirahoneybun: propose and we'll try to convert that , and see how does it looks.20:08
nik90ahoneybun: also we need to maintain compatibility with rest of the calendar app pages20:08
mihirnik90: pushed the code20:08
ahoneybunmihir: not sure how to propose other then it has to do with lp20:08
mihirnik90: this should give me grid in center, http://paste.ubuntu.com/8374780/20:09
mihiris that correct?20:09
* mihir get nuts with anchoring in qml :( heheh20:09
nik90I would expect so20:10
mihirahoneybun: i'll file bug , you can provide your inputs there befoe we start implementing it.20:10
nik90mihir: anchor is the best thing ever in qml..so much better than width and height :D20:10
ahoneybunmihir: ok will do20:10
mihirnik90: i know it is best, but i still get stuck and confused sometimes..20:10
nik90ahoneybun: also feel free to take inspirations from other mobile apps20:11
ahoneybunnik90: yea I was trying to find images of google cal and apple ones20:12
nik90ahoneybun: +20:12
ahoneybunnik90: mihir still working on i20:14
ahoneybun*ity20:14
mihirnik90: i have pushed so you can review that MR.20:15
nik90mihir: sec, I am testing the fix to the visual regression in our visual redesign branch20:16
mihirnik90: okay20:16
nik90mihir: btw the all days events are not shown in the week view20:16
nik90mihir: did you know about that issue?20:16
mihirahh, didn't get that..20:17
nik90mihir: it shows in the day view when appropriate20:17
mihirnik90: it behaves same as in weekview as well ain't it ?20:17
nik90mihir: well I created an all day event on sep 15. It shows up in the day view on sep 15. but in week view I don't see it at all20:18
ahoneybunnik90: mihir http://s7.postimg.org/ywfyvbesb/Event_Details_Design3.png20:19
nik90mihir: I pushed my fix20:19
nik90mihir: testing your color popup mp20:21
mihirahoneybun: how about accomodating time inside the first box,20:23
mihirso i can get all info at one go ?20:23
mihirto be precices , All Primary info ?20:23
ahoneybuninside the Blood Work box mihir ?20:24
mihirhmm yup20:24
ahoneybunok20:24
mihirnik90: i pulled , again the all day event bar in weekview is random , i believe20:27
nik90mihir: random?20:27
mihirand btw, nik90 there is bug if you create all day event , it creates on previous day20:27
mihirnik90: so i have all day events on 14th20:29
mihirhttp://i.imgur.com/ncbYiu5.png20:29
mihirnik90: it doesn't show on weekview http://i.imgur.com/qbKJfT7.png20:30
mihirnik90: but i have an event on 13th and it shows in both view , http://imgur.com/HnuAczf20:30
ahoneybunmihir: coming at you, http://s29.postimg.org/b55xclsnr/Event_Details_Design4.png20:32
nik90mihir: then we might need to revert the hiding of all day events :/ since it is high priority bug to have all days events not being shown in week view20:34
balloonsnik90, landing is re-running, I left comments on the mp20:34
* nik90 looks20:34
mihirhmmm nik90 , so lets don't hide as of now we'll take that look at that later one20:34
nik90yup20:34
nik90balloons: the tests are still being run before autolanding?20:35
balloonsnik90, yep, on the device20:37
balloonsit's not that kind of hack, heh20:37
mihir_ahoneybun: something like this , http://imgur.com/WKXEgzQ20:37
nik90balloons: ok :-)20:37
mihir_so i get all primary info from top :)20:38
ahoneybungot it mihir_20:41
mihir_ahoneybun: :) thatz how i was planning to re-design..20:42
ahoneybunmihir_: http://s27.postimg.org/6kalzpt0j/Event_Details_Design5.png20:47
mihir_ahoneybun: nice better, can you work on fonts size and all ? take your time no hurry20:47
ahoneybunmihir_: yea I was trying I'm using gimp and I don't know the tools to align fonts and such20:48
mihir_nik90: you pushing code for hiding all day events events ?20:51
nik90mihir_: sec, I might have fixed it20:51
mihir_nik90: ohh, okay :)20:52
balloonsnik90, ok, tests ran but returned unstabled20:54
balloonsbah timeouts20:54
* nik90 grabs a really later dinner20:56
nik90brb in 15 mins20:56
ahoneybunmihir_: http://s24.postimg.org/vvjq278r9/Event_Details_Design6.png20:56
* ahoneybun grabs some food as well20:57
mihir_nik90: i may off to bed , will chekc your inputs/fix tomorrow20:57
nik90mihir_: ok20:57
mihir_ahoneybun: :)20:57
nik90gnite20:57
mihir_nik90: gnite :)20:57
mihir_just leave your comments on colorpopup if any20:58
mihir_i forgot to push in ubuntu-cal-dev20:58
ahoneybunmihir_: you like it>20:59
ahoneybun?20:59
mihir_ahoneybun: yeah :)20:59
ahoneybungood21:00
ahoneybunmihir_: want the gimp file?21:00
mihir_ahoneybun: no need as of now.21:00
mihir_i believe , lets get other people inputs and can always make it better21:01
ahoneybunok yea we can always improve it21:01
mihir_also, if you can come up with new that will be again awesome.21:02
ahoneybunwith new?21:02
mihir_ahoneybun: hey , i'll be off to bed, you can get in touch with nik90 if you need any info..see you tomorrow21:02
ahoneybunnew calendar event?21:02
* ahoneybun could really use some lp karma points for his designs lol21:06
balloonsmmm.. karma is yummy21:19
=== salem_ is now known as _salem
renato___nik90, I can confirm that, the alarm is not working for me21:54
nik90renato___: even after the all the EDS files being deleted?21:54
renato___nik90, I am not sure if it stop to work after a flash of even before that21:55
nik90renato___: can you paste your task.ics file perhaps.21:55
renato___nik90, yes I did a flash with wipe21:55
renato___and works on the first day21:55
nik90hmm not sure21:55
renato___but is not working anymore21:55
renato___I can see the alarm on the app, but there is nothing in the indicator21:55
nik90interesting..may be since I keep reflashing phone I don't notice it21:56
nik90I will test it this weekend to confirm21:56
renato___nik90, are you flashing with "wipe" ?21:56
nik90renato___: no21:56
renato___and is working for you?21:57
nik90renato___: yeah it works fine for me everyday21:57
renato___strange21:57
renato___for sure is not working for me21:57
renato___it is not the first21:57
renato___in fact never works :(21:57
nik90renato___: so if you create an alarm to ring in the next 2 mins it won't ring?21:57
renato___let me try tha21:57
renato___that21:57
renato___I have only one alarm now for every day at 7 AM21:58
renato___ok after add a new alarm it appears on the indicator21:58
renato___but only the new one21:58
nik90renato___: can you screenshot your alarm page for me21:59
nik90I want to check if the repeating alarm is saved properly21:59
renato___sure21:59
renato___http://imgur.com/IFsx9OS22:00
renato___this is my indicator: http://imgur.com/nGdvcHX22:01
nik90that's really strange22:03
nik90renato___: can you pastebin your task.ics file.22:04
nik90I will check with charles why it doesn't appear in the indicator22:04
renato___nik90, well very strange22:05
renato___I have two folders in my task dir22:05
renato___drwx------ 2 root   root   4096 Jul 17 15:52 1400264520.14685.0@ubuntu22:05
renato___drwx------ 2 renato renato 4096 Jul 17 15:58 1402930526.1119.0@ubuntu22:05
renato___this looks wrong22:05
charleshurm22:05
nik90yeah why22:05
nik90that could be why22:06
renato___hi charles22:06
charlesrenato___, ~/.config/evolution/sources will tell you which one evolution thinks is the "right" one22:06
charleshi renato___22:06
charlesrenato___, you need more underscores in your name22:06
nik90:)22:06
renato___:D22:06
renato___why I have this folder with root as owner22:07
renato___I can not even go inside of it22:07
charlesrenato, I don't know22:07
charlesI haven't seen that before22:08
charlesjust thinking out loud, maybe if you removed configuration files while evolution was still running22:08
renato___I flashed my device with --wipe some days ago22:08
renato___but for sure I did not change anything in eds folder manually22:08
renato___and I only have that on task folder22:09
renato___-rw------- 1 root   root    117 May 16 15:22 1400264520.14685.0@ubuntu.source22:10
renato___I have the same folder in my sources folder22:10
charlesrenato___, which tasks.ics has the recurring 9:00 alarm? the one  in 14002* or the one in 14029*22:12
renato___let me check22:12
renato___charles, nik90, forget about the folders :D, I was looking in my desktop22:14
renato___:D22:14
renato___sorry22:14
renato___:D22:14
nik90lol22:14
charlesXD22:14
renato___let me get the tasks.ics22:15
renato___http://paste.ubuntu.com/8375422/22:15
renato___charles, nik90, ^^22:15
renato___charles, is that correct?22:16
charleslet's see what happens if I throw that onto my phone and restart evolution + the indicator22:18
nik90renato___: let me compare your task.ics with mine http://paste.ubuntu.com/8375428/22:18
nik90hmm it looks identical22:20
charlesrenato___, nik90, I get the English + 12h version of http://imgur.com/nGdvcHX22:21
charlesI see the 7pm alarm but no recurring alarm22:22
charlesas an aside the timestamp on that alarm would be really hard to read outside22:22
nik90charles: yup that's the issue22:22
nik90charles: renato___ 's task.ics is faulty,, why though22:22
nik90charles, renato___ : not sure if this is related but I noticed today that I create calendar events in the *local* calendar, it shows up as alarms in the indicator.22:25
nik90charles, renato___: If I create calendar events in the google calendar, they appear correctly as calendar events22:25
renato___nik90, I am not using the calendar app yet :D22:26
nik90ah ok22:26
charles(process:16511): Indicator-Datetime-DEBUG: got appointment from 2014-09-16 09:00:00 to 2014-09-16 09:00:00, uid 20140915T225035Z-20381-32011-1350-1@ubuntu-phablet status 1006022:27
charles(process:16511): Indicator-Datetime-DEBUG: adding appointment 'Ubuntu' ''22:27
renato___charles, what that means?22:28
renato___:D22:28
charlesthat's the only instance that shows up22:28
charlesindicator-datetime thinks this alarm has already expired22:28
charlesthat's the only instance that comes back when indicator-datetime calls e_cal_client_generate_instances()22:29
charlesinstead of 5 instances per week22:30
charlesmy first guess is the RRULE's enddate of (time_t)022:31
charlessecond guess is the DUE entry's use of (time_t)-122:32
charlesI think I'll see what happens if I poke those.22:32
charlesrenato___, nik90: yep. If I change X-EVOLUTION-ENDDATE to 2015 instead of 1970, it works22:34
charlesrenato___, nik90: now I'll see what happens if we leave it out altogether22:34
charlesrenato___, nik90: no good. we have to have an X-EVOLUTION-ENDDATE22:37
renato___let me try to create a recursive event on evolution22:38
renato___on desktop22:38
charleslooking at the EDS source now to see what the allowed values of X-EVOLUTION-ENDDATE are, such as a magic value for an endless alarm22:38
renato___charles, http://paste.ubuntu.com/8375540/22:40
renato___this is a task that I created on eds22:40
renato___but there is no way to create recursive tasks on evolution22:41
renato___ as you can see there is no end date field22:41
renato___can you try remove this field from our task.ics22:41
nik90renato___: but charles tried removing that field altogether and that didnt work though22:43
renato___let me try with a event22:43
charlesnope, EDS uses icaltime_from_string(foo) (via icalvalue_new_from_string(ICAL_DATETIME_VALUE, foo) so there are no magic numbers there22:45
charlesok e_cal_recur_obtain_enddate() requires that enddate if count != 022:46
charleswe want it where neither COUNT nor UNTIL are set; in this case, it recurs forever \o/22:47
* charles tries that in practice to confirm the code comments22:47
nik90popey: how's https://imgur.com/zlXeX9l22:47
nik90popey: lighter, striked out indicates past events22:48
popeyhmm22:49
popeynot sure about strikeout22:49
popeylooks messy22:49
nik90popey: any better way to distinguish between the two?22:49
nik90popey: would just making it lighter with italics be sufficient?22:50
popeydim/shade22:50
popeyi wouldn't change font22:50
popeysee google calendar web interface for an example22:50
popeythey just make it lighter coloured.22:50
popeyno other effect22:50
nik90I actually at first created it with different opaque values22:51
nik90however it causes issues when two events occur at the same time22:51
nik90in the calendar app, they overlap over one other...22:51
popeyyeah, opacity will cause overlapping mess with many calendar items at one time22:51
nik90popey: this is what it was first https://imgur.com/rPTgb1d22:51
popeyI think just dimming them is fine22:51
nik90but that causes https://i.imgur.com/nEIXY3y.png22:52
popeyhmm22:52
nik90ok22:52
nik90I will try dimming it more22:52
popeyyeah exactly what i expected22:52
popeydim but not opacity22:52
charlesnik90, renato___: okay the winning recipe is RRULE:FREQ=WEEKLY;BYDAY=MO,TU,WE,TH,FR22:55
nik90charles: :)22:55
nik90charles: so that requires a fix in the SDK?22:56
charlesI think so, yeah22:56
renato___or in the eds backend22:56
renato___I need to check22:56
nik90ok, I will have to explain all this zsombi22:56
nik90to zsombi(22:57
renato___nik90, let me check in eds backend first22:57
nik90renato___: ok22:57
charlesyeah, it's either in the toolkit or in qt5organizer-eds22:57
charlesI haven't looked at that yet, have been reading http://www.kanzaki.com/docs/ical/rrule.html and testing different combos in tasks.ics :)22:57
renato___charles, could you try to create a recurence event on evolution and see how it is stored22:59
renato___my evolution app is not working22:59
charlesrenato___, nik90, ok I suspect part of the problem is coming from qtorganizer5-eds's qtorganizer-eds-engine.cpp's QOrganizerEDSEngine::parseRecurrence()23:03
renato___ https://bugs.launchpad.net/qtorganizer5-eds/+bug/137134123:04
ubot5Launchpad bug 1371341 in Ubuntu Clock App "after a while recurring alarms stop to work" [Critical,New]23:04
charlesthere should be logic in there to set rule->count = 0 for unbounded alarms23:04
renato___should be easy to fix23:04
charlesthe default there is to set rule->count = qRule.limitCount()23:05
charleswhich I'd bet a Virginia Beer is -123:05
renato___:D23:06
* charles digs some more23:06
nik90renato___: is there any fix I need in the clock app side?23:06
nik90I am loving this list https://blueprints.launchpad.net/ubuntu-clock-app/+spec/alarms-rtm23:06
renato___probably note23:06
renato___not23:06
charleshttp://doc-snapshot.qt-project.org/qt-mobility/qorganizerrecurrencerule.html#limitCount23:07
charlesint QOrganizerRecurrenceRule::limitCount () const23:07
charlesReturns the "count" condition specified by the recurrence rule. The default count is -1 (ie. * unlimited)23:07
charlesso we have a semantics difference between QOrganizer and libical here23:07
charlesor rather, with EDS23:07
charlesEDS sees 0 as unlimited, QOrganizer sees -1 as unlimited23:08
charlesI don't understand how this has worked in the past though23:09
charleswe've had recurring alarms work before23:09
renato___I will fix that tomorrow :D23:09
renato___charles, thanks for the help23:10
charlesrenato___, it was fun to do; you're welcome23:11
renato___I will pay you a " Virginia Beer" :D23:11
nik90hehe23:11
charleseven more fun!23:12
renato___I:D23:13
renato___I need to go guys23:13
renato___I will fix that tomorrow23:13
renato___thanks23:13

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