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

Rename subtitle provider from TheWiz > Wizdom #2923

Merged
merged 6 commits into from
Jul 4, 2017

Conversation

fernandog
Copy link
Contributor

@fernandog fernandog commented Jun 29, 2017

URL also changed
code from @ofir123

@fernandog fernandog force-pushed the feature/rename_subs_provider branch from 7bf99c2 to 38d463a Compare June 29, 2017 12:40
@fernandog fernandog added this to the 0.1.14 milestone Jun 30, 2017
@fernandog
Copy link
Contributor Author

@ratoaq2 please?

@pymedusa pymedusa deleted a comment from duramato Jul 4, 2017
@fernandog
Copy link
Contributor Author

@ratoaq2 develop provider is broken as that URL is 404 now. Need to merge to this release

ratoaq2
ratoaq2 previously approved these changes Jul 4, 2017
Copy link
Contributor

@ratoaq2 ratoaq2 left a comment

Choose a reason for hiding this comment

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

Didn't review the code since this is a third party provider

p0psicles
p0psicles previously approved these changes Jul 4, 2017
@p0psicles
Copy link
Contributor

I says i dismissed a review. But thats because I resolved conflicts. And merged rebase it.

@p0psicles
Copy link
Contributor

@fernandog can you double check I resolved the conflicts the right way?

@fernandog
Copy link
Contributor Author

@p0psicles thanks

@p0psicles p0psicles merged commit b02bc07 into develop Jul 4, 2017
@p0psicles p0psicles deleted the feature/rename_subs_provider branch July 4, 2017 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants