Summit

Merge lp://staging/~mhall119/summit/create-attendee-for-bp-usernames into lp://staging/summit

  1. create-attendee-for-bp-usernames
  2. Merge into trunk
Proposed by Michael Hall
Status: Merged
Approved by: Chris Johnston
Approved revision: 551
Merged at revision: 549
Proposed branch: lp://staging/~mhall119/summit/create-attendee-for-bp-usernames
Merge into: lp://staging/summit
Diff against target: 177 lines (+82/-26)
3 files modified
summit/schedule/models/meetingmodel.py (+5/-24)
summit/schedule/models/summitmodel.py (+21/-1)
summit/schedule/tests/meeting_model.py (+56/-1)
To merge this branch: bzr merge lp://staging/~mhall119/summit/create-attendee-for-bp-usernames
Reviewer Review Type Date Requested Status
Chris Johnston Approve
Adnane Belmadiaf Approve
Review via email: mp+163626@code.staging.launchpad.net

Commit message

Creates Attendee records if no matching record exists when importing data from Launchpad

Description of the change

Creates Attendee records if no matching record exists when importing data from Launchpad

To post a comment you must log in.
lp://staging/~mhall119/summit/create-attendee-for-bp-usernames updated
551. By Michael Hall

Don't attempt to lookup or create an Attendee if no username is provided

Revision history for this message
Adnane Belmadiaf (daker) wrote :

+1, i suppose the tests are OK

review: Approve
Revision history for this message
Chris Johnston (cjohnston) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches