Skip to content
This repository has been archived by the owner on Aug 26, 2022. It is now read-only.

fix bug 1119545 - hide website fields for "new" users #3084

Conversation

groovecoder
Copy link
Contributor

This approach simply hides the website fields. We can re-visit this if we decide instead to use rel="nofollow" in bug 820555

@@ -187,6 +187,10 @@ def wiki_activity(self):
return (Revision.objects.filter(creator=self.user)
.order_by('-created')[:5])

@property
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make this a cached_property

@jezdez
Copy link
Contributor

jezdez commented Feb 19, 2015

LGTM other than the comments.

@jezdez
Copy link
Contributor

jezdez commented Feb 19, 2015

Correction, since this feature is considered part of building a "trust system" like the mozillians.org vouching system in https://bugzilla.mozilla.org/show_bug.cgi?id=1119545 I consider this whole idea not thought through yet. This smells like a drive-by work from @groovecoder without going into details what we strive to do with "trust" on MDN. I r- this to prevent feature creep.

@groovecoder groovecoder force-pushed the disable-spam-attracting-fields-1119545 branch from f92dabf to 0e6ecf7 Compare February 19, 2015 14:45
@groovecoder
Copy link
Contributor Author

I went ahead and made the updates (I was already done with them) and pushed them up here to GitHub to keep them around. I'll close this and start a thread about an MDN trust system.

@groovecoder
Copy link
Contributor Author

Note: please follow discussion in https://groups.google.com/forum/#!topic/mozilla.dev.mdn/0r0ZUP0nA1A

@groovecoder groovecoder deleted the disable-spam-attracting-fields-1119545 branch September 1, 2015 13:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants