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.

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

Removing a data model binding should delete the entire node #14296

Closed
mlqn opened this issue Dec 16, 2024 · 3 comments · Fixed by #14297
Closed

Removing a data model binding should delete the entire node #14296

mlqn opened this issue Dec 16, 2024 · 3 comments · Fixed by #14297
Labels
added-to-sprint area/ui-editor Area: Related to the designer tool for assembling app UI in Altinn Studio. frontend kind/bug Used when there is a defect / something is not working as it should. team/studio-domain2

Comments

@mlqn
Copy link
Contributor

mlqn commented Dec 16, 2024

Description of the bug

Removing a data model binding does not completely delete the associated node, which results in an error in the preview

Steps To Reproduce

datamodelbindings-delete.mov

In this example, these nodes should be removed:

"label": {
  "field": "",
  "dataType": ""
},
"metadata": {
  "field": "",
  "dataType": ""
}

Additional Information

No response

@mlqn mlqn added the kind/bug Used when there is a defect / something is not working as it should. label Dec 16, 2024
@mlqn mlqn added team/studio-domain2 area/ui-editor Area: Related to the designer tool for assembling app UI in Altinn Studio. labels Dec 16, 2024
@mlqn mlqn moved this to 📈 Todo in Team Studio Dec 16, 2024
@lassopicasso lassopicasso linked a pull request Dec 16, 2024 that will close this issue
4 tasks
@lassopicasso lassopicasso moved this from 📈 Todo to 🧪 Test in Team Studio Dec 17, 2024
@lassopicasso lassopicasso moved this from 🧪 Test to 🔎 Review in Team Studio Dec 17, 2024
@Jondyr Jondyr moved this from 🔎 Review to 🧪 Test in Team Studio Dec 17, 2024
@mlqn mlqn self-assigned this Dec 17, 2024
@mlqn
Copy link
Contributor Author

mlqn commented Dec 17, 2024

Tested in DEV. It seems that the issue still exists:

data-model-bindings.mov

@mlqn mlqn assigned lassopicasso and mlqn and unassigned mlqn and lassopicasso Dec 17, 2024
@mlqn
Copy link
Contributor Author

mlqn commented Dec 17, 2024

I'll see if I can fix it

@mlqn mlqn moved this from 🧪 Test to 👷 In Progress in Team Studio Dec 17, 2024
@lassopicasso
Copy link
Contributor

ah, I was pretty sure it worked yesterday testing locally. But I see now it doesn't in dev 😅 Thank you Michael for taking a look at it.

@mlqn mlqn moved this from 👷 In Progress to 🔎 Review in Team Studio Dec 17, 2024
@mlqn mlqn removed their assignment Dec 17, 2024
@mlqn mlqn moved this from 🔎 Review to 🧪 Test in Team Studio Dec 18, 2024
@github-project-automation github-project-automation bot moved this from 🧪 Test to Documentation in Team Studio Dec 18, 2024
@mlqn mlqn moved this from Documentation to ✅ Done in Team Studio Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
added-to-sprint area/ui-editor Area: Related to the designer tool for assembling app UI in Altinn Studio. frontend kind/bug Used when there is a defect / something is not working as it should. team/studio-domain2
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

2 participants