Skip to content
This repository was archived by the owner on Sep 11, 2025. It is now read-only.

Support optional params#269

Merged
JairusSW merged 13 commits intomainfrom
jairus/support-optional-params
Jul 9, 2024
Merged

Support optional params#269
JairusSW merged 13 commits intomainfrom
jairus/support-optional-params

Conversation

@JairusSW
Copy link
Contributor

@JairusSW JairusSW commented Jul 8, 2024

Completes HYP-1618

Requires hypermodeinc/functions-as#122

@JairusSW JairusSW requested a review from a team July 8, 2024 21:56
@JairusSW JairusSW requested a review from mattjohnsonpint July 8, 2024 22:01
@JairusSW JairusSW enabled auto-merge (squash) July 8, 2024 22:01
@linear
Copy link

linear bot commented Jul 8, 2024

HYP-1618 Support optional parameters in the Runtime

When a param metadata is optional:true, then the field should be optional in the generated graphql schema, which would mean a nullable type like Int instead of the default non-nullable type Int!

We also need to update the way functions are invoked, such that the default value is used when not supplied. This should align with changes from HYP-1617.

@hypermodeinc hypermodeinc deleted a comment from linear bot Jul 8, 2024
@mattjohnsonpint mattjohnsonpint changed the title Jairus/support optional params Support optional params Jul 8, 2024
JairusSW and others added 6 commits July 8, 2024 18:53
Co-authored-by: Matt Johnson-Pint <mjp@hypermode.com>
Co-authored-by: Matt Johnson-Pint <mjp@hypermode.com>
@JairusSW JairusSW merged commit 3e1247f into main Jul 9, 2024
@JairusSW JairusSW deleted the jairus/support-optional-params branch July 9, 2024 21:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants