jazzband
'djdt' is not a registered namespace #1405
Description
The error was 'djdt' is not a registered namespace. It occurred due to misconfiguration of the toolbar settings as well as the debug settings.
Fixes # (issue 1405)
- Added a check to result in an official error
- Added tests for the check
- Updated the docs with the right way to configure the toolbar to work with debug without an issue
Checklist:
- [X] I have added the relevant tests for this change.
- [X] I have added an item to the Pending section of
docs/changes.rst.
@elineda @matthiask @tim-schilling could you test it out and let me know if everything is good and any changes or suggestions you have at this point?
@elineda @matthiask @tim-schilling could you test it out and let me know if everything is good and any changes or suggestions you have at this point?
Is it okay to merge?
@VeldaKiara no, we have some miscommunications yet. I started on a change that will better communicate them. I'll try to push that up tonight.
@VeldaKiara no, we have some miscommunications yet. I started on a change that will better communicate them. I'll try to push that up tonight.
Awesome, I will be waiting on the changes.
@VeldaKiara I pushed some changes to the check and the settings in the example app. Let me know what you think.
I suspect we should ignore our new check in the test settings until we test it explicitly. In that case we should use the @override_settings decorator to enable it. That should get the tests to pass.
Regarding the documentation, maybe we keep the docs as they are, then link them to the example app to see how it's setup in an advanced configuration?
Let me check out the changes.
I think that would work to pointing to the example app
I have incorporated the changes requested. Let me know if you have any more suggestions.
I have removed the patch decorator, updated the docs and made some changes to the test_panel_empty check for the test to pass
