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 GitHub discussions total badge [GithubTotalDiscussions] #6472

Merged
merged 4 commits into from
May 11, 2021

Conversation

profgrammer
Copy link
Contributor

  • Fixed the request method - added the headers from the options body in the GitHub GraphQL V4 API
  • Added the Github Total Discussions badge with tests
    image

@shields-ci
Copy link

shields-ci commented May 10, 2021

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

Generated by 🚫 dangerJS against 31a8eda

@PyvesB PyvesB added the service-badge New or updated service badge label May 10, 2021
@shields-cd shields-cd temporarily deployed to shields-staging-pr-6472 May 10, 2021 19:05 Inactive
Copy link
Member

@PyvesB PyvesB left a comment

Choose a reason for hiding this comment

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

Thanks for working on this @profgrammer! The badge works well in the review app and apart form a minor suggestion, the implementation looks good to me. 👍🏻

data: Joi.object({
repository: Joi.object({
discussions: Joi.object({
totalCount: Joi.number().required(),
Copy link
Member

Choose a reason for hiding this comment

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

Could nonNegativeInteger be used here?

@shields-cd shields-cd temporarily deployed to shields-staging-pr-6472 May 10, 2021 19:30 Inactive
@PyvesB
Copy link
Member

PyvesB commented May 11, 2021

@profgrammer if you've got a Twitter account, I can post a little something mentioning you for this new badge!

@profgrammer
Copy link
Contributor Author

@profgrammer if you've got a Twitter account, I can post a little something mentioning you for this new badge!

I don't really use Twitter but I have an account:
https://twitter.com/prafful_javare

Thanks!

@PyvesB
Copy link
Member

PyvesB commented May 12, 2021

There you go: https://twitter.com/Shields_io/status/1392556555753635840

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
service-badge New or updated service badge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants