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

Added Container listing table and Filters #22920

Merged
merged 13 commits into from
Sep 26, 2022

Conversation

motasimvd
Copy link
Contributor

@motasimvd motasimvd commented Sep 2, 2022

Proposed Changes

  • Added Container listing table to Container list page
  • Added Filters to Container listing

Checklist

  • Tests
  • Translations
  • Security Implications Contemplated (add notes if applicable)

Additional Info

  • New page is located at <your_url>/dotAdmin/#/container-new

Screenshots

Original Updated
** original screenshot ** ** updated screenshot **

image

@motasimvd motasimvd changed the title Feature/container listing Feature/container listing (Copy) Sep 2, 2022
@motasimvd motasimvd changed the title Feature/container listing (Copy) [WIP] Feature/container listing (Copy) Sep 2, 2022
@motasimvd motasimvd changed the title [WIP] Feature/container listing (Copy) Added Container listing table and Filters Sep 12, 2022
actionHeaderOptions,
tableColumns
}: DotContainerListState) => {
this.containerBulkActions = containerBulkActions;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be a select in the component store that you use directly in your template.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

tableColumns
}: DotContainerListState) => {
this.containerBulkActions = containerBulkActions;
this.selectedContainers = selectedContainers;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This prop and all the releated logic belongs to the store.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

}: DotContainerListState) => {
this.containerBulkActions = containerBulkActions;
this.selectedContainers = selectedContainers;
this.addToBundleIdentifier = addToBundleIdentifier;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure why, but I can't find this one being used.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

Comment on lines 73 to 74
this.actionHeaderOptions = actionHeaderOptions;
this.tableColumns = tableColumns;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be a select in the component store that you use directly in your template.

Probably just one select that includes actionHeaderOptions , containerBulkActions and this one.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated


constructor(
private store: DotContainerListStore,
private route: ActivatedRoute,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since the data is coming now from the CS, this should be move into store as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

Comment on lines 52 to 53
this.store.updateTableColumns(this.setContainerColumns());
this.store.updateContainerBulkActions(this.setContainerBulkActions());
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't update the state twice because it will trigger the change detection twice.

Everything you are doing here, which is unit the state, can be done in the store itself as far as I can see.

The setContainerColumns and setContainerBulkActions can be done in the store.

More info: https://ngrx.io/guide/component-store/initialization#lazy-initialization

constructor(
private store: DotContainerListStore,
private dotMessageService: DotMessageService,
private dotAlertConfirmService: DotAlertConfirmService,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not being used.

@Injectable()
export class DotContainerListStore extends ComponentStore<DotContainerListState> {
constructor(private route: ActivatedRoute) {
super(defaultState);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines 55 to 83
this.containerBulkActions$ = this.vm$.pipe(
takeUntil(this.destroy$),
pluck('containerBulkActions')
);

this.tableColumns$ = this.vm$.pipe(takeUntil(this.destroy$), pluck('tableColumns'));

this.actionHeaderOptions$ = this.vm$.pipe(
takeUntil(this.destroy$),
pluck('actionHeaderOptions')
);

this.addToBundleIdentifier$ = this.vm$.pipe(
takeUntil(this.destroy$),
pluck('addToBundleIdentifier')
);

this.vm$
.pipe(takeUntil(this.destroy$))
.subscribe(({ selectedContainers }: DotContainerListState) => {
this.selectedContainers = selectedContainers;
});

this.vm$
.pipe(takeUntil(this.destroy$))
.subscribe(({ isEnterPrise, hasEnvironments }: DotContainerListState) => {
this.isEnterPrise = isEnterPrise;
this.hasEnvironments = hasEnvironments;
});
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of thing all this extra properties you can use the state from the store directly in the template:

<ng-container *ngIf="vm$ | async as vm">
    {{vm.containerBulkActions}}
    {{vm. selectedContainers}}
</ng-container>

And so on.

https://ngrx.io/guide/component-store/usage

image

Comment on lines 34 to 41
this.init();

this.route.data
.pipe(pluck('dotContainerListResolverData'), take(1))
.subscribe(([isEnterprise, hasEnvironments]: [boolean, boolean]) => {
this.updateResolvedProperties([isEnterprise, hasEnvironments]);
this.setContainerBulkActions();
});
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here you are updating the state three times and that means 3 change detection, you can do this in one pass.

});
}

private setContainerColumns(): DataTableColumn[] {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
private setContainerColumns(): DataTableColumn[] {
private getContainerColumns(): DataTableColumn[] {

* @returns { [key: string]: string }
* @memberof DotContainerListStore
*/
private setStateLabels = () => {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
private setStateLabels = () => {
private getStateLabels = () => {

];
}

private setActionHeaderOptions(): ActionHeaderOptions {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
private setActionHeaderOptions(): ActionHeaderOptions {
private getActionHeaderOptions(): ActionHeaderOptions {

};
};

private setLicenseAndRemotePublishContainerBulkOptions({
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
private setLicenseAndRemotePublishContainerBulkOptions({
private getLicenseAndRemotePublishContainerBulkOptions({

Comment on lines 187 to 193
const state = this.get();
this.dotPushPublishDialogService.open({
assetIdentifier: state.selectedContainers
.map((container) => container.identifier)
.toString(),
title: this.dotMessageService.get('contenttypes.content.push_publish')
});
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
const state = this.get();
this.dotPushPublishDialogService.open({
assetIdentifier: state.selectedContainers
.map((container) => container.identifier)
.toString(),
title: this.dotMessageService.get('contenttypes.content.push_publish')
});
const { selectedContainers } = this.get();
this.dotPushPublishDialogService.open({
assetIdentifier: selectedContainers
.map((container) => container.identifier)
.toString(),
title: this.dotMessageService.get('contenttypes.content.push_publish')
});

Comment on lines 202 to 205
const state = this.get();
this.updateBundleIdentifier(
state.selectedContainers.map((container) => container.identifier).toString()
);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
const state = this.get();
this.updateBundleIdentifier(
state.selectedContainers.map((container) => container.identifier).toString()
);
const { selectedContainers } = this.get();
this.updateBundleIdentifier(
selectedContainers.map((container) => container.identifier).toString()
);

Comment on lines +7 to +17
archived?: boolean;
categoryId?: string;
deleted?: boolean;
live?: boolean;
working?: boolean;
friendlyName?: string;
path?: string;
identifier: string;
name: string;
type: string;
title?: string;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you sure they are all optionals?

@@ -36,8 +36,11 @@ export class DotContainerListStore extends ComponentStore<DotContainerListState>
this.route.data
.pipe(pluck('dotContainerListResolverData'), take(1))
.subscribe(([isEnterprise, hasEnvironments]: [boolean, boolean]) => {
this.updateResolvedProperties([isEnterprise, hasEnvironments]);
this.setContainerBulkActions();
this.updateResolvedProperties([
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are still hitting the state twice, in the init and here and you have all to do it once.

Comment on lines 65 to 74
readonly updateResolvedProperties = this.updater<[boolean, boolean, MenuItem[]]>(
(state: DotContainerListState, [isEnterprise, hasEnvironments, containerBulkActions]) => {
return {
...state,
isEnterprise,
hasEnvironments,
containerBulkActions
};
}
);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This and init method can be one.

@fmontes fmontes marked this pull request as ready for review September 19, 2022 14:30
@fmontes fmontes merged commit 9c9637a into dotCMS:canary Sep 26, 2022
fmontes pushed a commit that referenced this pull request Sep 30, 2022
* [WIP] Added Container listing and filters to Container list page

* [WIP] Refactor Container list to use already provided Container Interface

* [WIP] Container status is now displayed correctly

* [WIP] Updated unit tests and minor refactor

* Fixed unit test and listing data table

* Added template outlet before listing table search, Container list style and ui refactor

* Added strings to Language file and updated string keys in Container List component

* Added Component Store to Container Component

* Modified container list component to better use selectors

* moved activeRoue logic to store

* Refactored ContainerList functionality to ContainerListStore

* Method name refactor and State update minimisation

* Fixed state initialization
fmontes added a commit that referenced this pull request Nov 7, 2022
* Create empty container portlet (#22783)

* copy container rest endpoint added with curl tests (#22944)

* copy container rest endpoint added with curl tests

* Addressed MR comments

* Addressed MR commentsby removing uuid validation as it is not needed

* fixed by importing required code

* Addressed PR comments by removing UUIDUtil import

* Added Tabs to Container Create Page and updated language strings (#22932)

* Added Tabs to Container Create Page and updated translation strings

* Removed unused comment

* Bulk delete container (#22973)

* bulk delete rest endpoint

* curl tests for bulk delete

* more ids added for bulk delete operation

* updated authorization user's password

* Container bulk operations (#23012)

* Container bulk publish/unpublish rest endpoints

* Container bulk archive/unarchive rest endpoints

* curl/postman tests for container bulk publish

* curl/postman tests for all bulk operations

* fixed failed test

* Container permission and history iframes (#23025)

* Added Container listing table and Filters (#22920)

* [WIP] Added Container listing and filters to Container list page

* [WIP] Refactor Container list to use already provided Container Interface

* [WIP] Container status is now displayed correctly

* [WIP] Updated unit tests and minor refactor

* Fixed unit test and listing data table

* Added template outlet before listing table search, Container list style and ui refactor

* Added strings to Language file and updated string keys in Container List component

* Added Component Store to Container Component

* Modified container list component to better use selectors

* moved activeRoue logic to store

* Refactored ContainerList functionality to ContainerListStore

* Method name refactor and State update minimisation

* Fixed state initialization

* Categories permissions iframe added (#23032)

* code adjustments as per error handling standard (#23031)

* Feature/container lisiting complete actions (#23030)

* [WIP] Added Container listing and filters to Container list page

* [WIP] Refactor Container list to use already provided Container Interface

* [WIP] Container status is now displayed correctly

* [WIP] Updated unit tests and minor refactor

* Fixed unit test and listing data table

* Added template outlet before listing table search, Container list style and ui refactor

* Added strings to Language file and updated string keys in Container List component

* [WIP] Added actions and Container service to handle actions

* Updated unit tests

* Added Component Store to Container Component

* Modified container list component to better use selectors

* moved activeRoue logic to store

* Refactored ContainerList functionality to ContainerListStore

* Method name refactor and State update minimisation

* Fixed state initialization

* [WIP] Added context menu actions and updated tests

* Fixed Unit tests

* Fixed Unit tests

* Update core-web/apps/dotcms-ui/src/app/api/services/dot-router/dot-router.service.ts

Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>

* Update core-web/apps/dotcms-ui/src/app/portlets/dot-containers/container-list/container-list.component.ts

Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>

* Context menu actions hooked up to container service, Updated unit tests

* Fixed store logic and fixed typos

* Refactored container list component methods to container list store, Fixed typos, Updated container routes

Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>

* Category rest endpoints for Add category, Update category and get children categories (#23029)

* Add category rest endpoint

* curl test fix

* rest endpoint added to get children categories by parent inode

* curl/postman tests

* addressed PR comments and also renamed API from children to _children

* reverted API name from _children to children

* Updated API documentation

* Code changes as per suggestions from code reviewing team

* removed unnecessary imports

* Update category rest endpoint added

* little changes

* fix for the curl test

* Delete category rest endpoint (#23080)

* Delete category rest endpoint

* addressed feedback from BE team

* Feature/container properties form (#23020)

* Added Tabs to Container Create Page and updated translation strings

* Added container properties component and added Inplace editor to it

* Removed unused comment

* [WIP] Properties form UI updated

* Added Monaco Editor to Properties Form

* add history and permission iframe

* refactoring

* create edit route for container and resolver

* Added controls to container form

* change component and folders name

* Added method docs, Added translations, Minor fixes

* Added missing translations

* Added MonacoEditor type model for reusability

* CSS refactor

* CSS refactor

* Renamed container create component as DotContainerCreateComponent

* Component and SCSS refactor

* Refactored Loop editor to new component

* Update core-web/apps/dotcms-ui/src/app/portlets/dot-containers/dot-container-create/dot-container-history/dot-container-history.component.ts

Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>

* Update core-web/apps/dotcms-ui/src/app/portlets/dot-containers/dot-container-create/dot-container-permissions/dot-container-permissions.component.ts

Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>

* Refactored Content Editor to new component

* Fixed Permissions and History componanets

* change css style and remove unnecessary ng deep

* change css style and remove unnecessary ng deep

* Update core-web/apps/dotcms-ui/src/app/portlets/dot-containers/dot-container-create/dot-container-create.component.scss

Co-authored-by: Freddy Montes <freddymontes@gmail.com>

* Update core-web/apps/dotcms-ui/src/app/portlets/dot-containers/dot-container-create/dot-content-editor/store/dot-content-editor.store.ts

Co-authored-by: Freddy Montes <freddymontes@gmail.com>

* Removed unnecessary monaco editor initializations

* fixed box height issue

Co-authored-by: Zulqarnain Huda <zulqarnain.huda@venturedive.com>
Co-authored-by: zulqarnainvd <113915849+zulqarnainvd@users.noreply.github.com>
Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>
Co-authored-by: Freddy Montes <freddymontes@gmail.com>

* Rest endpoint to update categories for sortOrder (#23091)

* Rest endpoint to update categories for sortOrder

* updated credentials

* Addressed PR feedback

* add permissions and history tab and load Iframe in Tabs (#23092)

* add permissions and history tab and load Iframe in Tabs

* write test cases using host component method and use variable in scss

* Export categories rest endpoint (#23108)

* Export categories rest endpoint

* addressed PR feedback

* Add/Get container details (#23126)

* improvements in old code

* add container rest endpoint added

* Container details endpoint added to fetch container + structures info combinely

* curl test + improvements

* addressed PR feedback for _add rest endpoint

* removed details endpoint and added support in the live and working endpoint to fetch contentTypes

* code cleanup

* Import categories rest endpoint (#23125)

* Import categories rest endpoint

* addressed PR feedback on categories import

* addressed further feedback on categories import

* Addressed PR feedback

* converted categoryDTO to immutable

* container rest endpoint modified to accept container code changes (#23157)

* container rest endpoint modified to accept container code changes

* curl/postman test added

* Curl test fix (#23161)

* changed admin user's password

* fixed curl test

* updated tests assertions

* removed obsolete code/classes for categories (#23168)

* removed obsolete code/classes for categories

* Removed useless entries

* removed servlet-mapping from web.xml

* Feature/categories permissions (#23047)

* categories initialization

* fix unit test of category component

* add Category Table and actions

* add sub category beadcrumb and update sub category in table

* modify subCategoryUrl and fix typo in table

* add permission component and create a store

* change folder name and css class convention

* add docs and refactor the code

* add doc

* disable home button

* changing code according to suggestions and write unit test of categorly listing component

* add action header and Table UI of categories

* add pagination and search filter

* feat: working on categories

* feat: working on categories

* add pagination and search filter

* refactoring and change doc

* otCMS/core#23253 Categories: Fix the sort order in the endpoint

* Revert "removed obsolete code/classes for categories (#23168)" (#23288)

This reverts commit 2762511.

* Container Form (#23110)

* Added Simple form to container properties

* add confirmation dialog on clear

* Container can now be added using the simple form

* fix create container route issue and get data in container properties

* Added inplace data variable and fixed inplace CSS

* Form is now editable

* add Variable popup model

* Added form validation and disabled button on zero max contentlets

* Load content type from different source

* add content type to menu state

* get contentStructure value from child component to parent

* Variable modal now works

* Fixes to container add code

* Addressed Feedback

* Addressed Feedback

* set variable in editor

* Added update logic, Fixed failing unit tests

* Fixed faorm initilisation issue

* Addressed Feedback

* changes according to PR

* Fixed notification issue in container list

* Fixed container list empty messages and linked butoon to create container

* changes according to new API and requested PR changes

* changes according to PR and working fedback UI 3 issues

* Container Add form CSS Fixes

* Added label and CSS fixes

* Fixed clear button

* Title field now enabled on container Add form

* disable save button when required fields is empty or on edit value is not changed

* #23144 Added API Link Component, Fixed button text

* fix UI issues

* fix UI issues and test cases

* refactor code

* Fixed CSS, Updated url to load system containers

* Disabled interaction in container list for System Container

* Updated tests

* fix contentType Variable issue and container listing scroll issue

* Removed optional identifier from container Request

* fix reported issues

* Removed  from container properties template

* refactoring and change name of resolver

* PR requested changes

* remove validation of description field

* Conatiner Properties Form UI Fixes

* Implemented Control Value Accessor for DotLoopEditor Component

* Code impovements and fixes

* #23146 Refactored Content Editor component into a angular form control

* #23146 [WIP] Adding data to form control

* Rename code component

Co-authored-by: Zulqarnain Huda <zulqarnain.huda@venturedive.com>
Co-authored-by: Freddy Montes <751424+fmontes@users.noreply.github.com>

* get categories childrens and add breadcrumb (#23260)

* get categories childrens and add breadcrumb

* refactor: changes according to PR

* refactor: add unit test case of category listing component

* #23252 working on categories feedbacks and write test cases

* #23252 working on categories feedbacks and write test cases

* Fix tests build for containers

* Clean up model

* Clean up renamed container

* #23252 fix category filter issue (#23308)

* Added children count in the top level categories list (#23263)

* Added children count in the top level categories list

* added categories paginator for additional information

* Addressed PR feedback

* added line space at line number 94

* added java doc on the CategoryResource API + added postman tests

* Fix import

Co-authored-by: Daniel Silva <daniel.silva@dotcms.com>
Co-authored-by: hassan-mustafa-baig <111717530+hassan-mustafa-baig@users.noreply.github.com>
Co-authored-by: motasimvd <62100713+motasimvd@users.noreply.github.com>
Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>
Co-authored-by: Zulqarnain Huda <zulqarnain.huda@venturedive.com>
Co-authored-by: zulqarnainvd <113915849+zulqarnainvd@users.noreply.github.com>
jcastro-dotcms pushed a commit that referenced this pull request Nov 14, 2022
* Create empty container portlet (#22783)

* copy container rest endpoint added with curl tests (#22944)

* copy container rest endpoint added with curl tests

* Addressed MR comments

* Addressed MR commentsby removing uuid validation as it is not needed

* fixed by importing required code

* Addressed PR comments by removing UUIDUtil import

* Added Tabs to Container Create Page and updated language strings (#22932)

* Added Tabs to Container Create Page and updated translation strings

* Removed unused comment

* Bulk delete container (#22973)

* bulk delete rest endpoint

* curl tests for bulk delete

* more ids added for bulk delete operation

* updated authorization user's password

* Container bulk operations (#23012)

* Container bulk publish/unpublish rest endpoints

* Container bulk archive/unarchive rest endpoints

* curl/postman tests for container bulk publish

* curl/postman tests for all bulk operations

* fixed failed test

* Container permission and history iframes (#23025)

* Added Container listing table and Filters (#22920)

* [WIP] Added Container listing and filters to Container list page

* [WIP] Refactor Container list to use already provided Container Interface

* [WIP] Container status is now displayed correctly

* [WIP] Updated unit tests and minor refactor

* Fixed unit test and listing data table

* Added template outlet before listing table search, Container list style and ui refactor

* Added strings to Language file and updated string keys in Container List component

* Added Component Store to Container Component

* Modified container list component to better use selectors

* moved activeRoue logic to store

* Refactored ContainerList functionality to ContainerListStore

* Method name refactor and State update minimisation

* Fixed state initialization

* Categories permissions iframe added (#23032)

* code adjustments as per error handling standard (#23031)

* Feature/container lisiting complete actions (#23030)

* [WIP] Added Container listing and filters to Container list page

* [WIP] Refactor Container list to use already provided Container Interface

* [WIP] Container status is now displayed correctly

* [WIP] Updated unit tests and minor refactor

* Fixed unit test and listing data table

* Added template outlet before listing table search, Container list style and ui refactor

* Added strings to Language file and updated string keys in Container List component

* [WIP] Added actions and Container service to handle actions

* Updated unit tests

* Added Component Store to Container Component

* Modified container list component to better use selectors

* moved activeRoue logic to store

* Refactored ContainerList functionality to ContainerListStore

* Method name refactor and State update minimisation

* Fixed state initialization

* [WIP] Added context menu actions and updated tests

* Fixed Unit tests

* Fixed Unit tests

* Update core-web/apps/dotcms-ui/src/app/api/services/dot-router/dot-router.service.ts

Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>

* Update core-web/apps/dotcms-ui/src/app/portlets/dot-containers/container-list/container-list.component.ts

Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>

* Context menu actions hooked up to container service, Updated unit tests

* Fixed store logic and fixed typos

* Refactored container list component methods to container list store, Fixed typos, Updated container routes

Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>

* Category rest endpoints for Add category, Update category and get children categories (#23029)

* Add category rest endpoint

* curl test fix

* rest endpoint added to get children categories by parent inode

* curl/postman tests

* addressed PR comments and also renamed API from children to _children

* reverted API name from _children to children

* Updated API documentation

* Code changes as per suggestions from code reviewing team

* removed unnecessary imports

* Update category rest endpoint added

* little changes

* fix for the curl test

* Delete category rest endpoint (#23080)

* Delete category rest endpoint

* addressed feedback from BE team

* Feature/container properties form (#23020)

* Added Tabs to Container Create Page and updated translation strings

* Added container properties component and added Inplace editor to it

* Removed unused comment

* [WIP] Properties form UI updated

* Added Monaco Editor to Properties Form

* add history and permission iframe

* refactoring

* create edit route for container and resolver

* Added controls to container form

* change component and folders name

* Added method docs, Added translations, Minor fixes

* Added missing translations

* Added MonacoEditor type model for reusability

* CSS refactor

* CSS refactor

* Renamed container create component as DotContainerCreateComponent

* Component and SCSS refactor

* Refactored Loop editor to new component

* Update core-web/apps/dotcms-ui/src/app/portlets/dot-containers/dot-container-create/dot-container-history/dot-container-history.component.ts

Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>

* Update core-web/apps/dotcms-ui/src/app/portlets/dot-containers/dot-container-create/dot-container-permissions/dot-container-permissions.component.ts

Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>

* Refactored Content Editor to new component

* Fixed Permissions and History componanets

* change css style and remove unnecessary ng deep

* change css style and remove unnecessary ng deep

* Update core-web/apps/dotcms-ui/src/app/portlets/dot-containers/dot-container-create/dot-container-create.component.scss

Co-authored-by: Freddy Montes <freddymontes@gmail.com>

* Update core-web/apps/dotcms-ui/src/app/portlets/dot-containers/dot-container-create/dot-content-editor/store/dot-content-editor.store.ts

Co-authored-by: Freddy Montes <freddymontes@gmail.com>

* Removed unnecessary monaco editor initializations

* fixed box height issue

Co-authored-by: Zulqarnain Huda <zulqarnain.huda@venturedive.com>
Co-authored-by: zulqarnainvd <113915849+zulqarnainvd@users.noreply.github.com>
Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>
Co-authored-by: Freddy Montes <freddymontes@gmail.com>

* Rest endpoint to update categories for sortOrder (#23091)

* Rest endpoint to update categories for sortOrder

* updated credentials

* Addressed PR feedback

* add permissions and history tab and load Iframe in Tabs (#23092)

* add permissions and history tab and load Iframe in Tabs

* write test cases using host component method and use variable in scss

* Export categories rest endpoint (#23108)

* Export categories rest endpoint

* addressed PR feedback

* Add/Get container details (#23126)

* improvements in old code

* add container rest endpoint added

* Container details endpoint added to fetch container + structures info combinely

* curl test + improvements

* addressed PR feedback for _add rest endpoint

* removed details endpoint and added support in the live and working endpoint to fetch contentTypes

* code cleanup

* Import categories rest endpoint (#23125)

* Import categories rest endpoint

* addressed PR feedback on categories import

* addressed further feedback on categories import

* Addressed PR feedback

* converted categoryDTO to immutable

* container rest endpoint modified to accept container code changes (#23157)

* container rest endpoint modified to accept container code changes

* curl/postman test added

* Curl test fix (#23161)

* changed admin user's password

* fixed curl test

* updated tests assertions

* removed obsolete code/classes for categories (#23168)

* removed obsolete code/classes for categories

* Removed useless entries

* removed servlet-mapping from web.xml

* Feature/categories permissions (#23047)

* categories initialization

* fix unit test of category component

* add Category Table and actions

* add sub category beadcrumb and update sub category in table

* modify subCategoryUrl and fix typo in table

* add permission component and create a store

* change folder name and css class convention

* add docs and refactor the code

* add doc

* disable home button

* changing code according to suggestions and write unit test of categorly listing component

* add action header and Table UI of categories

* add pagination and search filter

* feat: working on categories

* feat: working on categories

* add pagination and search filter

* refactoring and change doc

* otCMS/core#23253 Categories: Fix the sort order in the endpoint

* Revert "removed obsolete code/classes for categories (#23168)" (#23288)

This reverts commit 2762511.

* Container Form (#23110)

* Added Simple form to container properties

* add confirmation dialog on clear

* Container can now be added using the simple form

* fix create container route issue and get data in container properties

* Added inplace data variable and fixed inplace CSS

* Form is now editable

* add Variable popup model

* Added form validation and disabled button on zero max contentlets

* Load content type from different source

* add content type to menu state

* get contentStructure value from child component to parent

* Variable modal now works

* Fixes to container add code

* Addressed Feedback

* Addressed Feedback

* set variable in editor

* Added update logic, Fixed failing unit tests

* Fixed faorm initilisation issue

* Addressed Feedback

* changes according to PR

* Fixed notification issue in container list

* Fixed container list empty messages and linked butoon to create container

* changes according to new API and requested PR changes

* changes according to PR and working fedback UI 3 issues

* Container Add form CSS Fixes

* Added label and CSS fixes

* Fixed clear button

* Title field now enabled on container Add form

* disable save button when required fields is empty or on edit value is not changed

* #23144 Added API Link Component, Fixed button text

* fix UI issues

* fix UI issues and test cases

* refactor code

* Fixed CSS, Updated url to load system containers

* Disabled interaction in container list for System Container

* Updated tests

* fix contentType Variable issue and container listing scroll issue

* Removed optional identifier from container Request

* fix reported issues

* Removed  from container properties template

* refactoring and change name of resolver

* PR requested changes

* remove validation of description field

* Conatiner Properties Form UI Fixes

* Implemented Control Value Accessor for DotLoopEditor Component

* Code impovements and fixes

* #23146 Refactored Content Editor component into a angular form control

* #23146 [WIP] Adding data to form control

* Rename code component

Co-authored-by: Zulqarnain Huda <zulqarnain.huda@venturedive.com>
Co-authored-by: Freddy Montes <751424+fmontes@users.noreply.github.com>

* get categories childrens and add breadcrumb (#23260)

* get categories childrens and add breadcrumb

* refactor: changes according to PR

* refactor: add unit test case of category listing component

* #23252 working on categories feedbacks and write test cases

* #23252 working on categories feedbacks and write test cases

* Fix tests build for containers

* Clean up model

* Clean up renamed container

* #23252 fix category filter issue (#23308)

* Added children count in the top level categories list (#23263)

* Added children count in the top level categories list

* added categories paginator for additional information

* Addressed PR feedback

* added line space at line number 94

* added java doc on the CategoryResource API + added postman tests

* Fix import

Co-authored-by: Daniel Silva <daniel.silva@dotcms.com>
Co-authored-by: hassan-mustafa-baig <111717530+hassan-mustafa-baig@users.noreply.github.com>
Co-authored-by: motasimvd <62100713+motasimvd@users.noreply.github.com>
Co-authored-by: Rafael Velazco <rjvelazco21@gmail.com>
Co-authored-by: Zulqarnain Huda <zulqarnain.huda@venturedive.com>
Co-authored-by: zulqarnainvd <113915849+zulqarnainvd@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants