Give server rowIndex for validateRow trigger

[bjosttveit]

Description

• When using triggers=["validateRow"] the rowIndex will now be sent to the server for validations. Docs have been updated to reflect this.
• No longer filters out server-validations not related to the current row on validateRow. This was the behaviour only a few days ago, and has been changed back since it is totally legitimate to add validation errors outside of the current group row. The developer is now expected to use the rowIndex in servervalidations to deal with this if necessary. Not really a breaking change, as the validateRow trigger is basically brand new.
• Previously, the group validation trigger would not dispatch an update to validations unless there were new errors present. This was a problem because the server could send back validations that had been fixed, and they would not be updated if there were no new errors. This has been changed so that it always dispatches an update to validations. Also should not be a breaking change, since dispatching no new errors should not make a difference, and no existing tests failed 🤷

Related Issue(s)

• Slack-thread

Verification

• Manual testing
  • I have tested these changes manually
  • Creator of the original issue (or service owner) has been contacted for manual testing
  • No testing done/necessary
• Automated tests
  • Unit test(s) have been added
  • Cypress E2E test(s) have been added
  • No automatic tests are needed here
  • I want someone to help me make some tests
• User documentation @ altinn-studio-docs
  • Has been updated
    • update docs with new RowIndex header altinn-studio-docs#754
  • No changes/updates needed
• Changes/additions to component properties
  • Changes are reflected in both layout/index.ts and layout.schema.v1.json, and these are all backwards-compatible
  • No changes made
• Support in Altinn Studio
  • Issue(s) created for support in Studio
  • This change/feature does not require any changes to Altinn Studio
• Sprint board
  • The original issue (or this PR itself) has been added to the Team Apps project and to the current sprint board

[ivarne]

Can’t there be multiple row indexes? Like in a nested group?

[bjosttveit]

Can’t there be multiple row indexes? Like in a nested group?

Right, I suppose you do need the parent's row-index as well if a nested group has the trigger

[sonarqubecloud]

SonarCloud Quality Gate failed.   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

0.0% Coverage
0.0% Duplication