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.

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

feat: add base #162

Merged
merged 4 commits into from
Nov 27, 2024
Merged

feat: add base #162

merged 4 commits into from
Nov 27, 2024

Conversation

alfetopito
Copy link
Contributor

Description

Add Base network support

Changes

  • Bump cow-sdk version

How to test

  1. Use the following config:
{
  "networks": [
    {
      "name": "base",
      "rpc": "<set a base rpc here>",
      "deploymentBlock": 21794150,
      "processEveryNumBlocks": 5,
      "pageSize": 5000,
      "filterPolicy": {
        "defaultAction": "ACCEPT",
        "conditionalOrderIds": {},
        "transactions": {},
        "handlers": {},
        "owners": {}
      }
    }
  ]
}
  1. Run it with yarn cli
  • Should start and process blocks accordingly
    image

Note: Not yet able to place TWAP orders, so can't be fully tested yet

@alfetopito alfetopito self-assigned this Nov 5, 2024
@alfetopito alfetopito requested a review from a team November 5, 2024 17:52
Copy link

socket-security bot commented Nov 5, 2024

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

Package New capabilities Transitives Size Publisher
npm/@cowprotocol/cow-sdk@5.7.1 network 0 1.39 MB cowprotocol_dev

🚮 Removed packages: npm/@cowprotocol/cow-sdk@5.3.0

View full report↗︎

Copy link
Contributor

@anxolin anxolin left a comment

Choose a reason for hiding this comment

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

It works!

@anxolin
Copy link
Contributor

anxolin commented Nov 27, 2024

Backend will deploy tomorrow to production, so its good to move this forward.

@anxolin
Copy link
Contributor

anxolin commented Nov 27, 2024

I also tested running it locally and pointing to barn. Merging!

@anxolin anxolin merged commit fc2d231 into main Nov 27, 2024
4 checks passed
@anxolin anxolin deleted the feat/base branch November 27, 2024 17:19
@github-actions github-actions bot locked and limited conversation to collaborators Nov 27, 2024
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