Skip to content

Conversation

@dharit-tan
Copy link
Contributor

  • I accidentally created a non-ATA account on the authority of PHUTURES
  • Later created the ATA
  • /v1/coins was giving duplicate rows for PHUTURES
  • Fix by only selecting earliest-created account for that owner - as a proxy for the ATA, assuming we'd create it as part of the reward pool creation process.
  • But technically the PHUTURES response is still showing the wrong account (since I created the non-ATA before the ATA). Not a huge issue bc we don't use that value anywhere. But maybe I can just delete that row for this case?

Tested on prod

Copy link
Contributor

@rickyrombo rickyrombo left a comment

Choose a reason for hiding this comment

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

This works, but yeah I think we just delete the row we don't care about

@dharit-tan dharit-tan merged commit c7dc430 into main Nov 18, 2025
5 checks passed
@dharit-tan dharit-tan deleted the rt-coins branch November 18, 2025 18:20
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