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