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.

Sign up for GitHub

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

Jump to bottom

Financial Times translator issues #1075

Closed
owcz opened this issue Jun 10, 2016 · 2 comments · Fixed by #2384
Closed

Financial Times translator issues #1075

owcz opened this issue Jun 10, 2016 · 2 comments · Fixed by #2384

Comments

@owcz
Copy link
Contributor

owcz commented Jun 10, 2016

  • Pulls author as "Review by Demetri Sevastopulo" in Review — A bleak outlook on an unequal America
  • Pulls a paywalled screenshot when using the translator, even if the page isn't paywalled on view. Embedded Metadata's screenshot pulls what's currently on the page.

Should be easy to fix, but tracking here in case someone gets to it before me
@owcz
Copy link
Contributor Author

owcz commented Mar 15, 2017 •
edited

@adam3smith looks like this translator is totally down & not detecting on the article previously mentioned—likely needs a rewrite. Wanted to give you the heads up, but if you prefer, I can take a stab. Should note, though, that I don't have a subscription

@owcz owcz mentioned this issue Sep 7, 2017
Closed
@jrennstich
Copy link

Same here. It starts to detect, but then never finishes.

AbeJellinek added a commit to AbeJellinek/translators that referenced this issue May 25, 2021
@AbeJellinek
Financial Times: Remove FW, update for current site
2088c07 
The search test will fail on CI. Fixes zotero#1391 and zotero#1075.
@AbeJellinek AbeJellinek mentioned this issue May 25, 2021
Merged
AbeJellinek added a commit that referenced this issue May 28, 2021
@AbeJellinek
Financial Times: Remove FW, update for current site
3569f8e 
The search test will fail on CI. Fixes #1391 and #1075.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Financial Times: Remove FW, update for current site AbeJellinek/translators
2 participants
@owcz @jrennstich