search

electh / ReactFlux

Simple RSS reader for Miniflux
https://reactflux.pages.dev
MIT License
152 stars 40 forks source link

重构 axios 和 Settings：集中错误处理并优化组件结构 #19

Closed NekoAria closed 4 months ago

NekoAria commented 4 months ago
vercel[bot] commented 4 months ago

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

Name Status Preview Comments Updated (UTC)
react-flux ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 13, 2024 10:53am
electh commented 4 months ago

大佬，刚没看 pull requests ，我推了个变更上来，这显示冲突了...

electh commented 4 months ago

可以了，大佬就是大佬

NekoAria commented 4 months ago

其实从开发者工具里的 网络 一栏就能看到所有请求，比起通过日志打印出来的 response 对象更清楚。 所以我把那些日志打印操作都删掉了。

electh commented 4 months ago

我直接从ChatGPT抄的代码我会乱说吗，哈哈哈。另外不知道从哪个版本开始，各种动画都干没了，比如文章详情进入动画，文章列表进入动画，文章列表加载动画啥的

electh commented 4 months ago

其实从开发者工具里的 网络 一栏就能看到所有请求，比起通过日志打印出来的 response 对象更清楚。 所以我把那些日志打印操作都删掉了。

好像当时是我的 chrome 经常报 network changed 什么的错误, 然后我测试的时候又没开开发者工具的习惯，报错了再打开network 都是空的，所以就全干上 console.log 了

NekoAria commented 4 months ago

不知道从哪个版本开始，各种动画都干没了，比如文章详情进入动画，文章列表进入动画，文章列表加载动画啥的

最新版本也有这个问题吗？可能是我哪次重构的时候导致的。 现在拆分得这么细，应该很容易找到是哪里导致的问题。 你列个清单，我等会排查下吧。

electh commented 4 months ago

https://github.com/electh/ReactFlux/assets/83588235/2825768d-e100-4e39-9295-f288193dd30b

不知道从哪个版本开始，各种动画都干没了，比如文章详情进入动画，文章列表进入动画，文章列表加载动画啥的

最新版本也有这个问题吗？可能是我哪次重构的时候导致的。 现在拆分得这么细，应该很容易找到是哪里导致的问题。 你列个清单，我等会排查下吧。

目前发现就这三个：文章详情进入动画，文章列表进入动画，文章列表接口返回前的加载动画

electh commented 4 months ago

我发现重构以后我有点改不动了，react 我看完 useState 和 useEffect 就开干了，看来要恶补一下后面的功能了🤣

NekoAria commented 4 months ago

你看看在这个提交时的动画是否正常：https://e93de685.reactflux.pages.dev/

https://github.com/electh/ReactFlux/runs/22599489362

文章列表接口返回前的加载动画 这个应该是正常的，另外俩个好像没了。

electh commented 4 months ago

你看看在这个提交时的动画是否正常：https://e93de685.reactflux.pages.dev/

https://github.com/electh/ReactFlux/runs/22599489362

文章列表接口返回前的加载动画 这个应该是正常的，另外俩个好像没了。

文章详情进入动画没了，其余两个有

electh commented 4 months ago

另外刚刚看切换搜索标题和搜索全文的选择框报错了，好像是筛选的那个函数没正常传进来，但是我不知道怎么改🥲

NekoAria commented 4 months ago

另外刚刚看切换搜索标题和搜索全文的选择框报错了，好像是筛选的那个函数没正常传进来，但是我不知道怎么改🥲

这个倒是修好了。

UPDATE: 都修好了。