Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed #29417 -- Corrected two admin page titles for view-only users. #9966

Merged

Conversation

ramiro
Copy link
Member

@ramiro ramiro commented May 19, 2018

The Changelist and 'View object' views still contained the word 'change'
in their title and header.

@timgraham
Copy link
Member

Is it possible to write assertions with response.context in lieu of or before assertContains()? That's often easier to debug.

@timgraham timgraham changed the title Fixed #29417 -- Corrected two admin page titles for RO users. Fixed #29417 -- Corrected two admin page titles for view-only users. May 23, 2018
@timgraham timgraham force-pushed the t29417-ro-admin-clist-edit-views-titles branch from 5f4f003 to f8b93bb Compare May 23, 2018 14:30
@timgraham timgraham merged commit f40e71a into django:master May 23, 2018
@ramiro ramiro deleted the t29417-ro-admin-clist-edit-views-titles branch August 9, 2018 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants