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.

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

fix: Key warning in info stripes #11257

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

memoyil
Copy link
Collaborator

@memoyil memoyil commented Feb 25, 2025


PR-Codex overview

This PR primarily focuses on optimizing the nextItem calculation in InfoStripes/index.tsx by including CONFIG.length in its dependency array. Additionally, it simplifies the JSX structure in InfoStripeCommon.tsx by replacing a fragment with a single Fragment component.

Detailed summary

  • Updated the dependency array of nextItem in InfoStripes/index.tsx to include CONFIG.length.
  • Replaced an empty fragment with a Fragment component in InfoStripeCommon.tsx.
  • Removed unnecessary key assignment for the divider in the button mapping within InfoStripeCommon.tsx.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

vercel bot commented Feb 25, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 25, 2025 6:40pm
7 Skipped Deployments
Name Status Preview Comments Updated (UTC)
aptos-web ⬜️ Ignored (Inspect) Visit Preview Feb 25, 2025 6:40pm
blog ⬜️ Ignored (Inspect) Visit Preview Feb 25, 2025 6:40pm
bridge ⬜️ Ignored (Inspect) Visit Preview Feb 25, 2025 6:40pm
games ⬜️ Ignored (Inspect) Feb 25, 2025 6:40pm
gamification ⬜️ Ignored (Inspect) Visit Preview Feb 25, 2025 6:40pm
uikit ⬜️ Ignored (Inspect) Visit Preview Feb 25, 2025 6:40pm
ton ⬜️ Skipped (Inspect) Feb 25, 2025 6:40pm

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