Closed
Bug 967169
Opened 10 years ago
Closed 10 years ago
Merge apps.tags.models.Tag and apps.tags.models.TagStat
Categories
(Marketplace Graveyard :: Code Quality, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
2014-02-11
People
(Reporter: mat, Assigned: mat)
References
Details
TagStat model only has 2 fields: a OneToOneField to Tag, set as primary_key, and num_addons, an IntegerField. We do some crazy signals stuff to create TagStat instances dynamically, we should just stop and merge the two in the Tag model. This blocks bug 850885 - we get weird failures that are hard to debug with 1.6 with the current models.
Assignee | ||
Updated•10 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•10 years ago
|
||
Not sure what STR to provide for this. This is an internal re-factoring that might affect anything that uses tag on AMO or Marketplace. I've looked at https://addons-dev.allizom.org/en-US/firefox/tags/top and it seems fine. If you can think of any other test to perform, do it :)
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2014-02-11
Comment 2•10 years ago
|
||
I also checked adding a tag to an app and confirmed that it shows upon searching for that tag.Removing the tag and triggering an ES reindex removed the app from the search results.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•