[PR #25] [MERGED] fix(#24) 未设置github帐号时,不显示顶部同步header #694

New issue

Closed

opened 2026-03-02 03:41:20 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 03:41:20 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/codexu/note-gen/pull/25
Author: @NitronPlus
Created: 1/14/2025
Status: ✅ Merged
Merged: 1/15/2025
Merged by: @codexu

Base: dev ← Head: sync-optimize

📝 Commits (1)

4a4766a fix(#24) 未设置github帐号时,不显示顶部同步header

📊 Changes

3 files changed (+36 additions, -26 deletions)

View changed files

📝 src/app/core/image/image-header.tsx (+29 -24)
📝 src/app/core/image/no-data.tsx (+1 -1)
📝 src/lib/github.ts (+6 -1)

📄 Description

做了点相关的优化,对现在的前端不太熟悉,还停留在以前的jq时代,毕竟40+了
公司机器有点旧(7770),在dev模式下无法显示窗口内容(GT-720,NV驱动的问题),只能build后测试了

1.未设置gh相关信息时,不显示跟同步相关的信息

2.点击"前往设置"时,准确跳转到"同步"选项

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/codexu/note-gen/pull/25 **Author:** [@NitronPlus](https://github.com/NitronPlus) **Created:** 1/14/2025 **Status:** ✅ Merged **Merged:** 1/15/2025 **Merged by:** [@codexu](https://github.com/codexu) **Base:** `dev` ← **Head:** `sync-optimize` --- ### 📝 Commits (1) - [`4a4766a`](https://github.com/codexu/note-gen/commit/4a4766a8d31236a7374bb6a37e1019b8ef72b025) fix(#24) 未设置github帐号时,不显示顶部同步header ### 📊 Changes **3 files changed** (+36 additions, -26 deletions) <details> <summary>View changed files</summary> 📝 `src/app/core/image/image-header.tsx` (+29 -24) 📝 `src/app/core/image/no-data.tsx` (+1 -1) 📝 `src/lib/github.ts` (+6 -1) </details> ### 📄 Description 做了点相关的优化,对现在的前端不太熟悉,还停留在以前的jq时代,毕竟40+了公司机器有点旧(7770),在dev模式下无法显示窗口内容(GT-720,NV驱动的问题),只能build后测试了 1.未设置gh相关信息时,不显示跟同步相关的信息 ![image](https://github.com/user-attachments/assets/e259cbb2-b857-4292-89b6-a757f2033812) 2.点击"前往设置"时,准确跳转到"同步"选项 ![image](https://github.com/user-attachments/assets/fa404b10-4357-4c61-b604-18a35a1880cb) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>