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

Get manga info from tracker #1271

Merged
merged 17 commits into from
Dec 8, 2024
Merged

Conversation

NGB-Was-Taken
Copy link

@NGB-Was-Taken NGB-Was-Taken commented Sep 7, 2024

Allows setting manga info to the one provided by tracker (currently includes title, author, artist, thumbnail url and description).

Closes #1205

TODO:

  • Implement for all trackers.
  • Properly handle multiple trackers.
  • Handle rate-limits and other HttpExceptions (hoping a single try catch is enough)
  • Automatically fetch metadata when tracker is added to an entry for the first time (?) Decided against it (discord)
  • Replace strings with moko strings.

@NGB-Was-Taken NGB-Was-Taken marked this pull request as ready for review October 29, 2024 10:38
@NGB-Was-Taken

This comment was marked as outdated.

Copy link
Owner

@jobobby04 jobobby04 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not too much to comment on in this PR. Its mostly big because of the trackers which I don't care about lol

Co-authored-by: jobobby04 <jobobby04@users.noreply.github.com>
@jobobby04 jobobby04 merged commit fd120c5 into jobobby04:master Dec 8, 2024
2 checks passed
cuong-tran pushed a commit to komikku-app/komikku that referenced this pull request Dec 9, 2024
* Barebones setup (only AniList works)

* Show tracker selection dialog when entry has more than one tracker

* MangaUpdates implementation

* Add logging and toast on error.

* MyAnimeList implementation

* Kitsu implementation

* Fix MAL authors and artists

* Decode AL description

* Throw NotImplementedError instead of returning null

* Use logcat from LogcatExtensions

* Replace strings with MR strings

* Missed a string

* Delete unused Author class.

* Add Bangumi & Shikimori support for info edit (#2)

This adds the necessary API calls and DTOs to allow for editing an
entry's data to the data from a tracker, specifically adding support
for Bangumi and Shikimori.

* Exclude enhanced trackers from tracker select dialog

* MdList implementation

* Remember getTracks and trackerManager

Co-authored-by: jobobby04 <jobobby04@users.noreply.github.com>

---------

Co-authored-by: MajorTanya <39014446+MajorTanya@users.noreply.github.com>
Co-authored-by: jobobby04 <jobobby04@users.noreply.github.com>
(cherry picked from commit fd120c5081a74393f0f2c174f2b3573b14c6d4e4)
jeryjs pushed a commit to jeryjs/TachiyomiSY-with-discord-RPC that referenced this pull request Dec 10, 2024
* Barebones setup (only AniList works)

* Show tracker selection dialog when entry has more than one tracker

* MangaUpdates implementation

* Add logging and toast on error.

* MyAnimeList implementation

* Kitsu implementation

* Fix MAL authors and artists

* Decode AL description

* Throw NotImplementedError instead of returning null

* Use logcat from LogcatExtensions

* Replace strings with MR strings

* Missed a string

* Delete unused Author class.

* Add Bangumi & Shikimori support for info edit (#2)

This adds the necessary API calls and DTOs to allow for editing an
entry's data to the data from a tracker, specifically adding support
for Bangumi and Shikimori.

* Exclude enhanced trackers from tracker select dialog

* MdList implementation

* Remember getTracks and trackerManager

Co-authored-by: jobobby04 <jobobby04@users.noreply.github.com>

---------

Co-authored-by: MajorTanya <39014446+MajorTanya@users.noreply.github.com>
Co-authored-by: jobobby04 <jobobby04@users.noreply.github.com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 11, 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.

🎟️ Fetch info from Anilist
3 participants