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

Add translator for svt.se #1688

Merged
merged 4 commits into from Sep 15, 2018

Conversation

Projects
None yet
3 participants
@sebastian-berlin-wmse
Copy link
Contributor

sebastian-berlin-wmse commented Jul 5, 2018

This is the website for Sveriges nyheter, the Swedish public service TV.

Add translator for svt.se
This is the website for Sveriges nyheter, the Swedish public service TV.
@adam3smith
Copy link
Collaborator

adam3smith left a comment

Two additional comments here.

"priority": 100,
"inRepository": true,
"translatorType": 4,
"browserSupport": "gcibv",

This comment has been minimized.

@adam3smith

adam3smith Jul 7, 2018

Collaborator

gcisbv

Show resolved Hide resolved SVT Nyheter.js
Fix translator for svt.se according to code review
* Updated target regex.
* Added checks to avoid errors when values weren't found.
* Changed `var` to `let` in for loop.
* Added Safari to `browserSupport`.

@sebastian-berlin-wmse sebastian-berlin-wmse force-pushed the Wikimedia-Sverige:svt-nyheter branch from a3da382 to 46c9303 Aug 13, 2018

@sebastian-berlin-wmse

This comment has been minimized.

Copy link
Contributor

sebastian-berlin-wmse commented Aug 13, 2018

I've updated according to the comments here and in #1691. Let me know if there's anything else.

adam3smith added some commits Sep 6, 2018

@sebastian-berlin-wmse

This comment has been minimized.

Copy link
Contributor

sebastian-berlin-wmse commented Sep 14, 2018

Is there anything else that needs fixing for this?

@adam3smith

This comment has been minimized.

Copy link
Collaborator

adam3smith commented Sep 14, 2018

I don't think so, I'm just getting through reviews slowly, sorry.

@adam3smith adam3smith merged commit a08b22e into zotero:master Sep 15, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@adam3smith

This comment has been minimized.

Copy link
Collaborator

adam3smith commented Sep 15, 2018

Thanks!

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