Skip to content
This repository has been archived by the owner on Mar 14, 2024. It is now read-only.

Check contract details has time zone ID #663

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

djkelleher
Copy link

@djkelleher djkelleher commented Nov 26, 2023

Contracts about to expire may not have tradingHours, liquidHours, and timeZoneId set. This was causing an error when _parseSessions is called:

Exception has occurred: ValueError
ZoneInfo keys must be normalized relative paths, got: 

A simple way to avoid the error would be to check if the argument string is empty.

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.

1 participant