Skip to content

Conversation

@froozeify
Copy link
Member

@froozeify froozeify commented Nov 26, 2025

Checklist before requesting a review

  • I have read the CONTRIBUTING document.
  • I have performed a self-review of my code.
  • This change requires a documentation update. (add: new ITIL actors hooks docdev#187 - To merge once this PR is accepted)
  • I have added tests that prove my fix is effective or that my feature works.

Description

  • Add the ability to change the ITIL Actor icon
  • Gave the possibility to plugin to alter the count of Actors and to add (or alter) actors listing : POST_ITIL_ACTORS_LOAD and POST_ITIL_ACTORS_COUNT

Screenshots

image
  • Hook::POST_ITIL_ACTORS_COUNT called by the section title to defined the number (2)
  • Hook::POST_ITIL_ACTORS_LOAD called to virtually add the whatsapp user (with a custom actor icon to it)

@froozeify froozeify changed the title add: Add 2 hooks for actors ITIL object add: Add 2 hooks for ITIL actors object Nov 26, 2025
@froozeify froozeify changed the title add: Add 2 hooks for ITIL actors object add: Add 2 hooks for ITIL actors Nov 26, 2025
@cedric-anne cedric-anne added this to the 11.0.3 milestone Nov 26, 2025
@froozeify froozeify force-pushed the 11.0/add-itil-actors-hook branch from ee49ea7 to 4ec97ab Compare November 27, 2025 09:43
@froozeify froozeify marked this pull request as ready for review November 27, 2025 11:27
@froozeify froozeify changed the title add: Add 2 hooks for ITIL actors add: creation of 2 hooks for ITIL actors Nov 27, 2025
Copy link
Contributor

@AdrienClairembault AdrienClairembault left a comment

Choose a reason for hiding this comment

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

For extra context, this is needed for the whatapp plugin to create an anonymous ticket with a phone number (like we do for the mail collectors with the emails).

Personally I would prefer creating real users (with no profiles so they can't login) as it seem less hacky and allow all others features to work seamlessly (e.g. searching ticket by requester, which won't work here).

@AdrienClairembault AdrienClairembault removed their request for review December 1, 2025 10:18
@cedric-anne cedric-anne modified the milestones: 11.0.3, 11.0.4 Dec 1, 2025
@froozeify froozeify marked this pull request as draft December 2, 2025 14:47
@cedric-anne cedric-anne modified the milestones: 11.0.4, 11.0.5 Dec 4, 2025
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.

3 participants