search

pinax / symposion

a Django project for conference websites
BSD 3-Clause "New" or "Revised" License
299 stars 146 forks source link

Presentation detail link proposal #106

Closed miurahr closed 9 years ago

miurahr commented 9 years ago

Implemented #91

paltman commented 9 years ago

@jtauber was there a specific reason for the copying over of these fields? Could they ever change on one but not the other? If not, I think this is a decent change.

miurahr commented 9 years ago

@jtauber I've tested this code on my project and it does not use review app. I'm afraid whether review app and this modification work well or not.

miurahr commented 9 years ago

rebased.

miurahr commented 9 years ago

This PR make a bug to reverse-reference from Speaker to Presentation. Because it drop speaker field from Presentation and add a function to refer Proposal.speaker. This cause a problem.

miurahr commented 9 years ago

I'd like to mark it Invalid and close.