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
Adding GameStudies, Journal of Extension #916
Conversation
Added Journal of Extension (www.joe.org) to web translators.
Added GameStudies (www.gamestudies.org) to web translators.
Removed AccessDate property in testCase.
"translatorID": "8c2c716c-09ba-421a-ae23-a1590d90a95e", | ||
"label": "Game Studies", | ||
"creator": "John Weis", | ||
"target": "gamestudies\\.org", |
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.
could we be more restrictive here? ^https?://gamestudies\\.org/
(ending slash is also prefered since it works bettet with proxies)
That should be all. Let us know if anything is unclear, otherwise let us know when you've made all changes. (For future reference, we prefer one pull request per translator, though this is going to be fine). |
Cleaner extraction of Keyword (GameStudies), Abstract (JoE) sections
Snapshots added to both journals. JoE date extraction changed from .split() method to .match() regex.
Added test cases, better detect criteria for JoE
Please undo the renames of three unrelated translators. (did Scaffold do this?) |
"label": "Game Studies", | ||
"creator": "John Weis", | ||
"target": "^https?://gamestudies\\.org/", | ||
"minVersion": "4.0.26.4", |
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.
2.1.9 for current framework translators
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.
Changed for both translators
}, | ||
{ | ||
url :FW.Url(), | ||
title :"Article Snapshot", |
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.
just "Snapshot"
Changed MinVersion to 2.1.9 because of Framework compatibility. Fixed spaces, GameStudies detection regex.
Removed
This reverts commit 58b26cd. Conflicts: Game Studies.js Journal of Extension.js
Adding GameStudies, Journal of Extension
Thanks for working on this! I squashed your commits and merged them into master, then made a few tweaks. |
Added these two journals, with testing, to web translators.