Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Row UUIDs no longer included for data models changed by actions #883

Closed
olemartinorg opened this issue Nov 5, 2024 · 1 comment · Fixed by #885
Closed

Row UUIDs no longer included for data models changed by actions #883

olemartinorg opened this issue Nov 5, 2024 · 1 comment · Fixed by #885
Labels
kind/bug Something isn't working status/triage

Comments

@olemartinorg
Copy link
Contributor

Description of the bug

When adding new rows using a IUserAction, these new rows in the data model does not get any generated altinnRowId properties. This breaks app-frontend in strange ways, such as not showing the rows at all (and stalling the whole UI) until the user refreshes app-frontend.

Steps To Reproduce

  1. Use project references in ttd/frontend-test to use the latest app-lib-dotnet main
  2. Enable the 'shifting options' page
  3. Click the button to add 10 rows
  4. Observe that nothing happens

Additional Information
@olemartinorg olemartinorg added the kind/bug Something isn't working label Nov 5, 2024
@olemartinorg
Copy link
Contributor Author

Btw, I was unable to reproduce this in v8.3.7.

@ivarne ivarne mentioned this issue Nov 6, 2024
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/bug Something isn't working status/triage
Projects
Team Apps
Status: No status
Milestone
No milestone
1 participant
@olemartinorg