Skip to content
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.

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

Flow Params can be missing #244

Open
JLSteenwyk opened this issue Mar 30, 2023 · 0 comments
Open

Flow Params can be missing #244

JLSteenwyk opened this issue Mar 30, 2023 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@JLSteenwyk
Copy link
Contributor

[REQUIRED] Describe the bug
When registering a workflow on the SDK, there is a check that all arguments have metadata Latch parameters. This is super helpful. However, when using Sections and Flow, Params can be missing and then will not be part of the workflow interface.

[REQUIRED] To Reproduce
Test out removing a Param.
For example, consider this flow that accounts for all parameters:
flow=[ Section( "Input/Output", Params( "input", "output", ), ), ]

Then, if "input" is removed:
flow=[ Section( "Input/Output", Params( "output", ), ), ]

Then the workflow interface will not have a place to specify the input.

[REQUIRED] Expected behavior
It would be awesome if an error were thrown during registration if a Param is missing in the flow Section.

[REQUIRED] Platform Information

  • macOS Monterey 12.5.1
  • Chrome: Version 111.0.5563.110
  • Latch, version 2.15.0
@JLSteenwyk JLSteenwyk added the bug Something isn't working label Mar 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants