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

chore: update 0x deps #659

Merged
merged 2 commits into from
Jun 28, 2021
Merged

chore: update 0x deps #659

merged 2 commits into from
Jun 28, 2021

Conversation

kimpers
Copy link
Contributor

@kimpers kimpers commented Jun 22, 2021

Description

This bumps all @0x dependencies to the latest version (which includes Lido support) and fixes issues arising from swapping to Ajv for json schemas.

Depends on 0xProject/tools#38.

Testing Instructions

Checklist

  • Update documentation as needed. Website Documentation PR:

  • Prefix PR title with [WIP] if necessary.

  • Add tests to cover changes as needed.

  • Test changes on the staging environment with Matcha API Staging.

    • SRA/Limit orders
    • Swap endpoints
    • Meta transaction endpoints
    • Depth charts

    For more information see 0x API Matcha smoke test runbook in Quip.

@kimpers kimpers changed the title chore: update 0x deps & add Lido support chore: update 0x deps and add Lido support Jun 22, 2021
@kimpers kimpers marked this pull request as ready for review June 23, 2021 09:49
@kimpers kimpers requested a review from xianny June 23, 2021 15:46
test/sra_test.ts Outdated
reason: "should have required property 'baseToken'",
},
{
field: 'quoteToken', // FIXME (xianny): bug in jsonschemas module
Copy link
Contributor

Choose a reason for hiding this comment

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

you can remove my comment since you removed the ugly instance. prefix!

Copy link
Contributor

@xianny xianny left a comment

Choose a reason for hiding this comment

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

looks great, thanks so much for fixing this!

@romainbutteaud
Copy link
Contributor

thanks Kim, make sure you don't update asset-swapper and protocol-utils cause I already did it in https://github.com/0xProject/0x-api/pull/660/files

@kimpers kimpers changed the title chore: update 0x deps and add Lido support chore: update 0x deps Jun 28, 2021
@kimpers
Copy link
Contributor Author

kimpers commented Jun 28, 2021

deploy staging

@kimpers kimpers merged commit 2e301f9 into master Jun 28, 2021
@kimpers kimpers deleted the kim/feat/lido-deposit branch June 28, 2021 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants