Skip to content

Conversation

@koji
Copy link
Contributor

@koji koji commented Dec 29, 2025

Overview

refactor InputPrompt phase2
create a new custom hook, useInputPromptController and update InputPrompt and its test

phase 1 (need a review first)
#20458

close AUTH-2173

Test Plan and Hands on Testing

Changelog

  • create useInputPromptController
  • update InputPrompt and its test

Review requests

Risk assessment

low

koji added 6 commits December 28, 2025 03:45
refactor InputPrompt phase1 export functions to resources/utils from InputPrompt

close AUTH-2173 partially
refactor InputPrompt phase2 - create useInputPromptController hook

clsoe AUTH-2173
@koji koji requested a review from Elyorcv December 29, 2025 07:22
@koji koji marked this pull request as ready for review December 29, 2025 07:23
@codecov
Copy link

codecov bot commented Dec 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.26%. Comparing base (c888540) to head (d84f0cf).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             edge   #20460      +/-   ##
==========================================
- Coverage   56.26%   56.26%   -0.01%     
==========================================
  Files        3678     3677       -1     
  Lines      309823   309734      -89     
  Branches    45827    45814      -13     
==========================================
- Hits       174335   174258      -77     
+ Misses     135240   135228      -12     
  Partials      248      248              
Flag Coverage Δ
opentrons-ai-client 2.79% <ø> (-0.23%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 32 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants