-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Update ContentItemVersionId nullable in graphql #16030
Conversation
How do you get to this point, what's |
Ha, in case you haven't noticed, OC's graphql Api already supports |
Yeah, but what exactly is |
Hi @Piedone , {
"name": "",
"displayName": "",
"description": "",
"author": "",
"website": "",
"version": "",
"issetuprecipe": false,
"categories": [],
"tags": [],
"steps": [
{
"name": "Feature",
"enable": [
"OrchardCore.Users.CustomUserSettings",
"OrchardCore.Apis.GraphQL"
]
},
{
"name": "ContentDefinition",
"ContentTypes": [
{
"Name": "UserProfile",
"DisplayName": "UserProfile",
"Settings": {
"ContentTypeSettings": {
"Creatable": true,
"Listable": true,
"Draftable": true,
"Versionable": true,
"Stereotype": "CustomUserSettings",
"Securable": true
},
"FullTextAspectSettings": {
"IncludeFullTextTemplate": false,
"IncludeBodyAspect": true,
"IncludeDisplayText": true
},
"GraphQLContentTypeSettings": {
"Hidden": false
}
},
"ContentTypePartDefinitionRecords": [
{
"PartName": "UserProfile",
"Name": "UserProfile",
"Settings": {
"ContentTypePartSettings": {
"Position": "0"
}
}
}
]
},
{
"Name": "UserProfile2",
"DisplayName": "UserProfile2",
"Settings": {
"ContentTypeSettings": {
"Creatable": true,
"Listable": true,
"Draftable": true,
"Versionable": true,
"Stereotype": "CustomUserSettings",
"Securable": true
},
"FullTextAspectSettings": {
"IncludeFullTextTemplate": false,
"IncludeBodyAspect": true,
"IncludeDisplayText": true
},
"GraphQLContentTypeSettings": {
"Hidden": false
}
},
"ContentTypePartDefinitionRecords": [
{
"PartName": "UserProfile2",
"Name": "UserProfile2",
"Settings": {
"ContentTypePartSettings": {
"Position": "0"
}
}
}
]
}
],
"ContentParts": [
{
"Name": "UserProfile",
"Settings": {},
"ContentPartFieldDefinitionRecords": [
{
"FieldName": "TextField",
"Name": "Field1",
"Settings": {
"ContentPartFieldSettings": {
"DisplayName": "Field1",
"Position": "0"
}
}
},
{
"FieldName": "TextField",
"Name": "Field2"
}
]
},
{
"Name": "UserProfile2",
"Settings": {},
"ContentPartFieldDefinitionRecords": [
{
"FieldName": "TextField",
"Name": "TextField1",
"Settings": {
"ContentPartFieldSettings": {
"DisplayName": "TextField1",
"Position": "0"
}
}
}
]
}
]
}
]
} |
Have you enabled the "Custom User Settings" feature? Or use the above my updated recipe |
Because you haven't enabled the "Custom User Settings" feature, the user information page doesn't display these panels, but when you do enable it, you'll be able to see these panels, and the display driver will automatically handle the save logic, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, of course, I see. Looks good, then!
Why not assign a new id to it then and let it non-nullable? |
In the me field, the user may check contentItemVersionId, but we don't assign a value to it.