Merge lp://staging/~ris/loco-team-portal/fix-552762 into lp://staging/loco-team-portal

Proposed by Robert Sajdok
Status: Needs review
Proposed branch: lp://staging/~ris/loco-team-portal/fix-552762
Merge into: lp://staging/loco-team-portal
Diff against target: 1729 lines (+1420/-11) (has conflicts)
14 files modified
Makefile (+1/-1)
loco_directory/events/forms.py (+101/-5)
loco_directory/events/migrations/0010_auto__chg_field_attendee_guests.py (+184/-0)
loco_directory/events/migrations/0011_auto__add_field_teamevent_created_by.py (+185/-0)
loco_directory/events/migrations/0012_auto__add_teameventvenue.py (+196/-0)
loco_directory/events/migrations/0013_historical_venue.py (+195/-0)
loco_directory/events/migrations/0014_auto__del_field_teamevent_venue.py (+190/-0)
loco_directory/events/models.py (+10/-2)
loco_directory/events/views.py (+114/-3)
loco_directory/media/css/venues.css (+10/-0)
loco_directory/media/js/jquery.formset.js (+170/-0)
loco_directory/media/js/venues.js (+3/-0)
loco_directory/templates/events/team_event_new.html (+26/-0)
loco_directory/templates/events/team_event_update.html (+35/-0)
Text conflict in loco_directory/events/forms.py
Conflict adding file loco_directory/events/migrations/0010_auto__chg_field_attendee_guests.py.  Moved existing file to loco_directory/events/migrations/0010_auto__chg_field_attendee_guests.py.moved.
Conflict adding file loco_directory/events/migrations/0011_auto__add_field_teamevent_created_by.py.  Moved existing file to loco_directory/events/migrations/0011_auto__add_field_teamevent_created_by.py.moved.
Text conflict in loco_directory/events/views.py
Text conflict in loco_directory/templates/events/team_event_new.html
Text conflict in loco_directory/templates/events/team_event_update.html
To merge this branch: bzr merge lp://staging/~ris/loco-team-portal/fix-552762
Reviewer Review Type Date Requested Status
Adnane Belmadiaf Needs Fixing
Chris Johnston Needs Fixing
Review via email: mp+142553@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
Chris Johnston (cjohnston) wrote :

Please clean up the merge conflicts.

review: Needs Fixing
Revision history for this message
Robert Sajdok (ris) wrote :

> Please clean up the merge conflicts.
How can I do this on my local working directory?

Revision history for this message
Robert Sajdok (ris) wrote :

I've done this way:
$ bzr merge https://code.launchpad.net/~ltp-devs/loco-team-portal/0.2
I am working with conflicts.

555. By Robert Sajdok

Resolve conflicts

Revision history for this message
Robert Sajdok (ris) wrote :

> Please clean up the merge conflicts.
done, please review

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

> done, please review
Thanks Ris for your work, i'll take a look at your MP this week.

Revision history for this message
Robert Sajdok (ris) wrote :

Any progress?

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

Hi Robert, this MP still need more work, we need to add some lines of codes to keep the relation between the venues and the events, i mean this line :

226 + # Deleting field 'TeamEvent.venue'
227 + db.delete_column('events_teamevent', 'venue_id')

we need to store in memory all venues/events relatiosn before then create them once the new table TeamEventVenue is created.

review: Needs Fixing
556. By Robert Sajdok

South Changes for team event venue.

Revision history for this message
Robert Sajdok (ris) wrote :

> Hi Robert, this MP still need more work, we need to add some lines of codes to
> keep the relation between the venues and the events, i mean this line :
>
> 226 + # Deleting field 'TeamEvent.venue'
> 227 + db.delete_column('events_teamevent', 'venue_id')
>
> we need to store in memory all venues/events relatiosn before then create them
> once the new table TeamEventVenue is created.

I added it, please check it.

Unmerged revisions

556. By Robert Sajdok

South Changes for team event venue.

555. By Robert Sajdok

Resolve conflicts

554. By Robert Sajdok

LP: #552762: "LD needs to be able to allow for adding multiple locations for one event"
Add image files and migration.

553. By Robert Sajdok

LP: #552762: "LD needs to be able to allow for adding multiple locations for one event"

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