Skip to content

fix: header config#29

Merged
Wesley-0808 merged 1 commit intodevelopfrom
fix/header-config
Apr 6, 2025
Merged

fix: header config#29
Wesley-0808 merged 1 commit intodevelopfrom
fix/header-config

Conversation

@Wesley-0808
Copy link
Member

@Wesley-0808 Wesley-0808 commented Apr 6, 2025

🤔 这个 PR 的性质是?

  • 日常 bug 修复
  • 新特性提交
  • 文档改进
  • 样式/交互改进
  • CI/CD 改进
  • 重构
  • 代码风格优化
  • 分支合并
  • 其他

🔗 相关 Issue

💡 需求背景和解决方案

📝 更新日志

  • fix(WebManage): 修复header排序配置无效的问题。

  • chore(Repo): 修正仓库地址

  • chore: 更新服务端文件名

  • 本条 PR 不需要纳入 Changelog

☑️ 请求合并前的自查清单

⚠️ 请自检并全部勾选全部选项⚠️

  • 文档已补充或无须补充
  • 代码演示已提供或无须提供
  • TypeScript 定义已补充或无须补充
  • Changelog 已提供或无须提供

footer config

server path

hooks

repo link
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 13 out of 13 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

src/components/header.tsx:112

  • [nitpick] Consider using a unique identifier (if available) as the key instead of the array index to ensure stable rendering when the list order changes.
key={index}

src/components/footer.tsx:100

  • [nitpick] Consider using a unique identifier for each footer section instead of relying on the index for the key to improve rendering stability after sorting.
key={`desktop-${index}`}

@github-actions
Copy link
Contributor

github-actions bot commented Apr 6, 2025

完成

@Wesley-0808 Wesley-0808 merged commit 5974e2f into develop Apr 6, 2025
2 checks passed
@Wesley-0808 Wesley-0808 deleted the fix/header-config branch April 6, 2025 14:55
@github-actions github-actions bot mentioned this pull request Apr 6, 2025
14 tasks
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