Skip to content

Conversation

ashercoren
Copy link
Member

Refracted the canUpdate hooks to make them easier to use, and so that we can use them to toggle the "Push All" button

@ashercoren ashercoren requested a review from a team as a code owner May 19, 2025 07:17
@ashercoren ashercoren requested a review from leeorsky May 19, 2025 07:17
@ashercoren ashercoren force-pushed the disable-push-all-when-needed branch from 131ead1 to 9d22578 Compare May 19, 2025 07:18
Copy link

cloudflare-workers-and-pages bot commented May 19, 2025

Deploying vim-ai-scribe-demo-app-dev with  Cloudflare Pages  Cloudflare Pages

Latest commit: e021bdf
Status: ✅  Deploy successful!
Preview URL: https://6114c134.vim-ai-scribe-demo-app-dev.pages.dev
Branch Preview URL: https://disable-push-all-when-needed.vim-ai-scribe-demo-app-dev.pages.dev

View logs

Copy link

cloudflare-workers-and-pages bot commented May 19, 2025

Deploying vim-ai-scribe-demo-app-prod with  Cloudflare Pages  Cloudflare Pages

Latest commit: e021bdf
Status: ✅  Deploy successful!
Preview URL: https://3c2ab812.vim-ai-scribe-demo-app-prod.pages.dev
Branch Preview URL: https://disable-push-all-when-needed.vim-ai-scribe-demo-app-prod.pages.dev

View logs

Base automatically changed from SAM-69104-mic-badge to main May 19, 2025 13:05
@ashercoren ashercoren force-pushed the disable-push-all-when-needed branch from 9d22578 to 2f5f517 Compare May 19, 2025 13:06
hoveredSegment={hoveredSegment}
transcriptionSegments={transcriptionSegments}
renderHighlightedText={renderHighlightedText}
{...updateEncounterState}

Choose a reason for hiding this comment

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

use hook inside as well, dont need to prop drill

@ashercoren ashercoren requested a review from vimHopemanRyan May 20, 2025 09:06
@ashercoren ashercoren merged commit 12cd63e into main May 20, 2025
3 checks passed
@ashercoren ashercoren deleted the disable-push-all-when-needed branch May 20, 2025 19:14
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