Opened 9 years ago

Closed 9 years ago

#4818 closed (fixed)

Databrowse - missing step?

Reported by: kevin@… Owned by: Adrian Holovaty
Component: contrib.databrowse Version: master
Severity: Keywords: databrowse
Cc: Triage Stage: Design decision needed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

First, THANK YOU, THANK YOU -- I love Django!

Step 2. READS:
from django.contrib import databrowse

databrowse.site.register(SomeModel)
databrowse.site.register(SomeOtherModel)

I think instead of:
from django.contrib import databrowse

maybe it should read:
from someapp.models import *

Where 'someapp' is the name of the application. The documentation includes in step 3. the directive:
from django.contrib import databrowse

Change History (3)

comment:1 Changed 9 years ago by Chris Beaven

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Triage Stage: UnreviewedDesign decision needed

It's still important to import databrowse in step two, but perhaps it could read:

from django.contrib import databrowse
from myapp.models import SomeModel, SomeOtherModel

databrowse.site.register(SomeModel)
databrowse.site.register(SomeOtherModel)

I'm unsure if it's really necessary though.

comment:2 in reply to:  1 Changed 9 years ago by miguel.filho@…

I think it's nice this way.

comment:3 Changed 9 years ago by Adrian Holovaty

Resolution: fixed
Status: newclosed

(In [5866]) Fixed #4818 -- Improved 'Step 2' example in docs/databrowse.txt. Thanks, kevin@…

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