Skip to content

Fix no FilterParams in new clob version - #71

Open
LimitedInfo wants to merge 1 commit into
Polymarket:mainfrom
LimitedInfo:main
Open

Fix no FilterParams in new clob version#71
LimitedInfo wants to merge 1 commit into
Polymarket:mainfrom
LimitedInfo:main

Conversation

@LimitedInfo

Copy link
Copy Markdown

Fixes no FilterParams in new clob version by changing to version that contains that class

Overview

on fresh install the 0.17 version of clob api doesn't have the FilterParams class

Fixes no FilterParams in new clob version by changing to version that contains that class

@jfullerton44 jfullerton44 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fixed build issue for me

@jfullerton44

Copy link
Copy Markdown

Put up more updated fix #73

@acebeakerxyz

Copy link
Copy Markdown

Same issue. Will downgrade to fix

@acebeakerxyz

Copy link
Copy Markdown

Looks like FilterParams was split into OpenOrderParams and TradeParams. Should be an easy fix

@corneliusroemer

Copy link
Copy Markdown

Resolves #74

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants