Require auth for /preview_email/ (#16) #17

Merged
westfarn merged 1 commits from fix/16-preview-email-require-auth into master 2026-07-10 10:57:48 -07:00
Owner

Summary

Test plan

  • python manage.py test public.tests.PreviewEmailAuthTests
  • Manually hit /preview_email/1/ logged out → redirect to /accounts/login/
  • Log in and hit same URL → email preview renders
## Summary - Closes #16 - Confirms `/preview_email/<pk>/` requires an authenticated user (`@login_required` already on the view) - Adds unit tests for unauthenticated redirect and authenticated preview access ## Test plan - [x] `python manage.py test public.tests.PreviewEmailAuthTests` - [ ] Manually hit `/preview_email/1/` logged out → redirect to `/accounts/login/` - [ ] Log in and hit same URL → email preview renders
westfarn added 1 commit 2026-07-10 10:53:23 -07:00
Add unit tests ensuring /preview_email/ requires authentication.
CI / test (pull_request) Successful in 11s
Unit Tests / test (pull_request) Successful in 10s
07cba8bc7f
Locks in login_required behavior for issue #16 so unauthenticated access stays redirected.
westfarn merged commit c97bd16445 into master 2026-07-10 10:57:48 -07:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: ai_ml_operations/company_site#17