Merge lp:~dylanmccall/harvest/bugfix-lp-587948 into lp:harvest

Proposed by Dylan McCall
Status: Merged
Approved by: James Westby
Approved revision: 181
Merged at revision: 185
Proposed branch: lp:~dylanmccall/harvest/bugfix-lp-587948
Merge into: lp:harvest
Diff against target: 13 lines (+3/-0)
1 file modified
harvest/opportunities/models.py (+3/-0)
To merge this branch: bzr merge lp:~dylanmccall/harvest/bugfix-lp-587948
Reviewer Review Type Date Requested Status
Daniel Holbach Approve
Review via email: mp+26440@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Daniel Holbach (dholbach) wrote :

I'm happy for this to get merged, but I feel to fix the use-case mentioned in the bug report, we'd need to update ./harvest/opportunities/admin.py too.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'harvest/opportunities/models.py'
2--- harvest/opportunities/models.py 2010-03-08 16:32:02 +0000
3+++ harvest/opportunities/models.py 2010-05-31 18:11:21 +0000
4@@ -14,6 +14,9 @@
5 class PackageSet(models.Model):
6 name = models.SlugField(_("Name"), max_length=40)
7
8+ def __unicode__(self):
9+ return self.name
10+
11 @models.permalink
12 def get_absolute_url(self):
13 return ('packagesets', [self.name])

Subscribers

People subscribed via source and target branches

to all changes: