search

Altinn / altinn-cdn

Static assets used by Altinn solutions and apps.
https://altinncdn.no
1 stars 31 forks source link

Set "additionalProperties": false where it makes sense #59

Closed ivarne closed 2 years ago

ivarne commented 3 years ago

When additionalProperties is not set, the user will not get warnings for typos. or subfields of typos.

This might potentially break something if schemas are used for validation and invalid schemas are in use. It might also make it somewhat harder for local testing of new properties, if strict validation is required for some operation. I have not seen any instances where strict validation causes failiures, but I have not tried everything.

ivarne commented 3 years ago

This requires more work. Lots of the existing components only works because of the missing additionalProperties: false declaration.