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

System fields are editable #6684

Closed
supermx1 opened this issue Jul 7, 2021 · 5 comments · Fixed by #6806
Closed

System fields are editable #6684

supermx1 opened this issue Jul 7, 2021 · 5 comments · Fixed by #6806
Assignees
Labels
Milestone

Comments

@supermx1
Copy link

supermx1 commented Jul 7, 2021

Describe the Bug

So this going to sound funny. So I wanted to adjust the layout (full width or half-width) and translation of some fields in the Directus_users collection. This small change literally causes the fields or should say any field I adjust (either layer or translation) to turn into an input field (This includes fields like language, 2fa, tags, etc). Now they all don't work. The data still gets submitted to the database, but it is not as it should be.

I don't know if it is a bug or a feature, but it sure feels unsafe to use.

Screen Shot 2021-07-07 at 1 44 45 AM

Screen Shot 2021-07-07 at 1 45 43 AM

To Reproduce

Go to the directus_user collection and adjust either layout or transalation. Especially special fields like drop downs, 2fa etc

What version of Directus are you using?

RC.83

What version of Node.js are you using?

14.15.3

What database are you using?

10.4.17-MariaDB - Source distribution

What browser are you using?

Chrome

What operating system are you using?

MacOS

How are you deploying Directus?

Digital Ocean

@rijkvanzanten rijkvanzanten changed the title Making small changes in Directus_users breaks it Some parts of system fields are editable Jul 7, 2021
@rijkvanzanten rijkvanzanten changed the title Some parts of system fields are editable System fields are editable Jul 7, 2021
@rijkvanzanten rijkvanzanten added this to the v9.0.0-rc.84 milestone Jul 7, 2021
@rijkvanzanten rijkvanzanten self-assigned this Jul 7, 2021
@rijkvanzanten
Copy link
Member

System fields aren't (supposed to be) user editable 👍🏻

@connorwinston
Copy link
Member

@rijkvanzanten Me being the fool I am, also messed with the system fields and was trying to just hide some fields in the app. And now it has made my entire Directus instance come to a halt.

Any idea how I can just restore it to how it was? I just clicked hide within app on preferences_divider, theme, and tfa and now every page in Directus errors.

Version: ^9.0.0-rc.85

image

image

image

image

@rijkvanzanten
Copy link
Member

Removing all the records that relate to the directus_* collections from directus_fields should revert you back 👍🏻

@supermx1
Copy link
Author

@rijkvanzanten can we now edit directus_users? In the new RC85 without it breaking? I don’t want to risk testing it out. Lol. Thanks for the support mahn

@rijkvanzanten
Copy link
Member

You can (and always could) add/edit/remove custom fields. The bug was that the locked system fields rendered double which enabled editing the system fields, which can't be edited 👍🏻

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants