-
Notifications
You must be signed in to change notification settings - Fork 27
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
Feature/mikan url config #489
Conversation
b038361
to
77041ff
Compare
Codecov Report
@@ Coverage Diff @@
## master #489 +/- ##
==========================================
+ Coverage 42.19% 42.25% +0.05%
==========================================
Files 32 32
Lines 2237 2239 +2
==========================================
+ Hits 944 946 +2
Misses 1293 1293
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
bgmi/website/mikan.py
Outdated
@@ -14,13 +15,13 @@ | |||
from bgmi.website.base import BaseWebsite | |||
from bgmi.website.model import Episode, SubtitleGroup, WebsiteBangumi | |||
|
|||
server_root = "https://mikanani.me/" | |||
server_root = f"{cfg.mikan_url}/" |
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.
是不是检查一下最后的slash
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.
也行,如果他多打了就帮他干掉
@@ -299,7 +306,7 @@ def search_by_tag(self, tag: str, subtitle: Optional[str] = None, count: Optiona | |||
title_el = item.find("title") | |||
title = title_el.text if title_el is not None else None | |||
|
|||
xmlns = "{https://mikanani.me/0.1/}" | |||
xmlns = "{" + server_root + "0.1/}" |
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.
镜像站也会改写这里的ns吗?
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.
从前两天开始大陆地区访问不到 mikanani.me 了,这个 pr 给 mikan 增加了镜像站配置的支持。