-
Notifications
You must be signed in to change notification settings - Fork 0
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
Role + Description fields in People collection #204
Comments
I can't say I fully understand the directus schema yet, but from how I understand it we have the following situation. Maybe we can move the @jstet Do you think its feasible? Schema changes are of course always painful, but the further I get with error handling and logging the less painful it will get (yet we won't reach painless 😅). If its not feasible we can brainstorm more about client side solutions. |
@frie and I talked about this yesterday in a call. I did not add description and position to people, because people can have different positions and descriptions in different contexts, e.g. as a part of the remote office vs lc head vs content creator. We agreed to postbone this for now and see how much of an issue this is in writing content in the cms |
yes for postponing it after the launch. just to note: also came up with Content Creators today (description wise) |
note from #555 in my opinion the use case is not worth the complexity and added effort of maintaining them in different places. For blog posts, temporal stability is actually more of a problem (aka: Johannes 8 years ago has the same description as content creator Johannes today) but again, for me, i'd keep it as simple as possible. If you think this has implications too far for a PR. i can move my comment to the related issue and revert the changes (i guess i am just frustrated that i manually added things in the wrong place in directus) |
As I have just updated the board i had to think about this again. Personally I still think it makes sense to have Any new thoughts @friep or @KonradUdoHannes ? |
could we have a general description and role field in the People collection? then we could reuse this in different collections that inherit from the People collection instead of copy-pasting the description to different components/collections.
e.g. when I insert a Contact in the page builder and I don't see the need to write a dedicated text, i have to copy paste from the global admin.
maybe it could be so that the Title & description in this case could be optional and then:
or is this too confusing? 😬
The text was updated successfully, but these errors were encountered: