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

add logoSize to globalQueryParams #10110

Merged
merged 1 commit into from
Apr 22, 2024
Merged

Conversation

chris48s
Copy link
Member

After I merged #9191 I deployed it to staging, gave it a quick test and realised we missed something.
This param was only taking effect for static badges. It was ignored for dynamic ones.
This change fixes it, but it does also raise follow up questions which I'll note in an issue

@chris48s chris48s added the core Server, BaseService, GitHub auth label Apr 22, 2024
Copy link
Contributor

Messages
📖 ✨ Thanks for your contribution to Shields, @chris48s!

Generated by 🚫 dangerJS against a3ff051

@chris48s chris48s merged commit d6dde3f into badges:master Apr 22, 2024
22 checks passed
@LitoMore
Copy link
Contributor

The endpoint badge does not support logoSize for now. Maybe you can also support logoSize for the endpoint badge in this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Server, BaseService, GitHub auth
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants