-
-
Notifications
You must be signed in to change notification settings - Fork 317
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
Data saver #1082
Data saver #1082
Conversation
Sometimes it dosen't recognise the commonMain in source-api. This is a workaround so there are no problems with further coding. Is taken from a Tachiyomi commit
This adds the data saver from TachiyomiSY
app/src/main/java/eu/kanade/domain/source/manga/interactor/GetEnabledMangaSources.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/eu/kanade/domain/source/manga/interactor/GetEnabledMangaSources.kt
Outdated
Show resolved
Hide resolved
I would suggest renaming the exh package since thats mostly for Syer's ExHentai stuff. better to just name it something more generic |
Maybe |
@@ -0,0 +1,5 @@ | |||
package aniyomi.source |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this even used anywhere?
Sorry if I missed something
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah in GetEnabledMangaSources
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay but I don't see how this has any relevance because I believe this is related to SY's "special" sources
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@quickdesh I think you used SY, right? What is this for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wasn't sure at moment of implementing. Thats why I left it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merged Manga and Lewd are SY special sources, yes. These can be removed since they don't affect Aniyomi
Will reopen the pr. Messed some things up on my side |
Adds the Data Saver from TachiyomiSY.
Redo because old PR was scuffed or something. Don't know. Fuck lint and github