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

Hook implementation for event object display names #165

Merged
merged 7 commits into from
May 31, 2024

Conversation

raviks789
Copy link
Contributor

@raviks789 raviks789 commented Mar 21, 2024

resolves #132

Blocked by: #155, #166

requires: Icinga/ipl-web#217

@cla-bot cla-bot bot added the cla/signed CLA is signed by all contributors of a PR label Mar 21, 2024
@raviks789 raviks789 force-pushed the feature/event-object-hook-implementation branch from ced4902 to 0fe4280 Compare March 21, 2024 13:53
@raviks789 raviks789 self-assigned this Mar 21, 2024
@raviks789 raviks789 force-pushed the feature/event-object-hook-implementation branch 5 times, most recently from 39e364e to 6081322 Compare March 25, 2024 16:43
@raviks789 raviks789 changed the title WIP: Hook implementation for event object display names Hook implementation for event object display names Mar 25, 2024
@raviks789 raviks789 force-pushed the feature/event-object-hook-implementation branch 2 times, most recently from 4ea63c1 to 26f9a45 Compare March 27, 2024 11:11
@nilmerg nilmerg added the enhancement New feature or improvement label Apr 9, 2024
@nilmerg nilmerg added this to the Beta milestone Apr 9, 2024
@nilmerg
Copy link
Member

nilmerg commented Apr 9, 2024

Please rebase, fix conflicts and mark as ready for review.

@raviks789 raviks789 force-pushed the feature/event-object-hook-implementation branch from 26f9a45 to d9da5ee Compare April 9, 2024 14:42
@raviks789 raviks789 marked this pull request as ready for review April 9, 2024 14:42
@raviks789 raviks789 changed the base branch from main to fix/inefficient-list-views-which-render-object-names-153 April 9, 2024 14:44
@raviks789 raviks789 changed the base branch from fix/inefficient-list-views-which-render-object-names-153 to main April 9, 2024 14:45
@raviks789 raviks789 force-pushed the feature/event-object-hook-implementation branch 8 times, most recently from a41ed3d to 70fc00e Compare April 22, 2024 13:17
@sukhwinder33445 sukhwinder33445 self-requested a review May 17, 2024 07:18
@sukhwinder33445
Copy link
Contributor

Please rebase and resolve the conflicts.

@raviks789 raviks789 force-pushed the feature/event-object-hook-implementation branch from 70fc00e to 59dd1e6 Compare May 17, 2024 09:58
library/Notifications/Hook/EventsObjectsInfoHook.php Outdated Show resolved Hide resolved
library/Notifications/Hook/EventsObjectsInfoHook.php Outdated Show resolved Hide resolved
library/Notifications/Hook/EventsObjectsInfoHook.php Outdated Show resolved Hide resolved
library/Notifications/Hook/EventsObjectsInfoHook.php Outdated Show resolved Hide resolved
library/Notifications/Widget/Detail/EventDetail.php Outdated Show resolved Hide resolved
library/Notifications/Widget/ItemList/EventList.php Outdated Show resolved Hide resolved
library/Notifications/Widget/ItemList/EventList.php Outdated Show resolved Hide resolved
Copy link
Contributor

@sukhwinder33445 sukhwinder33445 left a comment

Choose a reason for hiding this comment

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

LGTM

library/Notifications/Hook/ObjectsRendererHook.php Outdated Show resolved Hide resolved
library/Notifications/Hook/ObjectsRendererHook.php Outdated Show resolved Hide resolved
library/Notifications/Hook/ObjectsRendererHook.php Outdated Show resolved Hide resolved
library/Notifications/Hook/ObjectsRendererHook.php Outdated Show resolved Hide resolved
library/Notifications/Hook/ObjectsRendererHook.php Outdated Show resolved Hide resolved
library/Notifications/Hook/ObjectsRendererHook.php Outdated Show resolved Hide resolved
library/Notifications/Hook/ObjectsRendererHook.php Outdated Show resolved Hide resolved
library/Notifications/Hook/ObjectsRendererHook.php Outdated Show resolved Hide resolved
library/Notifications/Widget/ItemList/IncidentList.php Outdated Show resolved Hide resolved
public/css/common.less Outdated Show resolved Hide resolved
@raviks789 raviks789 force-pushed the feature/event-object-hook-implementation branch 4 times, most recently from 0824684 to 20d556e Compare May 28, 2024 14:58
@raviks789 raviks789 requested a review from nilmerg May 28, 2024 14:59
library/Notifications/Hook/ObjectsRendererHook.php Outdated Show resolved Hide resolved
library/Notifications/Hook/ObjectsRendererHook.php Outdated Show resolved Hide resolved
library/Notifications/Hook/ObjectsRendererHook.php Outdated Show resolved Hide resolved
library/Notifications/Hook/ObjectsRendererHook.php Outdated Show resolved Hide resolved
library/Notifications/Hook/ObjectsRendererHook.php Outdated Show resolved Hide resolved
library/Notifications/Hook/ObjectsRendererHook.php Outdated Show resolved Hide resolved
@raviks789 raviks789 force-pushed the feature/event-object-hook-implementation branch 5 times, most recently from f2e7759 to 35befb7 Compare May 28, 2024 18:45
@raviks789 raviks789 force-pushed the feature/event-object-hook-implementation branch from 35befb7 to 1252985 Compare May 28, 2024 18:46
nilmerg
nilmerg previously approved these changes May 29, 2024
Because even if the hook(source) implementation is present, it is possible that
the it cannot find a result for the given object (object is deleted/changed).
@sukhwinder33445 sukhwinder33445 force-pushed the feature/event-object-hook-implementation branch from 435f166 to 67fee8b Compare May 31, 2024 09:14
@nilmerg nilmerg merged commit 82bfaf7 into main May 31, 2024
22 checks passed
@nilmerg nilmerg deleted the feature/event-object-hook-implementation branch May 31, 2024 09:40
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 enhancement New feature or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement hooks to render objects
3 participants