-
Notifications
You must be signed in to change notification settings - Fork 801
update CNKI family #3417
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
base: master
Are you sure you want to change the base?
update CNKI family #3417
Conversation
Some pages of kns.cnki.net can only be opened by IP in Chinese Mainland, and specific resource types even require institutional account login. However, the test cases shown here have been completed manually in the past one or two days. |
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.
Thanks, this is really nice work and looks like a big improvement. A few suggestions.
CNKI has a large number of websites designed for specific disciplines, which have different hosts and appearances but similar export formats. Therefore, I have written a unified import translator for them.