Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Deployment in all networks #2258

Merged
merged 3 commits into from
Jan 22, 2022
Merged

Deployment in all networks #2258

merged 3 commits into from
Jan 22, 2022

Conversation

anxolin
Copy link
Contributor

@anxolin anxolin commented Jan 22, 2022 •
edited
Loading

Summary

This PR setups the addresses for the 3 available claimings: Rinkeby, Gnosis Chain and Mainnet.

To Test

What networks can be tested?

Rinkeby, Gnosis Chain and Mainnet

What users can we use for testing?

  • Dummies: just some 10K accounts generated with a single menmonic.
  • Real data: Real claimings (or close enough, since there might be small changes in the final deployment). There's a partition on the accounts depending on the airdrop amount (<10K cow goest to Gnosis Chain, otherwise Mainnet)

Where can I find the dummy address to test

The dummies allow you to test any of the different kinds of claimings.

What about the PK?

  • Ask @anxolin or @fedgiac , otherwise the mnemonic is shared in 1password with all the team

On each network:

  1. [ ]Use the dummies accounts, or your REAL account to test
    • Real in GC
    • Dummies in GC (all roles)
  2. Make sure we test also Gnosis Safe account
    • Gnosis Safe in Mainnnet
    • Gnosis Safe in GC?

@github-actions
Copy link
Contributor

  • 🔭 GP Swap: CoW Protocol v2 Swap UI

@anxolin anxolin requested review from alfetopito, W3stside, fedgiac, nenadV91 and a team January 22, 2022 17:10
@anxolin anxolin marked this pull request as ready for review January 22, 2022 17:11
@anxolin
Copy link
Contributor Author

anxolin commented Jan 22, 2022

I think after some minimal validation, we can merge this branch to develop and make these contracts our default testing contract.

fedgiac
fedgiac reviewed Jan 22, 2022
View reviewed changes
Copy link
Contributor

@fedgiac fedgiac left a comment

Choose a reason for hiding this comment

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

The addresses and the branch look right. ✔️
Just a comment on the hardcoded constants.

@anxolin anxolin merged commit b9523af into develop Jan 22, 2022
maria-vslvn pushed a commit that referenced this pull request Jan 25, 2022
@lint-action @callil
feat: routing api integration (#2116)
cd22955 
* initial routing api integration

* add routing api slice

* display route in dialog

* addressed pr feedback

* improved routing

* switch to `get`

* first pass at integration new MultiRouteTrade

* initial implementation of RoutingDiagram

* add RoutingDiagram tests

* improve tests in RoutingDiagram

* integrate with v3-sdk 3.3.1

* removed references to MultiRouteTrade

* revert swapcallback

* fix abi compilation error

* added useRoute hook to build a Route from edges and nodes

* added react-hooks-testing-library

* integrated latest changes

* renamed router hook to routerTrade

* improve integration

* fixed routing

* usability

* mock RoutingDiagram children to reduce size

* undo mocked children

* adjust ui

* better support long routes

* use routing api logo and adjust ux

* set default percent to 0

* added intermediary hook to combine local and routing api trades

* added intermediary hook to combine local and routing api trades

* make account optional

* improve ux

* improve router

* fixed duplicate pool bug and inputAmount undefined bug

* extract input/outputAmounts from routes

* add todo

* fixed uninitialized issue and added %

* fixed tests

* fix duplicate pool bug

* added routing api setting

* change router label based on router version

* improve useRoutes and fix duplicate pool bug

* debounce routing api/local routing

* removed single hop setting

* fix bug when moving between v2/v3

* consider isUnitialized non loading

* ui fixes

* reverted change to usedebounce

* use new route schema

* visual updates

* log quoteId for polish session

* fix: persist advanced swap details toggle state

* fix no route found

* poll every 10s

* derive currencies from pool rather than input

* polish query status handling in useRouterTrade

* removed RouterVersion

* update ui

* update ui

* update loading state

* animate auto router

* apply loading treatment to out

* disable routing api on l2 and support auto slippage

* use opacity on the whole element

* show loading card when syncing

* updated gradient

* polished ui

* create routerlabel component

* disable router on all bu mainnet

* polish

* feat: [draft] routing api polish (#2224)

* show loading card when syncing

* updated gradient

* polished ui

* create routerlabel component

* disable router on all bu mainnet

* polish

* polished loading state

* add dashes

* fixed tooltip styles

* fixed merge conflict

* few updates

* polish

* updated yarn.lock

* fixed styles

* updated routing diagram

* Fix code style issues with ESLint

* routing api enabled without localstorage upgrade

* fixed lint error

* Fix code style issues with ESLint

* refined mocks in routing diagram tests

* addressed pr feedback

* polish

* revert sending eth

* improved loading animation

* handle stale routing api

* Fix code style issues with ESLint

* updated yarn.lock

* support native eth

* Compute gas adjusted quote for V2 trade and compare to V3 gas adjusted quote

* Incorporate approval gas cost estimate

* feat: simplify routing api ux (#2258)

* support native eth

* simplified ui

* perf optimization

* implement realized lp fee

* improved route realized lp fee

* fix lp realized fee

* fix auto router gradient

* initial route overlay

* add auto router svg

* adjusted ux to mocks

* fix lp fee

* upddated routing diagram

* optimize tradeBetter hook

* adjust type and name

* add useBetterTrade

* useBetterTrade takes gasEstimateWei

* implement gasEstimateForApproval

* import state from react

* use gas estimate

* improve integration with gas estimate comparison

* remove dependency on account

* fix currency switch bug

* improve syncing state

* add loadingbar

* style tooltip container

* updated tooltip styles

* increase opacity range

* always keep dependent currency input interactable

* show placeholders in tooltips

* Revert v2 gas estimates and approval estimates

* Add debug logs

* refactor

* fix bug

* removed comment

* update engish key

* add try-catch

* addressed pr feedback

* remove loading bar for price impact

* addressed pr feedback and bug bash feedback

* fix: use url to force version

* addressed pr feedback and bug bash feedback

* stop fetching when losing focus

* only show auto router label when activated

* avoid showing syncing status

* move V3TradeSTate to own file

* make useRoutes a function rather than hook

* use logo from active list when possible

* renamed and refactored hook

* renamed and refactored hook

* update status

* polish

* remove unused import

* fixed merge error

* updated combined trade tests

* remove priceimpact while loading

* Design tweaks

* polish latest design

* removed some styles

* log gaevent on tooltip open and clean up origin

* Small tweaks

* addressed pr feedback

* wrap route length in a loading container

* renamed local to clientside

* fix percent and token logo

* addressed pr feedback

* avoid comparing trades when v3 not ready

* some refactor

Co-authored-by: Lint Action <[email protected]>
Co-authored-by: Will Pote <[email protected]>
Co-authored-by: Callil Capuozzo <[email protected]>
@alfetopito alfetopito mentioned this pull request Jan 25, 2022
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@alfetopito alfetopito alfetopito approved these changes

@fedgiac fedgiac fedgiac approved these changes

@W3stside W3stside Awaiting requested review from W3stside

@nenadV91 nenadV91 Awaiting requested review from nenadV91

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

Successfully merging this pull request may close these issues.
3 participants
@anxolin @alfetopito @fedgiac