Skip to content
This repository has been archived by the owner on Jun 10, 2024. It is now read-only.

Raise NameError if model specified in ADMIN_REORDER can't be found #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pief
Copy link

@pief pief commented Feb 6, 2017

Makes it easier to catch configuration mistakes that otherwise just tend to get overseen.

@coveralls
Copy link

coveralls commented Feb 6, 2017

Coverage Status

Coverage decreased (-0.009%) to 0.962% when pulling 792f2a4 on pief:master into 6e3ea39 on mishbahr:master.

@jedie
Copy link

jedie commented Mar 1, 2017

IMHO a great idea!

But you can also raise an error, of 'app' is not found, too.

@pief
Copy link
Author

pief commented Mar 1, 2017

Not sure I understand?

@jedie
Copy link

jedie commented Mar 1, 2017

Ah, i see. Wrong app name seems to raise an error, already.

@emilyzzz
Copy link

" Wrong app name seems to raise an error, already.", I'm not seeing this.
I agree with original author with this PR, adding this PR will definitely help identify problem. I had a typo in my app name and took me a while to figure out why the app/models won't show up on admin list.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants