Pull Requests - edmozley/freeitsm GitHub Wiki
Every pull request contributed to FreeITSM, what happened to it, and where the review is written up.
Each pull request gets one verdict before anything is done with it:
- Merge - sound as it is. It goes in under the contributor's name.
-
Merge + fix - a sound design with a few fixable issues. It goes in under the contributor's name, then my own commits fix what needed fixing - usually the five complete languages, the in-app help, and anything that landed on
mainafter the branch was cut. - Rewrite - the idea is right but the code is not safe to apply. I write it myself from their design, credit them as co-author, and close the pull request as complete.
Why the reviews are written down: a review page says what was checked, what was changed on top and why, and what a later change must not undo. Read it before changing that code - it is how a fix stays fixed. Bugs found and fixed by a pull request are also listed in Bugs resolved; people who made a substantial contribution are credited in System → Contributors.
Want to contribute? Start with Contributing.
Newest first. "Write-up" is the review page where there is one; otherwise the page that documents what the pull request built.
| Merged | Pull request | What it did | By | Verdict | Write-up |
|---|---|---|---|---|---|
| Nov 2026 | #300 | Software inventory columns say what they count - Installations and Licence seats | Dr4inder | Merge + fix | Software count columns |
| Nov 2026 | #299 | Preference templates can include the two preferences they left out | Dr4inder | Merge + fix | Preference template coverage |
| Nov 2026 | #297 | CMDB relationships name both ends, so they read correctly in every language | Dr4inder | Merge + fix | CMDB relationship wording |
| Nov 2026 | #295 | Hyper-V VMs report their guest OS | Dr4inder | Merge + fix | Hyper-V guest OS |
| Nov 2026 | #291 | System → Maintenance, the home for tools that find data left behind | vsc55 | Merge + fix | System Maintenance |
| Nov 2026 | #290 | Impact and Risk impact told apart on a change | Dr4inder | Merge + fix | Change impact wording |
| Nov 2026 | #289 | Back from a full-screen ticket keeps your layout and your draft | Dr4inder | Merge + fix | Back from a full-screen ticket |
| Nov 2026 | #265 | Organisation chart follow-up - everybody under a manager counted | vsc55 | Merge | Organisation chart |
| Nov 2026 | #258 | Change risk matrix cells line up with their headings | Dr4inder | Merge (cherry-picked) | - |
| Nov 2026 | #256 | German says "Bearbeiter" for the ticket owner | Dr4inder | Merge | - |
| Nov 2026 | #255 | Knowledge article previews decode HTML codes | Dr4inder | Merge | Article previews showed HTML codes |
| Nov 2026 | #253 | Knowledge links can pick an internal article | Dr4inder | Merge | Knowledge |
| Nov 2026 | #252 | Links to Knowledge articles open in all four layouts | Dr4inder | Merge, review kept the folder move narrow | A link to a Knowledge article did not open |
| Nov 2026 | #251 | Clicking a folder while reading an article shows the folder | Dr4inder | Merge (cherry-picked) | Clicking a folder left the open article on screen |
| Nov 2026 | #247 | Empty Procedures fold with the other Work sections | Dr4inder | Merge, one small fix | Empty Procedures fold |
| Nov 2026 | #242 | People: an organisation chart from the reporting lines | vsc55 | Merge (cherry-picked), small fixes | Organisation chart |
| Nov 2026 | #237-#240 | Entra follow-ups: directory groups, protected imported people, automatic import, the script in cron/ | vsc55 | Merge, small fixes | Entra follow-ups |
| Nov 2026 | #236 | An unheld asset goes to its Intune primary user | vsc55 | Merge | Intune primary user |
| Nov 2026 | #235 | Microsoft Entra ID as a people source | vsc55 | Merge + fix | Entra directory sync |
| Oct 2026 | #223 | Ticket table: full width, split view and pager | Dr4inder | Merge + fix | Ticket table split view |
| Oct 2026 | #211-#213 | Ticket workspace, tasks edited inside it, paginated queue and CSV export | Dr4inder | Merge + fix | Ticket workspace |
| Oct 2026 | #198 | Long forms no longer cut off in a task's Work view | Dr4inder | Merge + fix | Long forms in a task's Work view |
| Oct 2026 | #195-#197 | REST API: custom asset fields, change evidence, inventory reports | Dr4inder | Merge + fix | Custom fields, evidence and inventory |
| Oct 2026 | #174 | Analyst provisioning in the REST API, with OIDC claim mapping | Kraleemil | Rewrite - built from the patch and design, credited as co-author | REST API - Analysts |
| Oct 2026 | #167 | Proxmox VE and VMware Cloud Director sync | turbay-a | Merge | Proxmox and Cloud Director - Developer Guide |
| Oct 2026 | #166 | Microsoft Teams and Mattermost channels, and CSAT in the chat | turbay-a | Merge + fix | Teams and Mattermost - Developer Guide |
| Oct 2026 | #164 | Asset reconciliation, tags and labels | srinivasansanthosh | Merge | Asset reconciliation, tags and labels |
| Oct 2026 | #159 | Telegram channel, contact matching, localisation and attachments | turbay-a | Merge + fix | Telegram - Developer Guide |
| Sep 2026 | Branch, issue #155 | SSO: roles given at first sign-in, and profiles kept in step | srinivasansanthosh | Merge + fix - review notes worked through before merging | SSO JIT roles and profile sync |
| Sep 2026 | #141 | The SOP Checklists module and its workflow action | srinivasansanthosh | Merge + fix | Checklists house style, closure gating |
| Sep 2026 | #140 | Background image uploads on every branding screen respect their scope | srinivasansanthosh | Merge | System branding - Developer Guide |
| Aug 2026 | #119 | Separate SMTP login for Basic IMAP mailboxes | Kraleemil | Rewrite - built on main, credited | Basic IMAP mailboxes |
| Jun 2026 | #23 | Google mailbox OAuth set-up | abdulaziz-git | Merge | Gmail mail collection - Developer Guide |
| Closed | Pull request | Why | By |
|---|---|---|---|
| Nov 2026 | #232 | Withdrawn by its author to meet the contribution guidelines; came back as #235-#240 | vsc55 |
| Oct 2026 | #206 | Split into three smaller pull requests, #211-#213 | Dr4inder |
| Oct 2026 | #165 | Replaced by its author with #166 | turbay-a |
| Sep 2026 | #139 | Replaced by its author with #140 | srinivasansanthosh |
- #264 by vsc55 - directory sources and Imported people. Waiting for its author to move Imported people under System → Maintenance (#291).
Adding a review: name the page <Thing>-PR-Review, start it with the line PR [#N](link) by [handle](link) · Verdict: ..., and cover what was checked, what was changed and what a later change must not undo - System Maintenance is a good model. Then add a row at the top of the table above, and a ↳ 🛠️ ... - review (#N) line under its module in the sidebar.