Closed goodoldneon closed 6 months ago
The if_exp field was being ignored because we didn't serialize it to if, which the Inngest server expects.
if_exp
if
The fixes are in these 2 files:
inngest/_internal/function_config.py
inngest/_internal/step_lib/base.py
The rest of the changes are for tests against regressions
We had to name the field if_exp instead of if because Python doesn't let you use statement keywords (if, for, etc.) as class fields names
for
Added automated tests that would've caught this
Description
The
if_exp
field was being ignored because we didn't serialize it toif
, which the Inngest server expects.The fixes are in these 2 files:
inngest/_internal/function_config.py
inngest/_internal/step_lib/base.py
The rest of the changes are for tests against regressions
Context
We had to name the field
if_exp
instead ofif
because Python doesn't let you use statement keywords (if
,for
, etc.) as class fields namesTesting
Added automated tests that would've caught this