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

Fix invoke: app ID default and data type #186

Merged

amh4r merged 2 commits into main from fix-invoke

Dec 6, 2024

Collaborator

amh4r commented Dec 6, 2024

Bug fixes:

Fix step.invoke using None instead of defaulting to the parent function's app ID.
Fix step.invoke not ensure that data is a dict.

Tests:

Add test for invoke + encryption. There isn't a known bug, but it's good to test it.

Tooling:

Enable auto-formatting on save.


          Fix invoke: app ID default and data type

6e1b562

amh4r requested review from cdzombak and jpwilliams

December 6, 2024 20:40


          Temporarily increase max request body size

d5995e8

tonyhb approved these changes

View reviewed changes

amh4r merged commit 02bdcbb into main

10 checks passed

amh4r deleted the fix-invoke branch

December 6, 2024 21:33

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

Labels

None yet