Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

(SP: 0.5) Avoid Using Array Index as Key in JSX Lists #2677

Closed
Mike-Popovych opened this issue Nov 11, 2024 · 0 comments · Fixed by #2685
Closed

(SP: 0.5) Avoid Using Array Index as Key in JSX Lists #2677

Mike-Popovych opened this issue Nov 11, 2024 · 0 comments · Fixed by #2685
Assignees
@Made1ra
Labels
Frontend part good first issue Good for newcomers sonarFix fix the sonar issues

Comments

@Mike-Popovych
Copy link
Contributor

Description:

Using array indices as keys in list components can cause unexpected behavior, especially when items are reordered or changed. Instead, use a unique ID for each item to help React handle updates more efficiently.

File Affected:

AddProfessionalCategoryModal.tsx
@Mike-Popovych Mike-Popovych added good first issue Good for newcomers Frontend part sonarFix fix the sonar issues labels Nov 11, 2024
@github-project-automation github-project-automation bot moved this to Project Backlog in SpaceToStudy Nov 11, 2024
@Made1ra Made1ra self-assigned this Nov 12, 2024
@Made1ra Made1ra linked a pull request Nov 12, 2024 that will close this issue
Fixed using array index as key in lists #2685
Merged
@Made1ra Made1ra moved this from Sprint Backlog to Merge request in SpaceToStudy Nov 13, 2024
Made1ra added a commit that referenced this issue Nov 13, 2024
@Made1ra
Fixed using array index as key in lists (#2677)
7b7d167
@github-project-automation github-project-automation bot moved this from Merge request to Done in SpaceToStudy Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Made1ra Made1ra

Labels
Frontend part good first issue Good for newcomers sonarFix fix the sonar issues
Projects
SpaceToStudy
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixed using array index as key in lists ita-social-projects/SpaceToStudy-Client
2 participants
@Made1ra @Mike-Popovych