Merge pull request #2 from acesknust/original-changes#3
Open
Simondelali wants to merge 119 commits intooriginal-changesfrom
Open
Merge pull request #2 from acesknust/original-changes#3Simondelali wants to merge 119 commits intooriginal-changesfrom
Simondelali wants to merge 119 commits intooriginal-changesfrom
Conversation
Merge pull request #1 from acesknust/main
Original changes
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@Simondelali is attempting to deploy a commit to the Owusu Prince 's projects team on Vercel, but is not a member of this team. To resolve this issue, you can:
To read more about collaboration on Vercel, click here. |
…bout page with content and navigation, and update header links to point to the About page.
UI enhancement and restructure files
Resolve module locations
…and navigation, and update header links to point to the About page.
Aboutpage
…r and Footer components.
Refactor import paths in About page to use absolute imports for Heade…
Fix HTML entity in About page text for proper rendering of apostrophe.
…; adjust logo size and source.
Update About page with new executive images, titles, and descriptions…
Feature/courses redesign
- Reuse existing PENDING orders (same email, 1h window) with select_for_update row locking - Atomically update cart items, amount, and coupon on order reuse - Guard coupon times_used decrement against PositiveIntegerField going below 0 - Add WebhookLog idempotency check to skip already-processed charge.success events - Add useRef-based double-click lock on frontend checkout - Improve cleanup_orders command (24h default, --dry-run) - Add cleanup_webhook_logs command (90-day retention, --dry-run) - All webhook responses return 200 to prevent Paystack retries
- Fix critical HMAC bug: cache raw_body before request.data (DRF stream issue) - Add master try-except: all 12 webhook paths now return HTTP 200 - Early event metadata extraction: fixes empty reference/event_type in logs - Add throttle_classes=[] to prevent rate-limiting on webhook endpoint - Strip sensitive headers (Authorization, Cookie) from WebhookLog - Add stock floor check to prevent negative inventory - Remove str(e) error leaks from client-facing responses - Fix misleading 'processed' status when _complete_verification fails - Add core/celery.py + shop/tasks.py for hourly order expiry - Add django_celery_beat + django_celery_results to INSTALLED_APPS - Update Procfile with worker and beat processes
redis==6.4.0 changed SSL defaults — without explicit ssl_cert_reqs, the Celery worker crashes instantly on startup with no logs. Add CELERY_BROKER_USE_SSL and CELERY_REDIS_BACKEND_USE_SSL with CERT_NONE, conditioned on the REDIS_URL starting with rediss://. Connection remains fully TLS-encrypted; only CA chain verification is skipped (standard practice for backend-to-managed-Redis traffic).
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
5 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.
Merge pull request #1 from acesknust/main