[04:59] hola [04:59] alguien aqui?? === head_v is now known as head_victim [12:37] ping mhall119 [15:16] Ronnie: pong [15:16] mhall119: hey ... [15:17] i was working on a 'supportmap' for django foundations [15:17] but ran into some troubles [15:17] i also say you made some bugreports to it [15:18] for example, howto handle the urls.py into the global urls.py [15:19] and the media directory, how to add items in there [15:20] mhall119: ^ [15:20] what do you mean by 'supportmap'? [15:21] each app would have it's own urls.py and ./media/ [15:22] we can provide a global urls.py that links to all the apps' urls.py [15:22] probably commented out, so you just uncomment what you'll be using [15:24] something I've done at work is to define a MEDIA_ROOT in each app's __init__.py [15:24] by supportmap, i mean a google map, where users can add a marker where they live, and add some contact information and otehr stuff [15:24] new users can get in contact with nearby ubuntu usrs to ask for help [15:25] its a bug succes in the dutch community [15:25] can you make a generic API, so developers can add any items to a google map? [15:25] I can see a user-specific one being useeful for locos [15:25] but non-loco webapps would probably also have a use for it [15:25] if they can define other items [15:25] mhall119: the generic API is already written (it needs to be extended and optimized tought) [15:26] cool, lets get that into the django foundations [15:26] i use same code (the own jquery maps plugin) on loco as on the supportmap code [15:27] mhall119: at the moment i get the error: ./manage.py pullapps => Unknown command: 'pullapps' [15:27] are you on the django foundations developers team? [15:27] no [15:28] what's your LP id? [15:28] ronnie.vd.c [15:29] added [15:29] i see [15:29] do you have bzr_apps locally? [15:30] no, do i need to to pull apps? [15:30] yeah, it can't pull itself before it's there [15:31] bootstrapping is still a bit of an issue [15:31] I'm working on ways to make that easier [15:32] the way Django handles INSTALLED_APPS doesn't help matters [15:33] mhall119: i branched now all the apps in foundations, (all in the folder src) but the command still not exists [15:33] is bzr_apps in your INSTALLED_APPS? [15:34] if you're using the foundations settings.py, it should [15:35] mhall119: in settings: import bzr_apps (but the folder is named bzr-apps ) [15:35] oh right [15:36] forgot that python doesn't like hyphens in package names [15:36] if i rename the folder, it works [15:37] I'll rename the series [15:37] ok great [15:37] grrr, except it seems launchpad doesn't like underscores :( [15:37] fail [15:38] hmm, then we need to change the commend to 'bzr branch lp:ubuntu-django-foundations/bzr-apps bzr_apps' [15:38] yeah [15:38] and if the bzr_apps import fails, show that command [15:39] or dont use _ or - at all [15:39] actually, if you use the 'bootstrap' management command, it should do it correctly for you, try that [15:40] bootstrap is in ./common/ [15:40] ah, ill try [15:40] * mhall119 needs to make a README and INSTALL files [15:40] indeed l) [15:43] we need to copy the local_settings stuff from loco too [15:47] good point [16:00] mornin [16:00] hey cjohnston :D [16:05] whats goin on today [16:05] a lot if django foundations ideas [16:07] and i see my merge request are almost brought back to 0 ;) [16:10] lol [16:11] I have three for ld [16:12] cjohnston: can you reproduce the bug for the calendar banch? [16:12] daker could not, like me [16:14] Ronnie: could you look into https://code.launchpad.net/~chrisjohnston/loco-directory/699692/+merge/45477 [16:14] its the same as your fix for https://code.launchpad.net/~ronnie.vd.c/loco-directory/697693/+merge/45263 [16:14] so i think the bug may be introduced with both [16:15] cjohnston: ill have a look [16:15] im starting the process for downloading to check the calendar thing again [16:15] on 3g today... so itll be a while [16:17] hmm, downloading from Loco is not the longest time [16:17] it is the database interaction ;) [16:18] Downloading the branch is up to more than 3 minutes [16:19] hmm, ok didnt notive it is that much, here is is less than a sec [16:19] cjohnston: disney good fun ? [16:19] ill have to eat in a minute, then ill look into the feedback of daker. i guess that also can create new bugs [16:20] bbuussyy [16:20] czajkowski: ^ [16:22] cjohnston: :) [17:02] Ronnie: im still importing.. :-/ been doing it since about the time i said bbuussyy [17:15] hi [17:15] pleas help me [17:16] whats up yerkin [17:16] i install ubuntu 10.10 server [17:16] then i try install yum [17:17] sorry my bad engl [17:17] sorry for my bad engl [17:17] Try #ubuntu-server for server help. [17:18] i write apt-get install yum [17:18] ok thank you [17:19] np [17:21] Ronnie: I'm still getting the error on the calendar merge [17:22] cjohnston: can you write step by step what you did? [17:23] branch, syncdb, migrate, init-ld, import-live-data, runserver, merge [17:28] then i click on any of the little calendar icons [17:29] oke, ill test in a moment, still looking into the problem of the copy [17:30] the bug that can arise fron dakers solution, is that 2 events are copied at 'the same time' (i.e. before its saved to the database). they both get the same id [17:33] what about from the initial patch [17:35] i can guess whats going on [17:35] the view is executed twice, load en POST [17:36] ic [17:36] in both a copy is made (.save()) but only in the POST, the team is linked [17:36] i have to look into the _new code how to solve it [17:36] but im now testing the ical [17:37] k [17:52] cjohnston: i can reproduce the error indeed [17:53] :-) [17:53] I guess that's a good thing [17:54] indeed, now i can start debugging :D [18:01] tell me how to specify a proxy [18:02] ubuntu 10.10 [18:05] Try a support channel or google please [18:11] cjohnston: fixed the NoneType error for ical [18:11] cool [18:12] did you push it? [18:14] yes [18:15] the problem was an empty description in an event [18:16] ic [18:17] hi [18:18] hi [20:30] Ronnie, how ? [21:38] daker, how what? [21:38] cjohnston: i can reproduce the error indeed [21:39] branch, syncdb, migrate, init-ld, import-live-data, runserver, merge [21:39] Ronnie, is it the ical error ? [21:39] yes [21:39] cjohnston: did you test it after my new push? [21:40] because your comment is afterwards, but it should be fixed in the new version [22:33] Ronnie: didnt notice the new commit [22:33] one sec [22:35] mhall119: ping [22:35] Ronnie: no error [22:36] cjohnston: the copy bug is harder than i expected. because TeamEvent subclasses from BaseEvent. i think i need the help from mhall119 to fix this [22:36] ok [22:37] cjohnston: is there another bug that needs to be solved before the next release? [22:37] the meeting bugs are stoppers [22:38] mhall119: any advise on this? http://docs.python.org/library/copy.html [22:40] cjohnston: the last one is hard: https://bugs.launchpad.net/loco-directory/+bug/688064 [22:40] Launchpad bug 688064 in loco-directory "Add meetings to 'upcoming events' on team's pages (affects: 1) (heat: 4)" [Wishlist,Triaged] [22:40] daker: ping [22:41] Yo [22:43] do you have any advise on the copy bug? [22:43] bug #692121 should be fixed before release, bug #686268 [22:43] Launchpad bug 692121 in loco-directory "Team event has been unlinked to loco team on loco.ubuntu.com (affects: 1) (heat: 5)" [High,Confirmed] https://launchpad.net/bugs/692121 [22:43] Launchpad bug 686268 in loco-directory "Add the ability to add agenda items (affects: 1) (heat: 5)" [Wishlist,In progress] https://launchpad.net/bugs/686268 [22:43] or the upcoming events page -> include meeting [22:44] 692121 is hard to solve, because the cause in not clear [22:44] bug 692121 [22:44] Launchpad bug 692121 in loco-directory "Team event has been unlinked to loco team on loco.ubuntu.com (affects: 1) (heat: 5)" [High,Confirmed] https://launchpad.net/bugs/692121 [22:44] yes [22:44] Events page should not list meetings.. thats what the meeting page is for [22:45] on the team page, it should list events and meetings [22:45] cjohnston: 686268 is already merged, why isnt it "Fix committed" [22:45] Ronnie: it only has the backend [22:45] it doesnt have the ui [22:49] Ronnie: in my mind, 686268 should behave like an event comment.. If you click add agenda item (or something similar), you get a text box at the top of the screen (just like the comment box) and then it records the user who made the comment, the time, etc [22:51] do you understand what im sayin Ronnie ? [22:52] yes [22:52] :-) [22:54] ill see what i can do tomorrow, got enough for today ;) [22:54] daker: would you mind running another test on the ical branch since changes were made? if you approve ill merge [22:54] sounds good Ronnie [22:55] With the bugs that need to be done for the release, I'm too lost to be of much assistance [22:55] indeed, the complexity is increasing at the end of the road [22:56] altough the ui shouldnt be that difficult. but i hate css stuff ;) [22:56] cjohnston, tell me the bug id pls ツ [22:56] I'm thinking CSS should be fairly easy, as it should be able to use the css from comments [22:56] daker: https://code.launchpad.net/~ronnie.vd.c/loco-directory/623288/+merge/44127 [23:00] Ronnie: could you also take a quick look at this one please bug 696661 [23:00] Launchpad bug 696661 in loco-directory "Expires Date / Date Expires does not make sense (affects: 1) (heat: 6)" [Low,In progress] https://launchpad.net/bugs/696661 [23:01] sure, looking is OK ;) [23:01] sorry... that was actually supposed to be for daker.. but you can review it too [23:01] :-) [23:02] cjohnston: hmm, not much that could go wrong with that merge. but ill do a small test to be sure [23:03] ty [23:06] cjohnston, it works for me [23:09] cjohnston, i'll review the other tomorrow it's 23:10 and i need to go to sleep ツ [23:13] cjohnston: on the page http://localhost:8000/teams/ubuntu-nl/edit its not translated [23:13] its still "Date Expires" [23:15] other pages are correct [23:23] Ronnie: can you bring me up to date on what the "copy bug" is? [23:23] wasn't the copy event feature working previously? [23:23] mhall119: sure [23:24] it had a glitch, there were 2 events created. one without a team [23:24] i even guess that the old (where its copied from) could be changed [23:25] Ronnie: how did it create 2? [23:25] i tried different approaches, but all failed (create a __dict__ from the model, delete the id field, and give the dict as initial) [23:25] mhall119: because the view is called twice, the first copy, and later the POST [23:26] right, but it only get's saved on POST, not on GET [23:26] 2nd method, use python deepcopy => the original event loses some information [23:27] there was needed a save() also for the GET, because if it has no id, the manytomany rel cannot be linked [23:28] crashed [23:29] hang on, it had been working [23:29] i thought by using initial=event.__dict__ to solve it [23:29] but the datetime are not inserted into the form [23:30] okay, let's back this up a bit [23:30] GET shouldn't need the new event created [23:30] it should just use the old event to pre-populate the new event form [23:31] it used to work.. then some change (i forget what) broke it.. [23:31] well then let's figure out what that change was [23:32] Ronnie: expires_date = models.DateField(_("Reapproval Date"), null=True, blank=True) doesnt change the date expires? [23:32] mhall119: i guess your right with GET/POST [23:32] ill do a test in a minute [23:33] cjohnston: looks like it didnt, is it hardcoded in the template? [23:33] yes [23:35] Ronnie: do you remember what broke the copy [23:35] cjohnston: did it broke, or wasnt it tested properly? [23:37] it works in the live version [23:39] okay, simple fix to the copy event [23:39] ? [23:40] mhall119: your idea seems to work here [23:42] you just needed to get a copy of the teams from the event before you delete the id/pk [23:43] http://pastebin.com/GCx0uba2 [23:47] mhall119: what abount this solution: https://code.launchpad.net/~ronnie.vd.c/loco-directory/697692-2/+merge/45654 [23:47] cjohnston: can you test it? [23:48] cjohnston: i cant find a copy feature in the live version [23:48] Ronnie: I don't think that will work [23:48] why not? [23:48] form.save() will save it's instance [23:49] It may be aware of the id fields and set them before saving [23:49] Ronnie: there is a copy team event [23:50] cjohnston: maybe i have not the rights to make a copy (altough i have rights to create new) [23:50] then you should [23:50] I see 'back' 'comment' 'edit details' 'delete' 'copy' 'ical' [23:51] in the secondary nav [23:52] lol, was looking at the wrong page (team page). duh [23:52] i think its getting late ;) [23:53] okay Ronnie, it looks like your fix will work, the form doesn't have a separate copy of the event id [23:54] mhall119: it indeed looks like its a reference to the actual model [23:54] I'd put it before form.is_valid() though [23:54] though I don't think it matters [23:54] cjohnston: team_update.html there are still some hardcoded strings, including "date expires" [23:55] mhall119: what happens if the form is not valid? [23:55] coulnd that break things? [23:56] it'll display the page again with the form as it is [23:56] like I said though, it shouldn't matter either way [23:56] oke, great [23:58] mhall119: i see my merge is missing the teams=... in meetings/views.py [23:58] can you add them too, and retest?