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

Dev #34

Open
wants to merge 100 commits into
base: master
Choose a base branch
from
Open

Dev #34

wants to merge 100 commits into from

Conversation

trevorspreadbury
Copy link
Collaborator

No description provided.

trevorspreadbury and others added 30 commits June 2, 2021 16:31
Pressing 'Download Results' button in ui results view downloads the
current results into a 'queryset' directory with images and a json.
Implements core functionality in new utilities.data module
The 'Download Results' method in the results UI now also downloads
subfigure label, and scale bar information, recreating a near complete
recreation of original exsclaim jsons (unassasigned fields are not in the
database). The download is down as a separate process and user can now
specify results location.
Nature's HTML format changed, removing the tag with metadata we used
to get the number of pages and results. Now this is retrieved from
numbers displayed to users.
Renames and reformats journalfamily subclass attributes to be more
intuitive. Implements method to obtain many search urls (varying
date range, journal, and ordering method) to allow greater than
1000 results for acs and nature.
- class Wiley(JournalFamily) added - get_search_query_urls 
- class JournalFamily params
1. removed the text in "" for teh pub_type param
2. added in journal_list param that was missing
3. ACS --> Wiley in coment
Journal specific code should stay abstracted from the base class.
Wiley year parameter is strange but can still fit same pattern
of date_range_param=start<sep>end if we see <sep> as &BeforeYear=
rather than the usual -
Changes made to add the Wiley  publication family to the tool
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants