Repository navigation
分拣台:刷新勾选收藏夹的列表时重算取消收藏记录的「还在别处」 - #436
Merged
Merged
Conversation
在 B站 把取消收藏过的视频收进或移出另一个勾选收藏夹后,B站 页面的「已取消收藏」跟着改;没读全的列表不算。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
改了什么
在分拣台把视频从唯一的勾选收藏夹 A 取消收藏,之后在 B站 直接把它收进勾选收藏夹 B,B站 普通页面仍标「已取消收藏」。反过来也一样:取消收藏时它还在 B,之后在 B站 从 B 移出,就一直不标。取消收藏记录里的
elsewhere只在取消收藏那一刻算一次。saveSnapshot),在同一把锁里按最新的缓存列表重算其他收藏夹取消收藏记录的elsewhere,只写变了的(TriageCore.refreshElsewhere、refreshElsewhereNow)。验证
node scripts/selftest.mjs:39 个通过。triage-core加了表格测试(两个方向、自己的收藏夹不算、没变的不重写、没有完整列表的收藏夹不改成「不在别处」、保留记录不碰),另有走真实saveSnapshot的测试;去掉saveSnapshot里那行调用,测试失败。