betcode-org / betconnect

betconnect - API client
MIT License
7 stars 6 forks source link

Add missing IDs to MyActiveBet #50

Open mberk opened 1 year ago

mberk commented 1 year ago

Tests currently failing. Test data needs to be updated

mberk commented 1 year ago

I have made the IDs Optional as logically this makes sense if they've been added relatively recently and would not appear on older bets. Tests now pass locally. Failing GitHub Actions tests suggest the problem is nothing to do with these changes per se and rather with pydantic 2.0 and may be related to https://github.com/betcode-org/betconnect/issues/49