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

hdev-703-xav-airdrop #3

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

Conversation

tracyarciaga
Copy link
Contributor

@tracyarciaga tracyarciaga commented Aug 15, 2022

Description of Changes

Added script to generate addresses eligible for XAV airdrop
Link to Jira Ticket

How To Test

Run the script
yarn hardhat xav-airdrop --network mainnet

Can use etherscan's token check tool to verify address balance at block height.
https://etherscan.io/tokencheck-tool

Developer Checklist:

  • I have followed the guidelines in our Contributing document
  • This PR has a corresponding JIRA ticket
  • My branch conforms with our naming convention i.e. feature/HDF-XXX-description
  • I have written new tests for your core changes, as applicable
  • I have successfully ran tests locally
  • I have formatted my code using format document in VSCode

Reviewers Checklist:

  • Code is readable and understandable; any unclear parts have explanations
  • UI/UX changes match the corresponding figma/other design resources, if applicable
  • I have successfully ran tests locally

const addresses: string[] = []
const snapshotBlockNumber = 15281097

// 1 - get rnbwContract contract instance

Choose a reason for hiding this comment

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

question - is this code forked?

If not, I guess we need to probably create a unit test here?

Choose a reason for hiding this comment

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

Screen Shot 2022-08-16 at 11 43 56 AM

I'm having issues in using the command yarn hardhat xav-airdrop --network mainnet :(

Copy link
Contributor

@chris247474 chris247474 left a comment

Choose a reason for hiding this comment

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

@tracyarciaga i'll leave it to others to do a deeper review, but i did some quick manual searches of xRNBW holders i know (some of our seed investors) and this address for example 0xC8E9bd09A38d27331DC5bDC2a7d4EB254F2aA61C isn’t in the xRNBW airdrop list (the one you sent me https://docs.google.com/spreadsheets/d/11G_UJ2_Wjf8BB0uY5wRFPKWHyFjUFzdiG_X9v9WbIlw/edit#gid=424991193) even if it was transferred xRNBW before Aug 5 8am UTC

@@ -0,0 +1,10 @@
export const internalRnbwHolders = [
Copy link
Contributor

Choose a reason for hiding this comment

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

@tracyarciaga the comments say xRNBW but the filenames and variables say rnbw. which one are we getting?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ctverceles I fixed the naming conventions
For the sheet it should be this one -> https://docs.google.com/spreadsheets/d/11G_UJ2_Wjf8BB0uY5wRFPKWHyFjUFzdiG_X9v9WbIlw/edit#gid=1457496626

const csvExporter = new ExportToCsv(options)
const airdropList: AirdropList[] = []
const addresses: string[] = []
const snapshotBlockNumber = 15281097
Copy link
Contributor

Choose a reason for hiding this comment

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

@tracyarciaga how did we get this btw? is this aug 5 8am UTC?

Copy link
Contributor Author

@tracyarciaga tracyarciaga Aug 16, 2022

Choose a reason for hiding this comment

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

@ctverceles Yes it's Aug-05-2022 08:00:12 AM +UTC here
https://etherscan.io/block/15281097
The block before that is Aug-05-2022 07:59:59 AM +UTC

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