Skip to content

CAS-229: Add current user to Leaderboard#154

Merged
dbslone merged 18 commits intomainfrom
CAS-229
Jul 15, 2022
Merged

CAS-229: Add current user to Leaderboard#154
dbslone merged 18 commits intomainfrom
CAS-229

Conversation

@jbluks
Copy link
Copy Markdown
Collaborator

@jbluks jbluks commented Jul 14, 2022

Ticket: CAS-229

Description:

Adds Current User to the Leaderboard, if wallet is connected. Leaderboard API now returns a user if passed an address.

Screen Shot 2022-07-14 at 4 26 26 PM

@dbslone
Copy link
Copy Markdown
Contributor

dbslone commented Jul 15, 2022

When I try to cast a vote on a community that I just created I receive the error ERROR: relation "user_achievements" does not exist (SQLSTATE 42P01)

@jbluks
Copy link
Copy Markdown
Collaborator Author

jbluks commented Jul 15, 2022

Checking it out - it seems like the database/migration are out of sync and that has created an issue. I have dropped everything locally, ran the migrations and voted via the UI and it all worked as expected.

Comment thread frontend/packages/client/src/hooks/useLeaderBoard.js Outdated
Comment thread frontend/packages/client/src/hooks/useLeaderBoard.js Outdated
Copy link
Copy Markdown
Contributor

@germanurrus germanurrus left a comment

Choose a reason for hiding this comment

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

LGTM! (frontend changes) thanks for the updates @jbluks

@dbslone dbslone merged commit 3f559d5 into main Jul 15, 2022
@dbslone dbslone deleted the CAS-229 branch July 15, 2022 22:08
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