Skip to content

[WIP] Address feedback on frontend video analysis pipeline PR#38

Merged
groupthinking merged 3 commits intomainfrom
copilot/sub-pr-33-one-more-time
Feb 27, 2026
Merged

[WIP] Address feedback on frontend video analysis pipeline PR#38
groupthinking merged 3 commits intomainfrom
copilot/sub-pr-33-one-more-time

Conversation

Copy link
Contributor

Copilot AI commented Feb 27, 2026

  • Fix ENABLE_VERTEX_AI truthiness check in gemini_service.py to only enable on 1/true/yes
  • Fix transcript_segments to use result.transcript?.segments?.length
  • Fix relative URLs in server-side fetch calls using getBaseUrl(request)
  • Fix clearTimeout not in finally block in apps/web/src/app/api/video/route.ts
  • Fix clearTimeout not in finally block in apps/web/src/app/api/transcribe/route.ts
  • Refactor summary extraction logic in video/route.ts to replace nested ternaries/IIFE with linear code

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@vercel
Copy link
Contributor

vercel bot commented Feb 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
v0-uvai Error Error Feb 27, 2026 11:07pm

@railway-app railway-app bot temporarily deployed to EventRelay / EventRelay-pr-38 February 27, 2026 23:01 Destroyed
@railway-app
Copy link

railway-app bot commented Feb 27, 2026

🚅 Deployed to the EventRelay-pr-38 environment in EventRelay

Service Status Web Updated (UTC)
EventRelay 🕐 Queued (View Logs) Web Feb 27, 2026 at 11:06 pm
guides 🕐 Queued (View Logs) Web Feb 27, 2026 at 11:06 pm
supabase 🕐 Queued (View Logs) Feb 27, 2026 at 11:06 pm
eslint-config 🕐 Queued (View Logs) Web Feb 27, 2026 at 11:06 pm
web 🕐 Queued (View Logs) Web Feb 27, 2026 at 11:06 pm

@groupthinking groupthinking marked this pull request as ready for review February 27, 2026 23:04
Copilot AI review requested due to automatic review settings February 27, 2026 23:04
@railway-app railway-app bot temporarily deployed to EventRelay / EventRelay-pr-38 February 27, 2026 23:04 Destroyed
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Base automatically changed from claude/slack-check-status-update-R47Ph to main February 27, 2026 23:05
@railway-app railway-app bot temporarily deployed to EventRelay / EventRelay-pr-38 February 27, 2026 23:06 Destroyed
@groupthinking groupthinking merged commit ed013e1 into main Feb 27, 2026
10 of 12 checks passed
@groupthinking groupthinking deleted the copilot/sub-pr-33-one-more-time branch February 27, 2026 23:06
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.

3 participants