Skip to content

feature/glpi-12.0 - #51

Merged
Rom1-B merged 7 commits into
mainfrom
feature/glpi-12.0
Oct 6, 2026
Merged

Rom1-B merged 7 commits into
mainfrom
feature/glpi-12.0

Conversation

@Rom1-B

@Rom1-B Rom1-B commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

GLPI 12 compatibility

@Rom1-B
Rom1-B force-pushed the feature/glpi-12.0 branch 3 times, most recently from 432caab to f34230c Compare August 18, 2026 07:09
@Rom1-B
Rom1-B force-pushed the feature/glpi-12.0 branch 2 times, most recently from 7de2d35 to 143039d Compare August 26, 2026 05:46
@Lainow Lainow self-assigned this Sep 1, 2026
@RomainLvr

RomainLvr commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

GLPI 12 no longer exposes FullCalendar globally (it is now bundled inside the core Vue planning, see glpi-project/glpi#24157), so the reservation question calendar from #56 was broken ("JS lib fullcalendar is not known").

  • Ship FullCalendar v6.1.21 (same version as core, MIT) in public/lib/fullcalendar, lazy-loaded once per page by the widget
  • Port the widget options from v4 to v6 (initialView, headerToolbar, slotMinTime...)
  • Drop Html::requireJs('fullcalendar') and the manual CSS loading (v6 injects its styles)
  • Update e2e selectors to the v6 DOM

@RomainLvr
RomainLvr requested a review from stonebuzz October 6, 2026 13:24

@stonebuzz stonebuzz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@Rom1-B
Rom1-B marked this pull request as ready for review October 6, 2026 14:31
@Rom1-B
Rom1-B merged commit eb7b15f into main Oct 6, 2026
7 checks passed
@Rom1-B
Rom1-B deleted the feature/glpi-12.0 branch October 6, 2026 14:31
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.

4 participants