Opened 14 years ago
Closed 12 years ago
#16346 closed New feature (wontfix)
Hide additional fields in comment form when user is authenticated.
Reported by: | krisys | Owned by: | nobody |
---|---|---|---|
Component: | contrib.comments | Version: | 1.3 |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Design decision needed | |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
I was using the django comments framework, but i realized that it is asking for name and few other fields even when the user is logged in. I went through few blogs on the net to find many suggestions which ask for modification in the template. But i feel it would be nice to have a constant COMMENT_ALLOW_UNAUTHENTICATED_USERS, so that only unauthenticated users get to see the name, email, url fields before commenting. while authenticated users will just see the comment box (which is a more common use case than the previous). This will reduce some burden on the users to do more work in the templates.
Additionally, we can show some of the fields if the user has not set it in his user profile, like name, email(if the user was created from the admin).
Change History (2)
comment:1 by , 14 years ago
Triage Stage: | Unreviewed → Design decision needed |
---|---|
Type: | Uncategorized → New feature |
comment:2 by , 12 years ago
Resolution: | → wontfix |
---|---|
Status: | new → closed |
django.contrib.comments
has been deprecated and is no longer supported, so I'm closing this ticket. We're encouraging users to transition to a custom solution, or to a hosted solution like Disqus.
The code itself has moved to https://github.com/django/django-contrib-comments; if you want to keep using it, you could move this bug over there.
This could most be achieved by making a custom CommentForm (and probably a view, 'cause you have to pass the user to the form), which means making a custom app, which seems like a overkill for something simple like this. Global setting is most likely not a good idea.