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

Configure helpdesk home per profile 1/2 #18682

Merged

Conversation

AdrienClairembault
Copy link
Contributor

@AdrienClairembault AdrienClairembault commented Jan 8, 2025

Checklist before requesting a review

  • I have performed a self-review of my code.
  • I have added tests that prove my fix is effective or that my feature works.

Description

Allow the helpdesk's home tiles to be configured by the administrator.
Tiles can be:

  • Reordered
  • Deleted

Two more actions will be added in a second PR to avoid overloading this one:

  • Edit a tile details
  • Add a new tile

Screenshots

Reorder tiles:

reorder_tiles

Delete tiles:

delete_tile

For both actions, there is a toast that confirm the correct execution (it can't be seen in the videos as Glpi's toast are located in the bottom right corner, which was outside the recording area).

@AdrienClairembault AdrienClairembault self-assigned this Jan 8, 2025
@AdrienClairembault AdrienClairembault force-pushed the configure-home-tiles branch 3 times, most recently from 07f072e to 2a3a0ea Compare January 8, 2025 16:16
@AdrienClairembault AdrienClairembault marked this pull request as ready for review January 9, 2025 08:21
src/Glpi/Helpdesk/Tile/TilesManager.php Outdated Show resolved Hide resolved
src/Profile.php Outdated Show resolved Hide resolved
@AdrienClairembault AdrienClairembault merged commit 0f67ad0 into glpi-project:main Jan 16, 2025
9 checks passed
@AdrienClairembault AdrienClairembault deleted the configure-home-tiles branch January 16, 2025 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants