Skip to content

Feature/live class share#25

Open
PRODHOSH wants to merge 2 commits into
masterfrom
feature/live-class-share
Open

Feature/live class share#25
PRODHOSH wants to merge 2 commits into
masterfrom
feature/live-class-share

Conversation

@PRODHOSH

Copy link
Copy Markdown
Member

No description provided.

25f2007308 and others added 2 commits July 20, 2026 23:36
Adds a Share button (WhatsApp, Telegram, Email, Copy Link) to the
mentor and admin live-classes consoles. Instead of exposing the raw
meeting link, each share links to a new dedicated per-class page
(/dashboard/live-classes/[id]) that enforces the existing enrollment
check server-side and shows a clear access-denied/not-found state.

Also fixes a live bug: lib/rate-limit/index.ts was an unused, broken
duplicate of lib/rate-limit.ts (it imported the uninstalled
@upstash/redis package and didn't export apiRateLimiter), and it was
shadowing the real module for every route importing "@/lib/rate-limit",
causing /api/live-classes and others to 500. Removed the dead file.

Also parallelizes the enrollment + class Supabase queries in both
live-classes API routes, and stops the live-classes list page from
polling every 30s while the tab is in the background.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@PRODHOSH
PRODHOSH requested a review from 25f2007308 July 22, 2026 03:10
@PRODHOSH
PRODHOSH removed the request for review from 25f2007308 July 22, 2026 03:11

@PRODHOSH PRODHOSH left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

love the share idea, but few issues are here

  • u deleted the lib/rate-limit/index.ts and that breaks the imports in payment and compiler api routes

meaning no one can buy our courses if i merge this

from here on, check the https://bsprep-api-docs.vercel.app and also the schema frm supabase

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants