Skip to content

Fix citation details #527

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

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

Fix citation details #527

wants to merge 2 commits into from

Conversation

kss682
Copy link
Collaborator

@kss682 kss682 commented May 16, 2025

PR fixes #522

  • Add user-agent to google scholar request
  • Align citation buttons
  • Update default values in config
  • Add citation id in config

image

- Add user-agent to google scholar request
- Align citation buttons
- Update default values in config
- Add citation id in config
@kss682
Copy link
Collaborator Author

kss682 commented May 16, 2025

Unfortunately, the citation request fails with user-agent too.
image

@MakisH
Copy link
Member

MakisH commented May 16, 2025

Are there any usage/request limits?

@kss682
Copy link
Collaborator Author

kss682 commented May 16, 2025

It does have a rate limit which is what happens when we run it locally multiple times, that usually gives a 429.
But the request from actions are forbidden.

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.

Landing page: Citation details outdated/broken
2 participants