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

Smart updates fix #268

Merged
merged 17 commits into from
Aug 24, 2024
Merged

Conversation

kana-shii
Copy link
Contributor

Eventually closes #212

I tried changing this line .takeIf { status != SManga.COMPLETED.toLong() } to make it interact with the smart updates option, but I wasn't able to figure out how. When I thought I was going to the right direction, it would give me a "Circular dependency" error.

@kana-shii kana-shii marked this pull request as draft August 20, 2024 10:51
still needs more testing though
currently, if the option is activated, all of the completed entries will have the N/A instead of the hourglass and vice versa.
@kana-shii kana-shii marked this pull request as ready for review August 21, 2024 17:37
@cuong-tran cuong-tran force-pushed the master branch 3 times, most recently from ef268d9 to 424c7c7 Compare August 22, 2024 18:13
@cuong-tran
Copy link
Collaborator

looks fine

kana-shii and others added 8 commits August 22, 2024 16:18
still needs more testing though
currently, if the option is activated, all of the completed entries will have the N/A instead of the hourglass and vice versa.
forgot to remove this as the code isn't using it anymore
@kana-shii kana-shii marked this pull request as draft August 23, 2024 15:36
@kana-shii kana-shii marked this pull request as ready for review August 23, 2024 15:49
@cuong-tran cuong-tran merged commit eaceb3e into komikku-app:master Aug 24, 2024
2 checks passed
when using !in it doesn't work, so I removed the !
@kana-shii kana-shii deleted the smart-updates-fix-cmplt branch August 24, 2024 14:23
cuong-tran pushed a commit that referenced this pull request Aug 25, 2024
* Allow check completed entries for update if un-tick option "Skip completed..."
* Always predict next-release date for all entries, even if it's completed.
cuong-tran pushed a commit that referenced this pull request Aug 25, 2024
* Allow check completed entries for update if un-tick option "Skip completed..."
* Always predict next-release date for all entries, even if it's completed.
cuong-tran pushed a commit that referenced this pull request Sep 19, 2024
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
(cherry picked from commit 31e9273)
cuong-tran pushed a commit that referenced this pull request Sep 19, 2024
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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.

add a default predicted release date for entries with no predictable date
2 participants