Skip to content

Conversation

@PascalEgn
Copy link
Contributor

Copy link
Contributor

@ioannistsanaktsidis ioannistsanaktsidis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Just to double check if passing both headers has an effect + bump of next

@PascalEgn PascalEgn force-pushed the 1013-add-x-els-api-key-header branch from be45121 to 63cead5 Compare July 23, 2025 07:20
@PascalEgn
Copy link
Contributor Author

@ioannistsanaktsidis @drjova Added the if else statement for the header depending on what is provided. If its fine we can merge and prepare the next bump.

@PascalEgn PascalEgn merged commit 9638fee into inspirehep:master Jul 23, 2025
8 checks passed
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.

3 participants