Skip to content

Update Observation Avalanche Problema Schema#1155

Open
kevinherdez wants to merge 1 commit intoNWACus:mainfrom
kevinherdez:kevinherdez/fixAvyProblemSchema
Open

Update Observation Avalanche Problema Schema#1155
kevinherdez wants to merge 1 commit intoNWACus:mainfrom
kevinherdez:kevinherdez/fixAvyProblemSchema

Conversation

@kevinherdez
Copy link
Copy Markdown
Collaborator

The d_size field for avalanche problems in Observations can contain null values. We don't show the avalanche problem in the Observation detail view so this will unblock observations from failing because of the schema mismatch.

@yuliadub
Copy link
Copy Markdown
Collaborator

The d_size field for avalanche problems in Observations can contain null values.

you are saying we have obs right now that do not have d_size but do have avalanche problems? Somehow I thought that all the forms had d_size be a required field so im confused but also it is very possible I am so wrong about that

@kevinherdez
Copy link
Copy Markdown
Collaborator Author

The d_size field for avalanche problems in Observations can contain null values.

you are saying we have obs right now that do not have d_size but do have avalanche problems? Somehow I thought that all the forms had d_size be a required field so im confused but also it is very possible I am so wrong about that

Observations contain both Avalanches and Avalanche Problems (Avalanche Problems is a part of the Advanced Fields). Both of these contain d_size but they're different fields. For Avalanches, d_size is a required string field (but it's marked optional because of NWAC), whereas on Avalanche Problems d_size is an option Array field that can contain string, number, or null in the array.

Avalanche Problems doesn't seem to be a very common field. In fact, we don't even show it in the app (could be something we add to get on par with web), but the issue in the schema is preventing observations from showing, if the array contains a null value

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants