search

sanger / stan-core

MIT License
0 stars 2 forks source link

x1167 Make lifestage optional #359

Closed khelwood closed 7 months ago

khelwood commented 7 months ago

The LifeStage field is now nullable in graphql and in the database. The client can offer an n/a option for life stage as long as it uses null in the request. (Making n/a an acceptable value in the LifeStage enum would have needed more extensive rewrite because a graphql enum cannot contain a slash.)

sabrine33 commented 7 months ago

This looks good to me, but there is no corresponding stan-sql pull request.

khelwood commented 7 months ago

This looks good to me, but there is no corresponding stan-sql pull request.

I added the patch file straight into the devel branch: https://github.com/sanger/stan-sql/blob/devel/schema/x1167_nullable_life_stage.sql