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

added theses.fr webtranslator #1672

Closed
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@pirpyn

pirpyn commented Jun 8, 2018

First version of the web translator for https://theses.fr

pirpyn and others added some commits Jun 8, 2018

@adam3smith

Some questions/comments in line. Biggest concern is though that this simply doesn't work for me -- is the site access restricted or is it currently broken? Links like
http://www.theses.fr/s197772
just take me to the empty search screen.

"translatorID": "65c7ee26-eee7-442a-ba09-41da30cf2864",
"label": "Theses.fr",
"creator": "Pierre Payen",
"target": "^https?://(www\\.)?theses.fr/[^?]",

This comment has been minimized.

@adam3smith

adam3smith Aug 12, 2018

Collaborator

[^?] can't be right -- that wouldn't match any search results

@adam3smith

adam3smith Aug 12, 2018

Collaborator

[^?] can't be right -- that wouldn't match any search results

// translator.setDocument(doc);
translator.setHandler('itemDone', function (obj, item) {
item.section = "Theses";

This comment has been minimized.

@adam3smith

adam3smith Aug 12, 2018

Collaborator

that's not part of the Zotero thesis item type. What are you trying to do here?

@adam3smith

adam3smith Aug 12, 2018

Collaborator

that's not part of the Zotero thesis item type. What are you trying to do here?

function scrape(doc, url) {
var translator = Zotero.loadTranslator('web');
translator.setTranslator('951c027d-74ac-47d4-a107-9c3069ab7b48');
// translator.setDocument(doc);

This comment has been minimized.

@adam3smith

adam3smith Aug 12, 2018

Collaborator

why is this commented out? I'm pretty sure you need this.

@adam3smith

adam3smith Aug 12, 2018

Collaborator

why is this commented out? I'm pretty sure you need this.

"creatorType": "author"
}
],
"abstractNote": ",",

This comment has been minimized.

@adam3smith

adam3smith Aug 12, 2018

Collaborator

That's no good. Clean empty abstracts up. Also, we really don't have a better place to get this than the Twitter description?

@adam3smith

adam3smith Aug 12, 2018

Collaborator

That's no good. Clean empty abstracts up. Also, we really don't have a better place to get this than the Twitter description?

],
"tags": [
{
"tag": "en français"

This comment has been minimized.

@adam3smith

adam3smith Aug 12, 2018

Collaborator

These tags are obviously useless.

@adam3smith

adam3smith Aug 12, 2018

Collaborator

These tags are obviously useless.

],
"tags": [
{
"tag": "Stratégie de potentialisation Tuberculose Ethionamide Isoniazide Multi-résistance en français"

This comment has been minimized.

@adam3smith

adam3smith Aug 12, 2018

Collaborator

These tags look good, but need to be properly separated out to be useful. Is there no place the tags are listed more systematically?

@adam3smith

adam3smith Aug 12, 2018

Collaborator

These tags look good, but need to be properly separated out to be useful. Is there no place the tags are listed more systematically?

@adam3smith adam3smith added waiting and removed waiting labels Aug 12, 2018

@adam3smith

This comment has been minimized.

Show comment
Hide comment
@adam3smith

adam3smith Aug 12, 2018

Collaborator

actually, never mind. This is a duplicate of #1645 , which looks in better shape

Collaborator

adam3smith commented Aug 12, 2018

actually, never mind. This is a duplicate of #1645 , which looks in better shape

@adam3smith adam3smith closed this Aug 12, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment