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.

Sign up for GitHub

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

Jump to bottom

we could decrease bandwidth by making a custom graphql request here that doesn't... #22

Open

github-actions bot opened this issue Dec 7, 2021 · 0 comments

Labels

optimization todo

github-actions bot commented Dec 7, 2021

we could decrease bandwidth by making a custom graphql request here that doesn't retrieve some of the unnecessary fields that this retrieves

https://github.com/adavila0703/dev-hub-bot/blob/c92f25d2f1919f9bdabdbfef15c4d36819374090/internal/github/contributions.go#L170

		Username: username,
	}

	// TODO: we could decrease bandwidth by making a custom graphql request here that doesn't retrieve some of the unnecessary fields that this retrieves 
	contributions, err := g.GetContributionsByUsername(ctx, options)
	if err != nil {
		return nil, errors.Wrap(err, "get contributions by username")

0694d8afd8b26dae8aebeb24589e9e2a01f08406

github-actions bot added the todo label

dillonstreator added the optimization label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment