add memo warning to transfer input

mike-dydx commented 4 months ago

Links (dYdX Internal Use Only)

Linear Ticket: MOB-536 : [iOS] Add memo for token transfer (non-USDC only)

Figma Design: https://www.figma.com/design/fMQodZKzn9B5aZTN5G0fLJ/dYdX-%E2%80%BA-Web?node-id=32590-11221&t=Po1xjzMkEe27WgXz-4

Description / Intuition

adds memo field to transfer
adds warning to display when memo field is not filled

Before/After Screenshots or Videos

Before	After

Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactoring or Technical Debt
[ ] Documentation update
[ ] Other (please describe: )

linear[bot] commented 4 months ago

MOB-536 [iOS] Add memo for token transfer (non-USDC only)

prashanDYDX commented 4 months ago

design notes:

"Required for Centralized Exchanges" text color doesn't match the designs, should be more subtle.
X button has too much space on the right

ruixhuang commented 4 months ago

IOS should show the paste option when tap-and-hold the text input, so we can do without the paste button.

prashanDYDX commented 4 months ago

IOS should show the paste option when tap-and-hold the text input, so we can do without the paste button.

Yeah I was thinking this as well - though having the Paste button maybe does help a bit to enforce the idea that this field should be a value copied from the CEX, not some random thing you fill out on your own?

dydxprotocol / v4-native-ios