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

refactor: db changes for syncing. #113

Merged
merged 1 commit into from
Jan 18, 2024

Conversation

kaiserbh
Copy link
Contributor

@kaiserbh kaiserbh commented Jan 18, 2024

Set the lastModifiedAt default to 0 on insert, this way when syncing it doesn't mark chapter as unread.

For example:
Device A: has a manga with 79 chapter main device used for reading and more up to date.
Device B: haven't done any update library so it's stale, something like 78 chapter now when the user sync it will get the data from Device B as latest since on insert previously sets the current time.
Device A: Marks the chapter that was read as unread.

Well another approach is to sync before library update, but with this changes they don't have to sync before updating library.

…cing.

So this should have been 0 on insert instead of the current time on insert. Essentially this issue arises: https://discord.com/channels/1099009852791083058/1099009853864812708/1190022356060614756

Signed-off-by: KaiserBh <kaiserbh@proton.me>
@AntsyLich AntsyLich merged commit e4a6565 into mihonapp:main Jan 18, 2024
1 check passed
kaiserbh added a commit to kaiserbh/mihon that referenced this pull request Feb 18, 2024
fix: sync marking chapter unread when we do library update before syncing.

So this should have been 0 on insert instead of the current time on insert. Essentially this issue arises: https://discord.com/channels/1099009852791083058/1099009853864812708/1190022356060614756

Signed-off-by: KaiserBh <kaiserbh@proton.me>
(cherry picked from commit e4a6565)
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants