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

Replace string usages for BaseItemDto.Type with the EBaseItemType #217

Merged
merged 3 commits into from
Dec 24, 2019

Conversation

AndreasGB
Copy link
Contributor

Also fixing dependency substitution now that we use explicit versions

This depends on the related PR jellyfin/jellyfin-sdk-kotlin#30 , which should be made available as release v0.5.0 before this is merged.

I'm sorry for the person that reviews this and makes sure that all conditional replacements are correct, it was a pain for me to write and review before sending the PR as well :(

Luckily this has to be done only once :)

@thornbill
Copy link
Member

I'm going to rebase this on master and force push to resolve the conflicts.

@thornbill thornbill removed the blocked Needs another PR first label Dec 23, 2019
@thornbill thornbill merged commit 25edae1 into jellyfin:master Dec 24, 2019
@AndreasGB AndreasGB deleted the typestring-to-enum branch December 24, 2019 13:56
@nielsvanvelzen nielsvanvelzen added this to In progress in v0.11.0 via automation Jan 28, 2020
@nielsvanvelzen nielsvanvelzen moved this from In progress to Done in v0.11.0 Jan 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
v0.11.0
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants