Skip to content

FIX / Handle group assignment on uninstall templates - #199

Merged
stonebuzz merged 5 commits into
mainfrom
fix/group-assignment-on-uninstall-template
Aug 28, 2026
Merged

FIX / Handle group assignment on uninstall templates#199
stonebuzz merged 5 commits into
mainfrom
fix/group-assignment-on-uninstall-template

Conversation

@Mary-Clb

@Mary-Clb Mary-Clb commented Apr 8, 2026

Copy link
Copy Markdown
Contributor
  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

  • It fixes !43139
  • Group assignment was not working due to a change on GLPI : the plugin now also supports items using GLPI's assignable group handling, instead of depending only on the legacy field detection.

@Mary-Clb Mary-Clb self-assigned this Apr 8, 2026
@Mary-Clb
Mary-Clb requested review from Rom1-B and stonebuzz April 8, 2026 08:12
Comment thread inc/uninstall.class.php Outdated
@Mary-Clb

Mary-Clb commented Apr 8, 2026

Copy link
Copy Markdown
Contributor Author

Waiting for customer approval

@Rom1-B

Rom1-B commented Apr 13, 2026

Copy link
Copy Markdown
Contributor

Please update CHANGELOG

Comment thread inc/uninstall.class.php
@stonebuzz

Copy link
Copy Markdown
Contributor

@Mary-Clb up =)

@stonebuzz

Copy link
Copy Markdown
Contributor

can you rebase?

@Mary-Clb
Mary-Clb force-pushed the fix/group-assignment-on-uninstall-template branch from d1cc5a5 to 553571e Compare August 28, 2026 13:26
@Mary-Clb

Copy link
Copy Markdown
Contributor Author

@stonebuzz I'm done

@stonebuzz

Copy link
Copy Markdown
Contributor

@Mary-Clb

Waiting for customer approval

@stonebuzz
stonebuzz merged commit 75ea800 into main Aug 28, 2026
3 checks passed
@stonebuzz
stonebuzz deleted the fix/group-assignment-on-uninstall-template branch August 28, 2026 13:48
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