Integrate newsletter feature from PR #3939 #4847
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
github-actions
bot
added
the
unresolved-conversations: 0
Co-authored-by: DonnieBLT <[email protected]>
Co-authored-by: DonnieBLT <[email protected]>
❌ Pre-commit checks failedThe pre-commit hooks found issues that need to be fixed. Please run the following commands locally to fix them: # Install pre-commit if you haven't already
pip install pre-commit
# Run pre-commit on all files
pre-commit run --all-files
# Or run pre-commit on staged files only
pre-commit runAfter running these commands, the pre-commit hooks will automatically fix most issues. 💡 Tip: You can set up pre-commit to run automatically on every commit by running: pre-commit installFor more information, see the pre-commit documentation. |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| messages.error(request, "Email address is required.") | ||
| return redirect("newsletter_subscribe") | ||
|
|
||
| if not re.match(r"[^@]+@[^@]+\.[^@]+", email): |
Check failure
Code scanning / CodeQL
Polynomial regular expression used on uncontrolled data High
There was a problem hiding this comment.
@copilot fix Polynomial regular expression used on uncontrolled data
Check failure: Polynomial regular expression used on uncontrolled data
This regular expression that depends on a user-provided value may run slow on strings with many repetitions of '?'.
| data = json.loads(request.body) | ||
| email = data.get("email") | ||
|
|
||
| if not email or not re.match(r"[^@]+@[^@]+\.[^@]+", email): |
Check failure
Code scanning / CodeQL
Polynomial regular expression used on uncontrolled data High
|
👋 Hi @copilot! This pull request needs a peer review before it can be merged. Please request a review from a team member who is not:
Once a valid peer review is submitted, this check will pass automatically. Thank you! |
github-actions
bot
added
unresolved-conversations: 2
❌ Pre-commit checks failedThe pre-commit hooks found issues that need to be fixed. Please run the following commands locally to fix them: # Install pre-commit if you haven't already
pip install pre-commit
# Run pre-commit on all files
pre-commit run --all-files
# Or run pre-commit on staged files only
pre-commit runAfter running these commands, the pre-commit hooks will automatically fix most issues. 💡 Tip: You can set up pre-commit to run automatically on every commit by running: pre-commit installPre-commit outputFor more information, see the pre-commit documentation. |
❌ Pre-commit checks failedThe pre-commit hooks found issues that need to be fixed. Please run the following commands locally to fix them: # Install pre-commit if you haven't already
pip install pre-commit
# Run pre-commit on all files
pre-commit run --all-files
# Or run pre-commit on staged files only
pre-commit runAfter running these commands, the pre-commit hooks will automatically fix most issues. 💡 Tip: You can set up pre-commit to run automatically on every commit by running: pre-commit installPre-commit outputFor more information, see the pre-commit documentation. |
Integrates comprehensive newsletter functionality from PR #3939, including subscription management, email distribution, and admin interface.
Models
Newsletter: Markdown content, featured images, publication workflow, email trackingNewsletterSubscriber: Double opt-in flow, preference management, UUID-based tokensPointsmodel withrelated_name="points"for leaderboard queriesViews & URLs
/newsletter/Templates
Admin
NewsletterAdmin: Bulk send action, publication managementNewsletterSubscriberAdmin: Confirmation email resend, bulk status updatesManagement Command
send_newsletter: Bulk email distribution with test modeDependencies
django-gravatar2==1.4.4for compatibilityDatabase
Two migrations:
token_created_atfor expiration tracking (48hr)Security
Token expiration, rate limiting, CSRF protection, input sanitization, timing attack prevention.
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.