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

Citoid: Remove the bibtex format #1046

Closed
wants to merge 1 commit into from

Conversation

d00rman
Copy link
Contributor

@d00rman d00rman commented Aug 9, 2018

Citoid will soon start using the new Zotero translation server, which
does not allow exporting references in bibtex. Therefore, remove it from
the public API as a possible format.

Bug: T201618

Citoid will soon start using the new Zotero translation server, which
does not allow exporting references in bibtex. Therefore, remove it from
the public API as a possible format.

Bug: T201618
@d00rman d00rman requested review from mvolz and Pchelolo August 9, 2018 15:26
@d00rman
Copy link
Contributor Author

d00rman commented Aug 9, 2018

Sad to see bibtex go </3

@d00rman
Copy link
Contributor Author

d00rman commented Aug 9, 2018

Not ready to be merged yet

@mvolz
Copy link
Collaborator

mvolz commented Aug 9, 2018

They are planning to enable it at some point, so maybe not forever!

@Pchelolo
Copy link
Contributor

Pchelolo commented Aug 9, 2018

We've found out that bibtex is a lot of requests https://grafana.wikimedia.org/dashboard/db/service-citoid?panelId=8&fullscreen&orgId=1 sooooo... maybe we should not get rid of it?

@d00rman
Copy link
Contributor Author

d00rman commented Aug 10, 2018

@Pchelolo this was all due to service-checked. I removed the x-ample that requested bibtex and all requests for it went away. Nevertheless, it seems that zotero v2 will implement it back soon-ish, so let's keep this PR open but not merge it yet.

@d00rman
Copy link
Contributor Author

d00rman commented Oct 18, 2018

Bibtex support has been added to the new translation server, so it's safe to close this.

@d00rman d00rman closed this Oct 18, 2018
@d00rman d00rman deleted the citoid/no-bibtex branch October 18, 2018 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants