vEnhance / otis-web / 19579893954 / 1

94%

main: 94%

push

github

refactor!: Extract Discord logging handler from submodule (#437)

* refactor: extract Discord logging into django-discordo package

Replace evans_django_tools submodule with new django-discordo package.
The evans_django_tools submodule was only being used for Discord logging
functionality. This change extracts that into a standalone PyPI package.

Changes:
- Remove evans_django_tools git submodule
- Replace all imports from evans_django_tools to django_discordo
- Update pyproject.toml to use django-discordo package
- Update README.md to remove submodule initialization steps
- Update claude.md to reflect new Discord logging package
- Add DJANGO_DISCORDO_PACKAGE_GUIDE.md with package structure and
  publishing instructions for the new django-discordo package

The django-discordo package provides:
- DiscordWebhookHandler for Django logging
- Custom log levels (VERBOSE, SUCCESS, ACTION)
- Automatic formatting and redaction of sensitive data
- Support for level-specific webhook URLs

Breaking change: Developers will need to install django-discordo
instead of initializing the evans_django_tools submodule.

* style: ruff import order

* refactor: use Django settings for discord-discordo configuration

Improve django-discordo package to be more Django-idiomatic by using
Django settings as the primary configuration method.

Changes to django-discordo package:
- Remove python-dotenv dependency (not needed)
- Update handler to check Django settings first, then env vars
- Support both simple (DISCORD_WEBHOOK_URL) and advanced
  (DISCORD_WEBHOOK_URLS dict) configuration styles
- Improve get_url() method with clearer priority order

Changes to otis-web:
- Add DISCORD_WEBHOOK_URL setting that reads from environment
- Add commented example of DISCORD_WEBHOOK_URLS dict config
- Update package guide with new configuration approach

Benefits:
- More idiomatic Django configuration
- Easier to override in tests
- Better type safety
- Cleaner separation of concerns (Django... (continued)

7 of 186 branches covered (3.76%)

Branch coverage included in aggregate %.

1043 of 1583 relevant lines covered (65.89%)

0.66 hits per line