Skip to content

Exclude RFQT from ZeroEx sources [SIM-217] #110

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
Merged
merged 1 commit into from
Apr 7, 2022
Merged

Exclude RFQT from ZeroEx sources [SIM-217] #110

merged 1 commit into from
Apr 7, 2022

Conversation

cgewecke
Copy link
Contributor

@cgewecke cgewecke commented Apr 7, 2022 •
edited
Loading

0x pinged us in telegram asking for this:

Lawrence Forman, [Apr 7, 2022 at 8:50:28 AM]:
Hey guys, are you guys using RFQ liquidity in your API requests?
Actually, just to be safe, could you add excludedSources=RFQT until we update your ZeroExApiAdapter contract?

(Will need to publish this when merged, update the package at set-ui and push to production)

@cgewecke cgewecke changed the title Exclude RFQT from ZeroEx sources Exclude RFQT from ZeroEx sources [SIM-217] Apr 7, 2022
@cgewecke cgewecke merged commit 863df61 into master Apr 7, 2022
@cgewecke cgewecke deleted the chris/exclude-rfqt branch April 7, 2022 16:22
cgewecke added a commit that referenced this pull request Apr 13, 2022
@cgewecke
Revert "Exclude RFQT from ZeroEx sources (#110)"
dc5acd3 
This reverts commit 863df61.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@dalberto dalberto dalberto approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cgewecke @dalberto