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.

Sign up for GitHub

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

Jump to bottom

Restructure entire repo #23

Merged

rkalis merged 9 commits into master from staging

Nov 11, 2022

Member

rkalis commented Nov 9, 2022

Restructure utils.ts into separate files
Merge all confirm_xxx.tsx pages into one confirm.tsx page
Move async fetches (e.g. token data) client side so popup is snappier
Add full typings for all previously any types
Make code DRYer

rkalis added 9 commits

November 8, 2022 00:40


          Restructure utils

20867a3


          Refactor into common WarningText component

ff97a3e


          Refactor confirm_allowance.tsx into Header/AllowanceInfo/WarningControls

299f2a4


          Merge all confirm pages into one and move async calls client-side

70e2d55

- Merge confirm-allowance, confirm-hash-signature and confirm-listing
- Move all async calls (e.g. getTokenData) to the React pages to improve
  popup times


          Add types for messages & refactor some code to match that

64101a2


          Dry some wet code

21a326d


          Dry some more wet code

2dc9fa8


          Update translations

f9c80c7


          Update localazy config

bc3cd20

rkalis merged commit 496645b into master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment