Closed yomybaby closed 2 weeks ago
Add the label “flow:merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “flow:hotfix” to add to the merge queue as a hot fix.
You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.
This stack of pull requests is managed by Graphite. Learn more about stacking.
Join @yomybaby and the rest of your teammates on Graphite
./react
St.:grey_question: |
Category | Percentage | Covered / Total |
---|---|---|---|
🔴 | Statements | 4.96% (+0.15% 🔼) |
360/7253 |
🔴 | Branches | 4.43% (+0.02% 🔼) |
221/4992 |
🔴 | Functions | 2.97% (+0.16% 🔼) |
71/2393 |
🔴 | Lines | 4.86% (+0.15% 🔼) |
345/7093 |
100 tests passing in 13 suites.
Report generated by 🧪jest coverage report action from 5702e0a3d064ef2970e4915374ef48070be7bcbe
Resolves #2819
Changes: Introduces a new
useMemoWithPrevious
hook to manage previous and current values with reset capability, replacing the manualuseRef
implementation in ContainerLogModal. This hook provides a more robust way to track state changes while maintaining previous values.Implementation Details:
useMemoWithPrevious
hook that returns both current and previous values along with a reset functionpreviousLastLineNumber
ref management with the new hook'sresetPrevious
functionalityTesting Requirements: