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

Not possible to delete a field from a type that is in use #12201

Closed
nkylstad opened this issue Jan 30, 2024 · 4 comments · Fixed by #12235
Closed

Not possible to delete a field from a type that is in use #12201

nkylstad opened this issue Jan 30, 2024 · 4 comments · Fixed by #12235
Assignees
Labels
area/data-modeling Area: Related to data models - e.g. create, edit, use data models. kind/bug Used when there is a defect / something is not working as it should. status/triage

Comments

@nkylstad
Copy link
Member

Description of the bug

In the datamodels tool.

Steps To Reproduce

  1. Go to an app with a datamodel
  2. Create a type with several fields
  3. Drag the type into the data model so that the data model has a field that references the type
  4. Go back to the type
  5. See that the delete icon for the type is disabled as expected (since the type is in use)
  6. Try to delete one of the fields within the type
  7. See that the field is not deleted
  8. See that an error message is printed to the console

Additional Information

No response

@nkylstad nkylstad added kind/bug Used when there is a defect / something is not working as it should. status/triage area/data-modeling Area: Related to data models - e.g. create, edit, use data models. labels Jan 30, 2024
@nkylstad nkylstad moved this to 📈 Todo in Team Studio Jan 30, 2024
@JamalAlabdullah JamalAlabdullah moved this from 📈 Todo to ⚠️ Blocked in Team Studio Jan 31, 2024
@JamalAlabdullah JamalAlabdullah moved this from ⚠️ Blocked to 👷 In Progress in Team Studio Jan 31, 2024
@JamalAlabdullah JamalAlabdullah self-assigned this Feb 1, 2024
@JamalAlabdullah JamalAlabdullah moved this from 👷 In Progress to 📈 Todo in Team Studio Feb 1, 2024
@JamalAlabdullah JamalAlabdullah removed their assignment Feb 1, 2024
@framitdavid framitdavid self-assigned this Feb 1, 2024
@framitdavid framitdavid moved this from 📈 Todo to 👷 In Progress in Team Studio Feb 1, 2024
@framitdavid framitdavid moved this from 👷 In Progress to 🧪 Test in Team Studio Feb 2, 2024
@framitdavid framitdavid removed their assignment Feb 2, 2024
@nkylstad
Copy link
Member Author

nkylstad commented Feb 5, 2024

Ref. today's session on error messages 😊
I think we should look at the warning message here again.

A suggestion from me:
"Feltet du prøver å slette tilhører en aktiv type som er i bruk i datamodellen. Om du sletter dette feltet vil det slettes alle steder i datamodellen der denne typen er i bruk. Vil du slette feltet?"

Feel free to disagree or iterate on the suggestion 😊

EDIT: Tested actually deleting the field, which works as expected!

@nkylstad nkylstad moved this from 🧪 Test to 👀 Test feedback in Team Studio Feb 5, 2024
@framitdavid
Copy link
Collaborator

Ref. today's session on error messages 😊 I think we should look at the warning message here again.

A suggestion from me: "Feltet du prøver å slette tilhører en aktiv type som er i bruk i datamodellen. Om du sletter dette feltet vil det slettes alle steder i datamodellen der denne typen er i bruk. Vil du slette feltet?"

Feel free to disagree or iterate on the suggestion 😊

EDIT: Tested actually deleting the field, which works as expected!

Great suggestion! Fixed in this PR

@framitdavid framitdavid moved this from 👀 Test feedback to 🧪 Test in Team Studio Feb 5, 2024
@framitdavid framitdavid moved this from 🧪 Test to 👀 Test feedback in Team Studio Feb 5, 2024
@framitdavid framitdavid assigned framitdavid and unassigned nkylstad Feb 5, 2024
@framitdavid framitdavid moved this from 👀 Test feedback to 🧪 Test in Team Studio Feb 5, 2024
@framitdavid
Copy link
Collaborator

It should be ready for retest in minutes, #12245

@framitdavid framitdavid assigned nkylstad and unassigned framitdavid Feb 5, 2024
@nkylstad
Copy link
Member Author

nkylstad commented Feb 5, 2024

Great!

@nkylstad nkylstad closed this as completed Feb 5, 2024
@nkylstad nkylstad moved this from 🧪 Test to ✅ Done in Team Studio Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/data-modeling Area: Related to data models - e.g. create, edit, use data models. kind/bug Used when there is a defect / something is not working as it should. status/triage
Projects
Archived in project
3 participants