Use badges for members list

ncsu-stars / Stars-CMS

A CMS written in Django for the STARS SLC at NCSU

http://research.csc.ncsu.edu/stars

BSD 2-Clause "Simplified" License

7 stars 2 forks source link

Use badges for members list #13

Closed mdraelos closed 12 years ago

mdraelos commented 12 years ago

Feature

Simple text listing of members should be replaced with member badges.
Badges should consist at least of member picture, name, and projects.
Badges should not completely duplicate member profile pages.

ewhitmire commented 12 years ago

Is there any reason why I can't add a ManyToMany relationship from Project to Member going through ProjectMember? This would make project iteration so much easier. No db changes required.

mdraelos commented 12 years ago

I don't see why not. That change in 60830a2ee06f6fafaeb5769b9517b5ebbd4621f7 broke projects/projects.html:37, but is easily fixed by replacing projects.members.all with projects.projectmembers_set.all. I'll let you make that change in your next commit.

I like the way the member badges are heading.

mdraelos commented 12 years ago

Commit 60830a2ee06f6fafaeb5769b9517b5ebbd4621f7 also breaks the ProjectForm (automatically generated from the Project model), which in turn breaks editing of Projects.

mdraelos commented 12 years ago

Commit a16e4035135f7eb58b304ff6732995bd2731bb31 fixes the broken ProjectForm by implementing a save() method that correctly manages Project.members.