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

Feature: address book + new send flow #1239

Merged
merged 160 commits into from
Jan 22, 2020
Merged

Conversation

estebanmino
Copy link
Contributor

@estebanmino estebanmino commented Dec 6, 2019

Description

Write a short description of the changes included in this pull request.

Checklist

  • There is a related GitHub issue
  • Tests are included if applicable
  • Any added code is fully documented

Issue

Resolves #128

@ibrahimtaveras00
Copy link
Contributor

ibrahimtaveras00 commented Jan 14, 2020

All above issues have been fixed 👍

Issue 10:

placeholder text is cut off on android devices (checked Pixel 3 and Nexus S)

image

Issue 11:

The tappable area for changing to currency is hard on the real iPhone SE, I can't seem to tap it.
Also, I accidentally tapped "Next" CTA and can't really see the error message since the keyboard is expanded with no real way to dismiss keyboard

image

Issue 12:

Nexus S (4 inch screen) collectible title

image

Issue 13:

Currently on playstore/appstore, if you scan a public address QA code, it pre-fills in the to field, however, on this branch I get sent to the send view without the address pre-filled; seen here = http://recordit.co/jqwo40zf9n

@ibrahimtaveras00
Copy link
Contributor

Issue 14:

Adding an account and removing an account in Settings > Contacts not updating correctly; seen here = https://recordit.co/73q6byZQVR

steps:

  • go to settings > contacts
  • I already have 1 account stored (esteban.team.mask.eth)
  • I then attempt to add another contact by scanning a QR code from a separate account on a different wallet
  • for the account name, I call it 228A Account
  • once added, Esteban shows twice
  • I have to leave back to settings then come back to contacts in order for it to display correctly
  • however, when I attempt to delete the newly added account, it makes it look like I deleted the Esteban account when I didn't until I leave and come back it displays correctly

Issue 15:

Transaction error

image

Flow seen here = https://recordit.co/OOpgd447A5

Steps:
use this seed phrase small sock office dragon oval word approve material borrow spread amazing eagle
send max to any account (rinkeby) and go through the flow

@ibrahimtaveras00
Copy link
Contributor

ibrahimtaveras00 commented Jan 22, 2020

Issues 10 - 13 have been resolved 👍

Issue 14 = Originally reported bug is fixed, however, it seems as though when I have 0 accounts and I attempt to add my first account, my contacts list isn't updated until I leave Contacts and go back into Contacts

seen here = https://recordit.co/g36Ea1MHs9 (0:01 - 0:49sec)

Issue 15 = Still happening

After doing 1 last run through, I noticed this:

Issue 16: = I was able to add a contact with a space as a character, as well as an emoji, so when it appeared on the list, there was no icon for the one with the space, and a different icon for the one with the emoji, do we want to group them under the "others"?

seen here = https://recordit.co/EBjVH4vwdh

image

Copy link
Contributor

@ibrahimtaveras00 ibrahimtaveras00 left a comment

Choose a reason for hiding this comment

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

All fixes look good on both OS's, QA Passed 👍

@estebanmino estebanmino merged commit e0c0b07 into develop Jan 22, 2020
@estebanmino estebanmino deleted the feature/address-book branch January 22, 2020 21:59
rickycodes pushed a commit that referenced this pull request Jan 31, 2022
* basic components

* basic components

* updates

* wip

* between my accounts

* basic add contact

* render list

* inputs func components

* handle from selection

* from identicon

* onchangetoaddress

* add to address book modal

* add from modal and parse asdress book

* scan qr

* select own accounts

* reverse ens and better state

* handle parse

* clear to

* handle account label

* next and redux

* handle to ens

* parse address book

* handle send flow navigation

* amount view

* render only tokens

* pick selected asset

* redux set selected address

* use max

* switch currency

* confirm

* clean

* init confirm

* confirm view

* render fiat in confirm

* handle eth txs

* handle erc20 txs

* parseTransactionData

* wait for gas estimation

* custom gas modal

* handle gas selection

* handle advanced tx fee

* custom gas improvs

* handle gas selected

* handle current selected when cancel

* use time estimates

* handle amount errors

* address search

* typo

* recents

* parsedRecents

* cleaner code

* handle address errors

* handle more error cases

* fix usemax

* handle global primary currency

* handle use max with fiat as prim currency

* switch currency working

* set asset before and fix switched symbol

* render gas wait time

* improve gas selectors

* add weeks and days

* better parsewaittime

* fix selected asset change

* improve conversion

* parseTransactionData
"

* validate gas on confirm

* handle use max with gas for eth

* add gas estimation in amount

* estimatedTotalGas once on mount

* send tx

* gas bug on amount

* render collectibles in list

* handle collectibles list and data

* handle collectible on confirm

* ignore No stops in gradient

* handle focus

* handle tradable collectibles and sort them

* handle tokens without exchange rate

* check collectible removal

* start tx from asset

* go to new send flow from everywhere

* hex data modal

* reset tx on unmount

* delete unused files

* reset tx

* start tx with asset

* redux update

* clean state

* isETH

* highlight address to

* use currency symbols and texttransfor uppercase

* handle currency code in amount input

* send to up to date design

* amount up to date design

* confirm up to date design

* send to up to date design android

* next buttons margin bottom

* input amount font

* fix texts color

* fix androi dpaste

* alignments

* handle mmm txs

* handle ens name when resolved

* formatting and locales

* add contacts from settings

* no need for edit component

* handle add validation

* locales

* rename route

* delete contact

* spanish

* handle scan and android'

* more locales

* address ready when edit

* snappppppsSs

* others

* fix reverse ens

* disable add address without an alias

* fixes

* address elemenr

* snaps

* custom gas validation

* locale missing

* check decimal

* comments

* snaps

* fix key

* fix assets

* fix use mac

* renderFiatAddition

* fix empty data

* snaps

* fix gas calculation for erc721

* rm logs

* validation

* handle change from without asset

* use flat list

* rm unused code

* rename

* fix delete

* fix empty bug

* fix margin issue 10

* issue 11

* 12

* send qr to old sendflow 13

* snaps

* fix others contacts

* fix edit

* use mac

* lte

* bn(0)

* fix 13

* handle space and emojis

* fix validation

* contact form with refs

* snapS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-qa Any New Features that needs a full manual QA prior to being added to a release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create address book
3 participants