Merge lp:~ronnie.vd.c/loco-team-portal/611304 into lp:loco-team-portal
| Status: | Merged | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Approved by: | Michael Hall on 2010-12-30 | ||||||||
| Approved revision: | 352 | ||||||||
| Merged at revision: | 352 | ||||||||
| Proposed branch: | lp:~ronnie.vd.c/loco-team-portal/611304 | ||||||||
| Merge into: | lp:loco-team-portal | ||||||||
| Diff against target: |
159 lines (+131/-13) 2 files modified
loco_directory/teams/management/commands/import-live-data.py (+116/-0) loco_directory/venues/models.py (+15/-13) |
||||||||
| To merge this branch: | bzr merge lp:~ronnie.vd.c/loco-team-portal/611304 | ||||||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Chris Johnston | 2010-12-20 | Approve on 2010-12-23 | |
|
Review via email:
|
|||
Description of the Change
Importing data from the Live (loco.ubuntu.com) server and put it in the local database
| Michael Hall (mhall119) wrote : | # |
Getting attendees ...
Traceback (most recent call last):
File "./manage.py", line 11, in <module>
execute_
File "/home/
utility.
File "/home/
self.
File "/home/
self.
File "/home/
output = self.handle(*args, **options)
File "/home/
return self.handle_
File "/home/
del old_su['_state']
KeyError: '_state'
| Ronnie (ronnie.vd.c) wrote : | # |
it could be that not every user has a '_state' key. I included a check for the key, before deleting it.
It should work now


Looks good to me. Would like a second opinion before merging though please.