-
Notifications
You must be signed in to change notification settings - Fork 7
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
Import Airdrop recipients via DnD or copy-paste #2705
Conversation
…s when DAO doesn't have any fungible assets. Implement DnD file import for importing recipients for airdrop from CSV
… address input in airdrop modal
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @mudrila this looks great.
Add a ":" after "Available Balance" so it is "Available Balance:"
"Drag & drop a .csv file here, or click to select one"
"address,amount one entry per row"
When the number of recipients is 1, it should say "recipient" not "recipients"
Please factor these items in before merging. Feel free to ping me on Slack if you need anything so I can respond quickly.
Deploying decent-interface with
|
Latest commit: |
f90a095
|
Status: | ✅ Deploy successful! |
Preview URL: | https://435de895.decent-interface.pages.dev |
Branch Preview URL: | https://feature-airdrop-import-recip.decent-interface.pages.dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @mudrila lfg
Changes
Testing
Safe has no any ERC-20 tokens
Safe has ERC-20 tokens
Screenshots
@nicolaus-sherrill @parkermccurley Given there's no rush - would be nice to get some UI / UX / Copy comments to polish this thing