Skip to content

Use post IDs for list keys and add stability test#2

Merged
mweiden merged 1 commit into
mainfrom
codex/update-postlist-to-use-unique-key
Sep 2, 2025
Merged

Use post IDs for list keys and add stability test#2
mweiden merged 1 commit into
mainfrom
codex/update-postlist-to-use-unique-key

Conversation

@mweiden

@mweiden mweiden commented Sep 2, 2025

Copy link
Copy Markdown
Owner

Summary

  • Use each post's id as the <li> key in PostList
  • Include id in Post component props
  • Add a unit test verifying list items keep identity when reordered
  • Wrap App test in a mocked Apollo provider so tests run without network

Testing

  • yarn test

https://chatgpt.com/codex/tasks/task_e_68b67139abe08324b7320eb9befe4b0a

@mweiden
mweiden merged commit cfd1d4e into main Sep 2, 2025
1 check passed
@mweiden
mweiden deleted the codex/update-postlist-to-use-unique-key branch September 2, 2025 04:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant