Opened 18 years ago
Closed 17 years ago
#3483 closed (fixed)
Document AUTH_PROFILE_MODULE
Reported by: | Owned by: | nobody | |
---|---|---|---|
Component: | Documentation | Version: | dev |
Severity: | Keywords: | ||
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
The AUTH_PROFILE_MODULE isn't documented, the whole "Profile" support is undocumented except for a small reference to the 'get_profile' method in [source:django/trunk/docs/authentication.txt authentication docs] maybe it should be documented somewhere, the best source of information I could find about it is this b-list post, there is also this wiki page which should be updated to reflect this new setting maybe with a note to the documentation if it gets added.
On the code you can get a few more information: [source:django/trunk/django/contrib/auth/models.py here]
Attachments (2)
Change History (13)
comment:1 by , 18 years ago
comment:2 by , 18 years ago
Triage Stage: | Unreviewed → Accepted |
---|
comment:4 by , 17 years ago
Why is still undocumented? Are you planning to change it? I'm gonna start a big project and I need this feature, but please tell me if you plan to change this.
Is the setting AUTH_PROFILE_MODULE the right way to get it working?
comment:5 by , 17 years ago
Why haven't we done it? Lack of time. Feel free to help out. If it's in the book it's not likely to change.
follow-up: 7 comment:6 by , 17 years ago
I'll start to work on it today or on Friday. I hope I can submit the patch before next week.
comment:7 by , 17 years ago
Replying to adrian@adrima.es:
I'll start to work on it today or on Friday. I hope I can submit the patch before next week.
By the way, I'm the same user as aribao@…, I just wrote other email.
adrian@… = aribao@… :-)
by , 17 years ago
Attachment: | authentication.profile.diff added |
---|
Patch adding "Extendig the profile system" to the documentation
comment:9 by , 17 years ago
Has patch: | set |
---|
I have documented the extendable profile system. Most of the text comes from the djangobook because I'm Spanish and I thought it would be better to copy it rather than write it by myself.
Important: The patch MUST be reviewed by some native English speaker before applying it.
I hope it's ok.
by , 17 years ago
Attachment: | authentication.profile-v2.diff added |
---|
added "login required" to the view, and other small changes
comment:10 by , 17 years ago
Thank you! Jacob's just checking out the legalities of what can/can't be merged in from the Djangobook (we want to be very careful here for obvious reasons), but once that's sorted out we'll deal with it :)
comment:11 by , 17 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
The Django book documents it: http://www.djangobook.com/en/beta/chapter12/#cn226
That can probably be rolled in to the official docs, per plans to do that with most of the stuff in the book.