Skip to content

Use new item list implementation#315

Merged
nilmerg merged 22 commits into
mainfrom
use-new-item-list-implementation
Apr 25, 2025
Merged

Use new item list implementation#315
nilmerg merged 22 commits into
mainfrom
use-new-item-list-implementation

Conversation

@sukhwinder33445
Copy link
Copy Markdown
Contributor

No description provided.

@sukhwinder33445 sukhwinder33445 self-assigned this Apr 2, 2025
@cla-bot cla-bot Bot added the cla/signed CLA is signed by all contributors of a PR label Apr 2, 2025
@sukhwinder33445 sukhwinder33445 force-pushed the use-new-item-list-implementation branch 6 times, most recently from 2e6714c to 7c984c9 Compare April 9, 2025 10:51
@sukhwinder33445 sukhwinder33445 requested a review from nilmerg April 9, 2025 11:16
@sukhwinder33445 sukhwinder33445 force-pushed the use-new-item-list-implementation branch from 21d099e to cb4d748 Compare April 10, 2025 07:13
Comment thread library/Notifications/Widget/ItemList/LoadMoreObjectList.php Outdated
Comment thread library/Notifications/Widget/Detail/EventDetail.php Outdated
Comment thread public/css/common.less Outdated
Comment thread library/Notifications/View/ScheduleRenderer.php Outdated
Comment thread public/css/list/schedule-list.less
Comment thread application/controllers/SchedulesController.php Outdated
Comment thread library/Notifications/View/EventRenderer.php Outdated
Comment thread library/Notifications/View/IncidentRenderer.php Outdated
Comment thread library/Notifications/View/IncidentHistoryRenderer.php Outdated
@sukhwinder33445 sukhwinder33445 force-pushed the use-new-item-list-implementation branch from cb4d748 to 42dc33f Compare April 17, 2025 14:44
@jrauh01 jrauh01 mentioned this pull request Apr 24, 2025
@jrauh01
Copy link
Copy Markdown
Contributor

jrauh01 commented Apr 24, 2025

Fixes #330

Comment thread public/css/list/schedule-list.less Outdated
Comment thread public/css/common.less Outdated
Comment thread public/css/list/item-list.less Outdated
- Links: Add missing channel links
- Remove obsolete code
- Links: Add missing contactAdd() method
- Remove obsolete code
- Remove obsolete code
- Remove obsolete code
@sukhwinder33445 sukhwinder33445 force-pushed the use-new-item-list-implementation branch 2 times, most recently from 6e5c73f to 75ac72a Compare April 25, 2025 12:26
- Remove obsolete code
- Adjust Schedule-list.less accordingly
- Remove obsolete code
- Remove obsolete code
- ObjectList: Add method to disable action-list functionality
- Remove obsoltete code
This padding is now handled by ipl-web

- Optimize code
@sukhwinder33445 sukhwinder33445 force-pushed the use-new-item-list-implementation branch from 75ac72a to decc109 Compare April 25, 2025 12:40
@sukhwinder33445 sukhwinder33445 force-pushed the use-new-item-list-implementation branch 2 times, most recently from eaebc58 to b0ff808 Compare April 25, 2025 13:13
Phpstan now ignores these error messages, so we can remove these.
@sukhwinder33445 sukhwinder33445 force-pushed the use-new-item-list-implementation branch from 5054d3f to b5cd11e Compare April 25, 2025 14:05
@sukhwinder33445 sukhwinder33445 requested a review from nilmerg April 25, 2025 14:05
Copy link
Copy Markdown
Member

@nilmerg nilmerg left a comment

Choose a reason for hiding this comment

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

LGTM!

@nilmerg nilmerg merged commit 4cd1af7 into main Apr 25, 2025
22 checks passed
@nilmerg nilmerg deleted the use-new-item-list-implementation branch April 25, 2025 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla/signed CLA is signed by all contributors of a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants