Fix WAGTAILADMIN_BASE_URL warning in Django system checks#176
Merged
Fix WAGTAILADMIN_BASE_URL warning in Django system checks#176
Conversation
Add WAGTAILADMIN_BASE_URL setting to all Django settings files to fix the wagtailadmin.W003 warning that appeared during system checks. The project had a BASE_URL setting but Wagtail expects the setting to be named WAGTAILADMIN_BASE_URL for admin URLs outside the admin interface. Changes: - base.py: Set WAGTAILADMIN_BASE_URL = BASE_URL (https://python.ie) - tests.py: Override with http://testserver for test environment - dev.py: Override with http://localhost:8000 for local development This ensures admin URLs in notification emails and the user bar will display correctly in all environments. Fixes #175 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
ArcaneIRE
approved these changes
Dec 24, 2025
Resolve merge conflict in pythonie/pythonie/settings/tests.py by combining: - WAGTAILADMIN_BASE_URL setting from the PR - Improved DATABASE_URL configuration from master
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR fixes the
wagtailadmin.W003warning that appeared during Django system checks by adding theWAGTAILADMIN_BASE_URLsetting to all Django settings files.Problem
When running tests or Django system checks, a warning appeared:
Changes
WAGTAILADMIN_BASE_URL = BASE_URLto use the existing production URL (https://python.ie)http://testserverfor the test environmenthttp://localhost:8000for local developmentTesting
All 33 tests pass with no warnings:
Impact
This ensures admin URLs appearing outside the admin interface (such as in notification emails and the user bar) will display correctly in all environments.
Fixes #175
🤖 Generated with Claude Code