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.

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

swaps v2 gas #5526

Merged
merged 27 commits into from
Apr 3, 2024
Merged

swaps v2 gas #5526

merged 27 commits into from
Apr 3, 2024

Conversation

dereknelson
Copy link
Contributor

@dereknelson dereknelson commented Mar 18, 2024 •
edited

Fixes APP-1229

What changed (plus any additional context for devs)

mostly utils & types

Screen recordings / screenshots

image

What to test

@linear Linear
Copy link

linear bot commented Mar 18, 2024

APP-1229 Implement gas estimations

@dereknelson dereknelson requested a review from walmat March 18, 2024 16:21
@dereknelson dereknelson marked this pull request as ready for review March 20, 2024 19:13
@socket-security Socket Security
Copy link

socket-security bot commented Mar 28, 2024 •
edited

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@ledgerhq/errors@6.16.3 None 0 190 kB ldg-github-ci
npm/@ledgerhq/hw-transport@6.30.5 None +1 331 kB ldg-github-ci
npm/@metamask/utils@8.4.0 None +5 832 kB metamaskbot
npm/@noble/hashes@1.4.0 None 0 773 kB paulmillr
npm/@scure/base@1.1.6 None 0 80.4 kB paulmillr
npm/array-includes@3.1.8 None +1 35.5 kB ljharb
npm/es-abstract@1.23.2 None +37 3.1 MB ljharb
npm/express@4.19.1 environment, filesystem, network +10 591 kB wesleytodd
npm/hasown@2.0.2 None +1 40.2 kB ljharb
npm/node-fetch-native@1.6.3 network 0 735 kB pi0
npm/object.values@1.2.0 None +1 40.7 kB ljharb
npm/ufo@1.5.3 None 0 103 kB pi0
npm/which-typed-array@1.1.15 None +4 92.7 kB ljharb

🚮 Removed packages: npm/@babel/code-frame@7.23.5, npm/@babel/compat-data@7.23.5, npm/@babel/generator@7.23.6, npm/@babel/helper-create-class-features-plugin@7.24.0, npm/@babel/helper-module-imports@7.22.15, npm/@babel/helpers@7.24.0, npm/@babel/highlight@7.23.4, npm/@babel/plugin-bugfix-safari-id-destructuring-collision-in-function-expression@7.23.3, npm/@babel/plugin-bugfix-v8-spread-parameters-in-optional-chaining@7.23.3, npm/@babel/plugin-bugfix-v8-static-class-fields-redefine-readonly@7.23.7, npm/@babel/plugin-syntax-export-default-from@7.23.3, npm/@babel/plugin-syntax-flow@7.23.3, npm/@babel/plugin-syntax-import-assertions@7.23.3, npm/@babel/plugin-syntax-import-attributes@7.23.3, npm/@babel/plugin-syntax-jsx@7.23.3, npm/@babel/plugin-syntax-typescript@7.23.3, npm/@babel/plugin-transform-arrow-functions@7.23.3, npm/@babel/plugin-transform-async-generator-functions@7.23.9, npm/@babel/plugin-transform-async-to-generator@7.23.3, npm/@babel/plugin-transform-block-scoped-functions@7.23.3, npm/@babel/plugin-transform-block-scoping@7.23.4, npm/@babel/plugin-transform-class-properties@7.23.3, npm/@babel/plugin-transform-class-static-block@7.23.4, npm/@babel/plugin-transform-classes@7.23.8, npm/@babel/plugin-transform-computed-properties@7.23.3, npm/@babel/plugin-transform-destructuring@7.23.3, npm/@babel/plugin-transform-dotall-regex@7.23.3, npm/@babel/plugin-transform-duplicate-keys@7.23.3, npm/@babel/plugin-transform-dynamic-import@7.23.4, npm/@babel/plugin-transform-exponentiation-operator@7.23.3, npm/@babel/plugin-transform-export-namespace-from@7.23.4, npm/@babel/plugin-transform-flow-strip-types@7.23.3, npm/@babel/plugin-transform-for-of@7.23.6, npm/@babel/plugin-transform-function-name@7.23.3, npm/@babel/plugin-transform-json-strings@7.23.4, npm/@babel/plugin-transform-literals@7.23.3, npm/@babel/plugin-transform-logical-assignment-operators@7.23.4, npm/@babel/plugin-transform-member-expression-literals@7.23.3, npm/@babel/plugin-transform-modules-amd@7.23.3, npm/@babel/plugin-transform-modules-commonjs@7.23.3, npm/@babel/plugin-transform-modules-systemjs@7.23.9, npm/@babel/plugin-transform-modules-umd@7.23.3, npm/@babel/plugin-transform-new-target@7.23.3, npm/@babel/plugin-transform-nullish-coalescing-operator@7.23.4, npm/@babel/plugin-transform-numeric-separator@7.23.4, npm/@babel/plugin-transform-object-rest-spread@7.24.0, npm/@babel/plugin-transform-object-super@7.23.3, npm/@babel/plugin-transform-optional-catch-binding@7.23.4, npm/@babel/plugin-transform-optional-chaining@7.23.4, npm/@babel/plugin-transform-parameters@7.23.3, npm/@babel/plugin-transform-private-methods@7.23.3, npm/@babel/plugin-transform-private-property-in-object@7.23.4, npm/@babel/plugin-transform-property-literals@7.23.3, npm/@babel/plugin-transform-regenerator@7.23.3, npm/@babel/plugin-transform-reserved-words@7.23.3, npm/@babel/plugin-transform-shorthand-properties@7.23.3, npm/@babel/plugin-transform-spread@7.23.3, npm/@babel/plugin-transform-sticky-regex@7.23.3, npm/@babel/plugin-transform-template-literals@7.23.3, npm/@babel/plugin-transform-typeof-symbol@7.23.3, npm/@babel/plugin-transform-typescript@7.23.6, npm/@babel/plugin-transform-unicode-escapes@7.23.3, npm/@babel/plugin-transform-unicode-property-regex@7.23.3, npm/@babel/plugin-transform-unicode-regex@7.23.3, npm/@babel/plugin-transform-unicode-sets-regex@7.23.3, npm/@babel/preset-flow@7.24.0, npm/@babel/preset-typescript@7.23.3, npm/@ledgerhq/errors@6.16.2, npm/@ledgerhq/hw-transport@6.30.4, npm/@metamask/utils@8.3.0, npm/@scure/base@1.1.5, npm/@webassemblyjs/ast@1.11.6, npm/@webassemblyjs/wasm-parser@1.11.6, npm/array-includes@3.1.7, npm/core-js-compat@3.36.0, npm/enhanced-resolve@5.15.1, npm/es-abstract@1.22.5, npm/express@4.18.3, npm/fast-xml-parser@4.3.5, npm/hasown@2.0.1, npm/node-fetch-native@1.6.2, npm/object.values@1.1.7, npm/redis-errors@1.2.0, npm/safe-array-concat@1.1.0, npm/ufo@1.4.0, npm/which-typed-array@1.1.14

