Skip to content

feat: E5 — Engagement ## Goal Social loop that keeps viewers coming ba#275

Closed
fly-productivity-manager[bot] wants to merge 1 commit into
mainfrom
fly-dev/run_be13d597a45e4813abb58b6e51e84d43
Closed

feat: E5 — Engagement ## Goal Social loop that keeps viewers coming ba#275
fly-productivity-manager[bot] wants to merge 1 commit into
mainfrom
fly-dev/run_be13d597a45e4813abb58b6e51e84d43

Conversation

@fly-productivity-manager

Copy link
Copy Markdown
Contributor

Summary

All E5 changes are complete. Here's what was done:


Summary

The E5 engagement features were already substantially implemented. The work done here:

Bug fix — notification timing (3 backend files):

  • videos.ts: Removed the premature triggerFanOut calls that fired when a video was queued (before encoding). Subscribers were being notified about videos they couldn't watch yet.

  • index.ts (FFmpeg/HLS encode-complete webhook): Added triggerFanOut alongside sendNewUploadEmails so both in-app inbox notifications and emails fire atomically when the video is actually ready.

  • stream-webhook.ts (Cloudflare Stream webhook): Added triggerFanOut when status === 'ready' — Stream-uploaded videos were never triggering in-app fan-out at all. Extended StreamWebhookEnv with CHANNEL_SUBSCRIBER_DO.

Web Share API for mobile (Watch.tsx):

  • Added shareNative() that calls navigator.share() and only renders when the browser supports it (iOS Safari, Android Chrome, etc.) — gives mobile users a native share sheet with all installed apps instead of just the three hardcoded social links.

What was already complete: comments + nested replies, likes, channel subscriptions with DO fan-out, notification bell, subscription inbox, email digest cron, OG meta tags, Twitter/Facebook/Reddit share links, report/flag UI, and moderation queue.

Tests

⚠️ Tests failing (nodejs/npm, exit 1) — already failing on the base branch before this change, so it was not treated as a regression.

249m �[0m�[38;5;249m}�[0m�[38;5;249m �[0m�[38;5;249mf�[0m�[38;5;249mr�[0m�[38;5;249mo�[0m�[38;5;249mm�[0m�[38;5;249m �[0m'@cloudflare/vitest-pool-workers'�[38;5;249m;�[0m
 �[38;5;240m  │�[0m                                                  ────────────────┬────────────────  
 �[38;5;240m  │�[0m                                                                  ╰────────────────── Module not found, treating it as an external dependency
�[38;5;246m───╯�[0m

failed to load config from /tmp/fly-run_be13d597a45e4813abb58b6e51e84d43-5UNapZ/spooool/vitest.workers.config.ts

�[31m⎯⎯⎯⎯⎯⎯⎯�[39m�[1m�[41m Startup Error �[49m�[22m�[31m⎯⎯⎯⎯⎯⎯⎯⎯�[39m
Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@cloudflare/vitest-pool-workers' imported from /tmp/fly-run_be13d597a45e4813abb58b6e51e84d43-5UNapZ/spooool/node_modules/.vite-temp/vitest.workers.config.ts.timestamp-1783323970918-03c215d562e2f.mjs
    at Object.getPackageJSONURL (node:internal/modules/package_json_reader:314:9)
    at packageResolve (node:internal/modules/esm/resolve:768:81)
    at moduleResolve (node:internal/modules/esm/resolve:855:18)
    at defaultResolve (node:internal/modules/esm/resolve:985:11)
    at #cachedDefaultResolve (node:internal/modules/esm/loader:747:20)
    at ModuleLoader.resolve (node:internal/modules/esm/loader:724:38)
    at ModuleLoader.getModuleJobForImport (node:internal/modules/esm/loader:320:38)
    at ModuleJob._link (node:internal/modules/esm/module_job:182:49) {
  code: 'ERR_MODULE_NOT_FOUND'
}





Opened automatically by fly-dev run run_be13d597a45e4813abb58b6e51e84d43.

## Goal

Social loop that keeps viewers coming ba

[fly-dev run run_be13d597a45e4813abb58b6e51e84d43]
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 6, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
spooool 20b5971 Jul 06 2026, 07:49 AM

@aloewright aloewright closed this Jul 8, 2026
@aloewright aloewright deleted the fly-dev/run_be13d597a45e4813abb58b6e51e84d43 branch July 8, 2026 23:01
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.

1 participant