Skip to content

chore: remove unused field ids ref in useFieldArray#13066

Merged
bluebill1049 merged 1 commit intoreact-hook-form:masterfrom
uk960214:refactor/field-ids-ref
Sep 29, 2025
Merged

chore: remove unused field ids ref in useFieldArray#13066
bluebill1049 merged 1 commit intoreact-hook-form:masterfrom
uk960214:refactor/field-ids-ref

Conversation

@uk960214
Copy link
Copy Markdown
Contributor

Summary

Remove dead code left over from PR #7447

Changes

  • Remove unused _fieldIds ref declaration and assignment
  • Clean up dead code that was no longer needed after the ids ref separation

Context

In PR #7447, the useFieldArray implementation was optimized by separating the management of field ids from the main fields ref. However, the _fieldIds ref declaration and its assignment (_fieldIds.current = fields) remained as dead code after this optimization.

This is a simple refactoring to clean up the unused code with no functional changes.

Copy link
Copy Markdown
Member

@bluebill1049 bluebill1049 left a comment

Choose a reason for hiding this comment

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

Awesome

@bluebill1049 bluebill1049 merged commit 23c699a into react-hook-form:master Sep 29, 2025
6 checks passed
@uk960214 uk960214 deleted the refactor/field-ids-ref branch September 29, 2025 03:36
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