View full report↗︎

skylarbarrera
skylarbarrera suggested changes Apr 2, 2024
View reviewed changes
src/__swaps__/references/index.ts Outdated Show resolved Hide resolved
src/__swaps__/screens/Swap/components/GasButton.tsx Show resolved Hide resolved
src/__swaps__/screens/Swap/components/GasButton.tsx Outdated Show resolved Hide resolved
src/__swaps__/screens/Swap/components/GasButton.tsx Outdated Show resolved Hide resolved
src/__swaps__/screens/Swap/components/GasButton.tsx Outdated Show resolved Hide resolved
src/__swaps__/screens/Swap/components/GasButton.tsx Outdated Show resolved Hide resolved
src/__swaps__/utils/gasUtils.ts Outdated Show resolved Hide resolved
src/__swaps__/utils/time.ts Show resolved Hide resolved
src/state/gas/gasStore.ts Outdated Show resolved Hide resolved
src/state/gas/gasStore.ts Outdated Show resolved Hide resolved
skylarbarrera
skylarbarrera reviewed Apr 3, 2024
View reviewed changes
Copy link
Contributor

@skylarbarrera skylarbarrera left a comment

Choose a reason for hiding this comment

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

shore

@skylarbarrera skylarbarrera self-requested a review April 3, 2024 21:29
@dereknelson
Copy link
Contributor Author

@skylarbarrera admin merge king??

@skylarbarrera skylarbarrera merged commit e148fa5 into develop Apr 3, 2024
5 of 6 checks passed
@skylarbarrera skylarbarrera deleted the @derek/gasv2 branch April 3, 2024 22:20
BrodyHughes added a commit that referenced this pull request Apr 10, 2024
@BrodyHughes
Merge branch 'develop' of github.com:rainbow-me/rainbow into brody/e2…
76addf0 
…e-changes

* 'develop' of github.com:rainbow-me/rainbow: (44 commits)
  allow open in new tab (#5610)
  added warning for unknown price impact (#5597)
  fix cloudflare protection (#5609)
  improve type checking on web preferences (#5607)
  fix scrolltoindex firing on last card dismissal (#5606)
  make account network switcher work (#5604)
  Dapp browser fixes (#5596)
  Fix close tab btn (#5598)
  browser: add account context menu (#5603)
  Fix instant screenshot setting (#5602)
  swaps: bump sdk (#5583)
  Browser: tab transitions, state update queue (#5582)
  audit: undici (#5594)
  [SWAPS V2]: Add token search logic and ability to select assets (#5547)
  swaps v2 gas (#5526)
  tx sim: other natives (#5585)
  Brody/bump 1.9.21 3 (#5588)
  fix sheet bg (#5590)
  Only hold the active tab ref in BrowserContext (#5579)
  Dapp browser: disable tab closing for empty state (#5573)
  ...
BrodyHughes added a commit that referenced this pull request Apr 10, 2024
@BrodyHughes
Merge branch 'develop' of github.com:rainbow-me/rainbow into develop
2a839e5 
* 'develop' of github.com:rainbow-me/rainbow:
  make account network switcher work (#5604)
  Dapp browser fixes (#5596)
  Fix close tab btn (#5598)
  browser: add account context menu (#5603)
  Fix instant screenshot setting (#5602)
  swaps: bump sdk (#5583)
  Browser: tab transitions, state update queue (#5582)
  audit: undici (#5594)
  [SWAPS V2]: Add token search logic and ability to select assets (#5547)
  swaps v2 gas (#5526)
  tx sim: other natives (#5585)
  Brody/bump 1.9.21 3 (#5588)
  fix sheet bg (#5590)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skylarbarrera skylarbarrera skylarbarrera approved these changes

@benisgold benisgold Awaiting requested review from benisgold benisgold is a code owner

@jinchung jinchung Awaiting requested review from jinchung jinchung is a code owner

@walmat walmat Awaiting requested review from walmat walmat is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@dereknelson @walmat @skylarbarrera