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

vCow contracts #2029

Merged
merged 6 commits into from
Dec 22, 2021
Merged

vCow contracts #2029

merged 6 commits into from
Dec 22, 2021

Conversation

alfetopito
Copy link
Contributor

Summary

Added vCow related stuff

  • ABI

  • Generated types

  • Temporary Rinkeby address

  • Hook

    To Test

No testing possible yet

@alfetopito alfetopito requested a review from a team December 21, 2021 17:38
@alfetopito alfetopito self-assigned this Dec 21, 2021
@github-actions
Copy link
Contributor

  • 🔭 GP Swap: Gnosis Protocol v2 Swap UI

@W3stside
Copy link
Contributor

@alfetopito does the auto-generated type stuff have auto compile in package json? where is it coming from

@alfetopito
Copy link
Contributor Author

@alfetopito does the auto-generated type stuff have auto compile in package json? where is it coming from

No, because the default path is for Uniswap contracts.
I manually changed it to /custom to generate them => https://github.com/gnosis/cowswap/blob/claim/package.json#L159

I don't think we need to re-generate them every time and also it would overwrite some custom changes we need, namely exporting the originally generated exports.

@alfetopito alfetopito merged commit b1628bb into claim Dec 22, 2021
@alfetopito alfetopito deleted the vcow-contracts branch December 22, 2021 13:20
nenadV91 pushed a commit that referenced this pull request Jan 3, 2022
* revert to remove ui polish changes

* left align blurb and fix bp issue

* fix translation wrapper

* remove preview screen for create, hide submitted txn UI for create, copy updates

* small fixes

* copy updates for create blurb

* add set price range title

* add translation

* fix some translations

Co-authored-by: Justin Domingue <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants