-
-
Notifications
You must be signed in to change notification settings - Fork 143
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
MyAnimeList Support, Profiles, and List Editor. #527
base: master
Are you sure you want to change the base?
Conversation
idk why an array is returned for some seasons resulting in '3,03' so instead of it failing and returning the default season we can now return the first entry in the array.
Hahaha no problem, hope you have a great vacation. |
Quick heads up: the pasting on linux if the protocol handler doesn't work, doesn't actually work. This means that if your linux protocol handler doesn't work, you cannot log in (probably on both anilist and mal, not tested) |
Update: that seems to be an issue with an undefined variable in settings.js#63 and settings.js#64 where |
…bol 'line' in linux's clipboard paste handler
I don't know how it got there
Fix line feed at the end of state returned from MAL and undefined symbol 'line' in linux's clipboard paste handler
Confirmed, thanks for catching that. |
so the TLDR is no. this PR is simply too massive for me to review and merge all at once and it touches a lot of things it really shouldn't, it also builds on top of my shitty code which caused a lot of problems for you, and many od these things can be done cleaner HOWEVER there's a lot of good shit, please don't remove this PR as I'll likely use it as reference for many Mal things for miru v6 |
yeah I figured as much. The MAL implementation should be a good reference, and the auth tokens have been properly refreshing after they expire without the need for any user input so the implementation has been working well. I plan to keep maintaining my fork of Miru since it's finally at a stage where it does everything I need it to in order for me to move away from using streaming sites and addons. So I'll keep this PR open and create a separate detached fork so I don't further clutter things here. You can feel free to delete this whenever you no longer have need of it. Thanks for all the time you have put into Miru. It's been an absolute life changer and has been working perfectly. I have tried many other "similar" apps but nothing can compare to it. |
Just curious, could you elaborate more on this? I'm interested in using this fork and tinkering with it myself. |
I can't wait for an implementation that supports MAL! |
Hi, I apologize in advance as this is kinda lengthy. I absolutely love this app but I ran into a few things that were missing or issues I was having so I created a fork that I have been working on for a few months. I figured since a lot of this is stuff being requested by other users I would attempt to contribute it here.
I have zero expectations of you accepting it as-is as this is quite a large PR and I am sure there are places where my code can be improved. I am open to any advice/criticisms, I would love to contribute these changes if you are interested. I am happy to make any changes to get this closer to something you would consider accepting.
The biggest changes are the MAL support, Profiles, List Editor, Tags to search filtering, Mal-Dubs filtering, and Hide-My-Anime filtering. To clarify on MAL/Profiles support. MAL is completely independent of Anilist, a login for Anilist is NOT required. With profiles users do not need to have multiple profiles for Miru login to function. You do not need to login for Miru to function, and only a single login from either Anilist or MAL is required for list functionality.
Here are some screenshots of the major changes;
Changes to relations (and added recommendations) using small cards.
Currently, I have the clientId for MAL authentication setup with my dev account, if your able you can create your own clientId and replace mine or do nothing it is up to you.
As a final note,
helper.js
was added to help bridge the gap between authorizations, for the most part being a single location to add support for additional authorizations in the future like Kitsu.Added:
*
Romance*
and hit enter to force search that genre.Fixed:
\n
was not being respected, addedwhite-space: pre-line
to container-wrapper.Changed: