-
-
Notifications
You must be signed in to change notification settings - Fork 132
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
Extend MediaType #435
Extend MediaType #435
Conversation
Would one of you be interested in maintaining this library? I'm no longer using TMDb, so it's a chore. :/ |
I think the current pull requests can be accepted, they contain a lot of features and bug fixes. |
@LordMike Hey. We've used your library for a while in Jellyfin and we might be willing to take over. Would you be willing to transfer the repo to us? Or would you prefer a fork? |
I could transfer it. I think that makes most sense.
Would this work? |
Yes, that works for us. |
Are there any updates on the transfer of package maintenance to the Jellyfin project? As a downstream consumer of this package I haven't heard of any news in the Jellyfin #development channel on the discord server. |
I added @cvium as a member to this repo a few days ago - I do hope he got an email :P I couldn't transfer the repo myself because of rights. |
I just checked - the invite was expired. I've reinvited. |
Yes, sorry, I keep forgetting |
@LordMike I think you need to give me permissions to transfer |
@cvium ok - so I think that for personal projects like this, there can be only one admin. At least, all the pictures I can find indicate a drop down for role, but its simply not present. What I can do is transfer the project to you (which I can't because you already have a project named TMDbLib - delete that) and then you will be an admin. Or I can become an admin in Jellyfin and then have the permission to create projects there (which is required to do transfers). Let me know. |
@LordMike I have deleted my fork. I can't give you admin in Jellyfin. Even I'm not an admin there :) |
@cvium check your .. something .. there should be a transfer request. |
Aand, now it is at Jellyfin. Good job everyone! :) |
The roundabout transfer is complete: https://github.com/jellyfin/TMDbLib. Unfortunately, our GH admin is asleep, so your permissions will be set up later |
We also need to transfer the NuGet package ownership. @cvium |
I think we need @anthonylavado for that |
Isn't that just adding Jellyfin as an owner of the package? |
I have added jellyfin. There should be request awaiting approval somewhere. |
Well. It was accepted. I assume all is good and I can remove myself from the package? |
Fixes: #414, #415, #420, #430
This will do the trick, but it would be nicer if we could map two different enum values to the same enum.
Like this:
or