[15:55] <daker_> yo cjohnston
[16:51] <cjohnston> hey daker_
[17:33] <cjohnston> daker_: ping
[17:33] <daker_> cjohnston: yo
[17:34] <cjohnston> daker_: I pushed a couple of changes for LTP
[17:34] <cjohnston> I'm working on some others that will make a couple of things easier that we use in Summit
[17:34] <cjohnston> I was doing this while trying to get a working LTP back up to do reviews for you
[17:34] <daker_> i'll look at them :)
[17:34] <daker_> thanks
[17:35] <cjohnston> daker_: antoher thing that I am working on, best I can tell it works with ltp, is the ability to just update-openids -u chrisjohnston
[17:35] <cjohnston> instead of having to wait for all
[17:36] <daker_> oh yes that's cool
[17:37] <cjohnston> the one css one that I reviewed last night just needs to be put in the new file and I'm pretty sure itll be good
[17:37] <daker_> ok
[17:42] <cjohnston> daker_: how do you do the lint testing and such?
[17:42] <cjohnston> id like to set it up for summit too
[17:43] <daker_> cjohnston: https://bazaar.launchpad.net/~ltp-devs/loco-team-portal/0.2/revision/528
[17:44] <cjohnston> k
[17:46] <daker_> but for me i use Pylinter plugin for Sublime text 2
[17:57] <cjohnston> daker_: I've got myself setup, and am going to do reviews... if you can update things as I go (don't know if you have time) then I'll re-review stuff
[17:57] <daker_> i am with you
[17:57] <cjohnston> cool
[17:58] <cjohnston> just reviewed one.. I think its probably just merge conflicts
[18:00] <daker_> what should i do exaclty ?
[18:01] <daker_> merge with the trunk then fix the conflicts ?
[18:01] <cjohnston> if you can
[18:02] <cjohnston> if not you may have to manually change the code again..
[18:04] <cjohnston> daker_: do you know that there are three failing tests?
[18:04] <cjohnston> (in trunk)
[18:05] <daker_> the guest test ?
[18:05] <cjohnston> i dont remember.. possibly
[18:05] <cjohnston> I ran them yesterday
[18:06] <cjohnston> I need to learn how to do testing so that I can get the tests in summit working, and write tests as i build my new site
[18:07] <cjohnston> ERROR: testAdminIsATeamChecks (teams.tests.ACLChecksTestCase)
[18:07] <cjohnston> Test if check for a more complicated LoCo Team setup works
[18:07] <cjohnston> ERROR: test_update_user_details_long_names (django_openid_auth.tests.test_auth.OpenIDBackendTests)
[18:07] <cjohnston> FAIL: test_anonymous_user (teams.tests.EditTeamTestCase)
[18:08] <cjohnston> ERROR: test_current_site_in_context_after_login (django.contrib.auth.tests.views.LoginTest)
[18:08] <daker_> no those are the openid_auth tests
[18:08] <daker_> or not
[18:16] <cjohnston> daker_: I'm just checking...
[18:16] <cjohnston> "datetime.datetime.now()" doesn't require the import time?
[18:18] <cjohnston> and def create_feed(user, url): doesnt require User?
[18:21] <daker_> where ? articles app ?
[18:27] <cjohnston> i believe so
[18:27] <cjohnston> yes
[18:31] <daker_> cjohnston: now https://code.launchpad.net/~daker/loco-team-portal/fix.978901/+merge/102019
[18:32] <cjohnston> ok
[18:37] <cjohnston> daker_: https://code.launchpad.net/~daker/loco-team-portal/fix.978901/+merge/102019
[18:38] <daker_> which file ?
[18:39] <cjohnston> line 167 of the mp...
[18:39] <cjohnston> I don't remember which file it is
[18:39] <cjohnston> https://code.launchpad.net/~daker/loco-team-portal/fix.identica/+merge/105567  <--- if you give feedback I'll re-review
[18:39] <cjohnston> loco_directory/templates/events/team_event_detail.html
[18:43] <daker_> done
[18:51] <cjohnston> https://code.launchpad.net/~daker/loco-team-portal/fix.978901/+merge/102019
[18:54] <daker_> hmm?
[18:54] <cjohnston> added a second comment to it
[18:54] <daker_> ok
[18:54] <cjohnston> just a couple things to clean up the look of the form
[18:54] <daker_> change what to :
[18:54] <daker_> Change to:
[18:54] <daker_> <div class="row">
[18:54] <daker_>     <section class="span-8">
[18:54] <daker_> ?
[18:55] <cjohnston> <article class="main-content">
[18:56] <daker_> ah ok
[19:00] <cjohnston> ill be back in a couple hours.. back to the beach.. kids are up from naps
[19:01] <daker_> ok
[19:02] <daker_> have fun
[20:50] <cjohnston> daker_: I'm back
[20:56] <daker_> try now https://code.launchpad.net/~daker/loco-team-portal/fix.960695.meetings-app/+merge/108492 cjohnston
[20:56] <daker_> brb
[20:59] <cjohnston> daker_: mhall119_ is ltp still on cranberry?
[21:08] <cjohnston> daker_: http://ubuntuone.com/1PM23ezrCNB2pqSMb6VM1z   does the horizontal line between my 'test' comment and the line with the date look like its a solid line vs all the others being dotted?
[21:10] <daker_> 404
[21:11] <cjohnston> http://ubuntuone.com/1PM23ezrCNB2pqSMb6VM1z  ?
[21:11] <daker_> yes
[21:11] <daker_> 404
[21:12] <cjohnston> hrm
[21:12] <cjohnston> its working for me
[21:12] <cjohnston> i just tried again
[21:13] <cjohnston> http://imgur.com/4HMPm
[21:13] <cjohnston> daker_: ^
[21:13] <daker_> https://one.ubuntu.com/1PM23ezrCNB2pqSMb6VM1z 404
[21:13] <daker_> https
[21:14] <daker_> cjohnston: what are you testing ?
[21:14] <daker_> ah events
[21:14] <cjohnston> thats with https://code.launchpad.net/~daker/loco-team-portal/fix.978901/+merge/102019  I think I see a problem, but I want someone else to confirm
[21:14] <cjohnston> its the event comments
[21:21] <cjohnston> daker_: if you answered, I missed it.. https://code.launchpad.net/~daker/loco-team-portal/fix.960695.articles-app/+merge/108428  for the articles, it doesn't require User and the datetime doesnt require time?
[21:21] <cjohnston> just want to make 100% sure
[21:23] <daker_>  it doesn't require User
[21:23] <daker_> datetime ? where ?
[21:24] <cjohnston> print "[%s] Updating: %s" % (datetime.datetime.now(), feed.title)
[21:24] <cjohnston> I guess it requires datetime not time
[21:25] <daker_> yes and we already import it
[21:27] <cjohnston> I see that now.. approved :-)
[21:29] <daker_> ok
[21:38] <cjohnston> daker_: check your email
[21:40] <daker_> thanks
[21:40] <cjohnston> its getting really annoying
[21:42] <daker_> :)
[21:42] <cjohnston> daker_: it also looks like you need commit messages in LP
[21:42] <daker_> ?
[21:42] <daker_> ah yes yes
[21:43] <cjohnston> daker_: https://code.launchpad.net/~chrisjohnston/loco-team-portal/update-openids-update/+merge/112914  <-- when you test this, ./manage.py update-openids -u chrisjohnston   <-- the one user meathod
[21:43] <cjohnston> method
[21:44] <cjohnston> please ensure though that the old way works.. I have been having internet connection troubles and cant keep a solid connection long enough to go through all of it
[21:44] <daker_> maybe you have to add an except
[21:44] <daker_> if this user = User.objects.get(username=username) fails ?
[21:45] <cjohnston> doing the one user works.. its when I try to update all of the openids that I lose internet
[21:45] <cjohnston> its a conenction problem and not a script problem, I'm certain, but I would like it properly eested
[21:46] <cjohnston> tested
[21:46] <cjohnston> I'll be home in 5 minutes, I hope to get back on in an hour or so and work more
[21:47] <daker_> cjohnston:
[21:47] <daker_> http://paste.ubuntu.com/1068434/
[21:48] <cjohnston> but its updating the openid
[21:49] <cjohnston> doesnt matter if the user already exists
[21:52] <daker_> no if the user doesn't exist the get function will raise a traceback
[21:53] <daker_> The DoesNotExist exception is raised whenever the lookup with get() fails
[21:54] <daker_> and the command will not execute the
[21:54] <daker_> launchpad.set_user_openid(user, force=True)
[21:55] <cjohnston> gotcha
[21:55] <cjohnston> i was thinking you were trying to do something backwards
[21:56] <daker_> :)
[21:56] <daker_> and i am lost
[21:57] <cjohnston> I dont think you are
[21:59] <cjohnston> are you saying replace lines 24-29 of the MP with your paste?
[22:01] <daker_> yes
[22:02] <daker_> i rejected this https://code.launchpad.net/~daker/loco-team-portal/fix.960695.events-app/+merge/108462
[22:02] <daker_> will work on another one
[22:02] <daker_> i hate conflicts
[22:03] <cjohnston> ok
[22:09] <daker_> cjohnston: https://code.launchpad.net/~daker/loco-team-portal/fix.960695.common-app/+merge/108439
[22:47] <daker> cjohnston: https://code.launchpad.net/~daker/loco-team-portal/fix.960695.events-app2/+merge/112916
[22:52] <cjohnston> ok daker..
[22:53] <cjohnston> I'm home.. had dinner.. back on my desktop
[22:53] <cjohnston> lets see what we can work out
[22:58] <cjohnston> daker: doing the "try" thing doesnt work when you don't specify a user (i.e. doing all
[22:58] <cjohnston> )
[22:59] <daker> no i think there is something wrong
[23:01] <cjohnston> http://paste.ubuntu.com/1068536/
[23:02] <daker> cjohnston: http://paste.ubuntu.com/1068538/ ?
[23:04] <daker> cjohnston: the sys.exit('Seems that this user does not exist') should inside the except
[23:04] <cjohnston> that looks better
[23:04] <cjohnston> seems to be running
[23:19] <daker> cjohnston: like this http://paste.ubuntu.com/1068561/
[23:20] <daker> look at yours https://bazaar.launchpad.net/~chrisjohnston/loco-team-portal/update-openids-update/view/head:/loco_directory/common/management/commands/update-openids.py
[23:20] <cjohnston> ya.. i havent pushed it again
[23:20] <daker> ok
[23:20] <cjohnston> im back on good internet so im testing updating all
[23:21] <daker> ok time for bed :)
[23:21] <cjohnston> I feel the same way after a day at the beach
[23:21] <cjohnston> sleep well
[23:21] <daker> :)