Opened 13 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 Łukasz Rekucki, 13 years ago

Triage Stage: UnreviewedDesign decision needed
Type: UncategorizedNew feature

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.

comment:2 by Jacob, 12 years ago

Resolution: wontfix
Status: newclosed

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.

Note: See TracTickets for help on using tickets.
Back to Top