-
Notifications
You must be signed in to change notification settings - Fork 467
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
feat(edit content) #27289 : Add a button to the Content Type screen to enable the new edit mode #27333
feat(edit content) #27289 : Add a button to the Content Type screen to enable the new edit mode #27333
Conversation
…o enable the new edit mode
…een-enable-new-edit-mode
…een-enable-new-edit-mode
dotCMS/src/main/java/com/dotmarketing/portlets/structure/model/Structure.java
Outdated
Show resolved
Hide resolved
dotCMS/src/main/java/com/dotmarketing/portlets/structure/model/Structure.java
Outdated
Show resolved
Hide resolved
…een-enable-new-edit-mode
…een-enable-new-edit-mode
@hmoreras conflictos :D |
…een-enable-new-edit-mode
...app/portlets/shared/dot-content-types-edit/components/form/content-types-form.component.scss
Outdated
Show resolved
Hide resolved
core-web/libs/dotcms-scss/angular/dotcms-theme/components/form/_checkbox.scss
Outdated
Show resolved
Hide resolved
@@ -83,7 +83,7 @@ export class DotEditContentFormComponent implements OnInit { | |||
this.onFormChange(value); | |||
}); | |||
|
|||
this.formData.fields.forEach((field) => { | |||
this.formData.contentType.fields.forEach((field) => { |
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.
Does this mean that we are passing fields twice? formData.fields
and formData.contentType.fields
?
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.
And why the change?
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.
Because a change in EditContentPayload
before in that interface we had a property called layout
fields
and contentType
this last one as a string refereing to the conten type variable. All this props are actually in one single object, DotCMSContentType
, so move that to a single prop.
core-web/libs/edit-content/src/lib/feature/edit-content/edit-content.layout.component.scss
Outdated
Show resolved
Hide resolved
…een-enable-new-edit-mode
…een-enable-new-edit-mode
…een-enable-new-edit-mode
…een-enable-new-edit-mode
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
…f the `metadata` column
Closing this PR as the merge with the latest master caused several files to go missing |
Proposed Changes
structure
table calledmetadata
that will allow us to add any sort of permanent or temporary configuration parameters for every Content Type.Add bars to the edit content type dialog and the new edit content screen when the feature flag is on.
Screenshare.-.2024-01-19.10_24_42.PM.mp4