Skip to content

fix: transcribe UI subtitle were blocking chat#16097

Merged
PeerRich merged 9 commits intomainfrom
16091-cal-4118-transcription-block-too-long
Aug 14, 2024
Merged

fix: transcribe UI subtitle were blocking chat#16097
PeerRich merged 9 commits intomainfrom
16091-cal-4118-transcription-block-too-long

Conversation

@PeerRich
Copy link
Copy Markdown
Member

@PeerRich PeerRich commented Aug 6, 2024

image

@PeerRich PeerRich linked an issue Aug 6, 2024 that may be closed by this pull request
@linear
Copy link
Copy Markdown

linear Bot commented Aug 6, 2024

@github-actions github-actions Bot added cal video 🐛 bug Something isn't working labels Aug 6, 2024
@dosubot dosubot Bot added ai area: AI, cal.ai ui area: UI, frontend, button, form, input labels Aug 6, 2024
@graphite-app graphite-app Bot requested a review from a team August 6, 2024 16:33
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Aug 6, 2024

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (08/06/24)

1 reviewer was added to this PR based on Keith Williams's automation.

@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ai ❌ Failed (Inspect) Aug 14, 2024 1:18pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Aug 14, 2024 1:18pm
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Aug 14, 2024 1:18pm

@PeerRich PeerRich added the ⚡ Quick Wins A collection of quick wins/quick fixes that are less than 30 minutes of work label Aug 7, 2024
@PeerRich PeerRich added this to the v4.4 milestone Aug 7, 2024
@PeerRich PeerRich enabled auto-merge (squash) August 7, 2024 17:32
Copy link
Copy Markdown
Contributor

@Udit-takkar Udit-takkar left a comment

Choose a reason for hiding this comment

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

On mobile view It's blocking chat input box

Screenshot 2024-08-07 at 11 52 55 PM

@PeerRich
Copy link
Copy Markdown
Member Author

PeerRich commented Aug 9, 2024

On mobile view It's blocking chat input box

Screenshot 2024-08-07 at 11 52 55 PM

ahh thats gonna be tricky to fix, cause right now the transcription is "outside" of the embed.

we could put it higher up but then it looks odd...

this PR is still better than before. maybe we just hide transcription on mobile

<DailyProvider callObject={daily}>
<div
className="mx-auto"
className="mx-auto hidden sm:block"
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.

@Udit-takkar this should hide transcription on mobile

@PeerRich
Copy link
Copy Markdown
Member Author

a lot of workflow and webhook related E2E failing here

CleanShot 2024-08-13 at 15 12 32@2x

@github-actions
Copy link
Copy Markdown
Contributor

E2E results are ready!

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

Labels

ai area: AI, cal.ai 🐛 bug Something isn't working cal video ⚡ Quick Wins A collection of quick wins/quick fixes that are less than 30 minutes of work ready-for-e2e ui area: UI, frontend, button, form, input

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CAL-4118] transcription block too long

3 participants