Component marked as not required: remove properties that is updated to not set (undefined)

[lassopicasso]

Description

When a component is set to not required, any updates to its properties that result in undefined are now removed. This ensures cleaner property configurations.

Related to the work with #12559

Related Issue(s)

• itself

Verification

• Your code builds clean without any errors or warnings
• Manual testing done (required)
• Relevant automated test added (if you find this hard, leave it and we'll help out)

Documentation

• User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.16%. Comparing base (93a4747) to head (db0ac1c).

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #12726   +/-   ##
=======================================
  Coverage   90.16%   90.16%           
=======================================
  Files        1304     1304           
  Lines       18673    18677    +4     
  Branches     2268     2269    +1     
=======================================
+ Hits        16837    16841    +4     
  Misses       1577     1577           
  Partials      259      259           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ghost]

LGTM 😄

[nkylstad]

Seems to work as expected for text/number input properties. F.ex. saveWhileTyping and maxLength.

Boolean properties (f.ex. renderAsSummary) are set to false when unchecked, but that it probably ok for now.

Another observation is that it is not possible to remove values that are set in a regular select. F.ex:

Out of scope for this issue I would say, but we should maybe look into a pattern on how to remove a property from the layout in general 🤔 What do you think @Annikenkbrathen?

Moving this to "Done", as the behaviour seems to work as expected!

[Annikenkbrathen]

Yes, certainly! I don't have a deep understanding of this yet, but we can create an issue and look into it. I need to know when to remove a property, why, and what happens. Are we removing it from a single field, component, or page? Which properties are we talking about?