Bug/989 timing issue regarding updating data stateless app

[framitdavid]

Description

Ensure to cancel previous requests for stateless apps to avoid updating the redux state with staled data.

Related Issue(s)

• closes Timing issue regarding updating the data model (Stateless) #989

Verification/QA

• Manual functionality testing
  • I have tested these changes manually
  • Creator of the original issue (or service owner) has been contacted for manual testing (or will be contacted when released in alpha)
  • No testing done/necessary
• Automated tests
  • Unit test(s) have been added/updated
  • Cypress E2E test(s) have been added/updated
  • No automatic tests are needed here (no functional changes/additions)
  • I want someone to help me make some tests
• UU/WCAG (follow these guidelines until we have our own)
  • I have tested with a screen reader/keyboard navigation/automated wcag validator
  • No testing done/necessary (no DOM/visual changes)
  • I want someone to help me perform accessibility testing
• User documentation @ altinn-studio-docs
  • Has been added/updated
  • No functionality has been changed/added, so no documentation is needed
  • I will do that later/have created an issue
• Changes/additions to component properties
  • Changes are reflected in both src/layout/layout.d.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
  • I don't have permissions to do that, please help me out
• Labels
  • I have added a kind/* label to this PR for proper release notes grouping
  • I don't have permissions to add labels, please help me out

[olemartinorg]

Good stuff! 🚀 Maybe we could add a regression test for this using Cypress? We have a couple of stateless apps we could extend with some checkboxes/radios, and Cypress is definitely fast enough for us to not have to bother with any request throttling.. 😉 At some point, when we implement #345 we might not have sagas anymore, so it's important we don't end up re-implementing this bug at that point.

[framitdavid]

Good stuff! 🚀 Maybe we could add a regression test for this using Cypress? We have a couple of stateless apps we could extend with some checkboxes/radios, and Cypress is definitely fast enough for us to not have to bother with any request throttling.. 😉 At some point, when we implement #345 we might not have sagas anymore, so it's important we don't end up re-implementing this bug at that point.

Thanks! 🙏 I agree with you that it’s important to have a cypress test for this fix within one of our stateless apps. I fix.

[olemartinorg]

Test looks good! 🚀

[sonarqubecloud]

SonarCloud Quality Gate failed.   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

16.7% Coverage
0.0% Duplication