#8936: Add view (read-only) permission to admin
-------------------------------------+-------------------------------------
Reporter: simon | Owner: Olivier
Type: New feature | Status: closed
Component: contrib.admin | Version: master
Severity: Normal | Resolution: fixed
Keywords: | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Comment (by Olivier):
Replying to [comment:48 Tim Graham]:
> I'm okay with merging this tomorrow, although I find the tests a bit
lacking. In particular, many of the template changes are untested. I added
some `assertContains()` assertions but more are needed. This could be done
after the alpha release, I suppose.
Thanks for the merge ! Yes I think I could spare some time to add some
tests. But would you have some concrete examples of tests you'd like to
have added ?
--
Ticket URL: <https://code.djangoproject.com/ticket/8936#comment:50>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/063.192e4606d7ee9f9338119caa5c40e075%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
