Skip to content
This repository has been archived by the owner on Sep 3, 2019. It is now read-only.

Remove calculatewinnings #1546

Merged
merged 11 commits into from
Jul 11, 2019
Merged

Remove calculatewinnings #1546

merged 11 commits into from
Jul 11, 2019

Conversation

BernardHan
Copy link
Contributor

@BernardHan BernardHan self-assigned this Jul 10, 2019
export async function biggestWinners(client, args) {
return new GraphQuery(client, QUERY_BIGGEST_WINNERS, args).execute();
export async function globalLeaderboardEntries(client, args) {
const res = await new GraphQuery(client, QUERY_GLOBAL_LEADERBOARD_ENTRIES, args).execute();
Copy link
Member

Choose a reason for hiding this comment

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

just return this. no reason to assign it first unless you were using res.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh right.... this was for testing, forgot to change back.

@BernardHan BernardHan merged commit cb78522 into dev-070419 Jul 11, 2019
@BernardHan BernardHan deleted the remove-calculatewinnings branch July 11, 2019 21:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants