liftedinit / alberto

A crypto wallet for any ledger supporting the Many protocol
https://alberto.app
Apache License 2.0
6 stars 7 forks source link

feat: Rework "Send" flow #138

Closed stanleyjones closed 1 year ago

stanleyjones commented 1 year ago

Closes #134 .

Before

No Asset Selected

Screenshot 2023-09-07 at 11 11 50 AM

Asset Selected

Screenshot 2023-09-07 at 11 13 22 AM

Now

No Asset Selected

Screenshot 2023-09-07 at 10 57 49 AM

Asset Selected

Screenshot 2023-09-07 at 10 58 08 AM

(I added the "required" asterisk after taking these screenshots.)

netlify[bot] commented 1 year ago

Deploy Preview for lifted-alberto ready!

Name Link
Latest commit 06235fab713fbe820dd931543bef00f7723829d6
Latest deploy log https://app.netlify.com/sites/lifted-alberto/deploys/64ff197e55739b0008d3f893
Deploy Preview https://deploy-preview-138--lifted-alberto.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

stanleyjones commented 1 year ago

@fmorency This one is also ready for review. I think my organization permissions must have changed because I can't request reviewers or self-assign anymore.

codecov[bot] commented 1 year ago

Codecov Report

Merging #138 (06235fa) into main (983fe14) will increase coverage by 0.14%. The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main     #138      +/-   ##
==========================================
+ Coverage   33.93%   34.08%   +0.14%     
==========================================
  Files          88       88              
  Lines        2260     2265       +5     
  Branches      656      659       +3     
==========================================
+ Hits          767      772       +5     
  Misses       1482     1482              
  Partials       11       11              
Files Changed Coverage Ξ”
.../transactions/components/send-asset/send-asset.tsx 64.12% <66.66%> (+1.42%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

stanleyjones commented 1 year ago

Ah crap, I tried to get rid of the extra commits and made it worse. Hold on.