[PR #1793] [CLOSED] TenantID in Alerts #3419

New issue

Closed

opened 2026-03-02 13:58:15 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 13:58:15 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/KelvinTegelaar/CIPP/pull/1793
Author: @Yoshify
Created: 10/9/2023
Status: ❌ Closed

Base: dev ← Head: dev-alerts-pr

📝 Commits (10+)

cd62bde Merge pull request #1457 from KelvinTegelaar/dev
38d806d Update README.md
bd6a41a Merge pull request #1498 from KelvinTegelaar/dev
96c0501 Merge pull request #1526 from KelvinTegelaar/dev
bb32207 Merge pull request #1528 from KelvinTegelaar/dev
06907de Merge pull request #1536 from KelvinTegelaar/dev
a007b94 Replaced old docs link
bc59a45 Merge pull request #1543 from rvdwegen/patch-2
f5304da Merge pull request #1567 from KelvinTegelaar/dev
654525c Merge pull request #1587 from KelvinTegelaar/dev

📊 Changes

7 files changed (+76 additions, -13 deletions)

View changed files

➕ .github/ISSUE_TEMPLATE/CLA.yml (+54 -0)
📝 .github/workflows/Comment_on_Issues.yml (+1 -1)
📝 LICENSE.CustomLicenses (+3 -1)
📝 README.md (+0 -11)
📝 src/store/api/app.js (+2 -0)
📝 src/views/cipp/CIPPSettings.js (+7 -0)
📝 src/views/cipp/Logs.js (+9 -0)

📄 Description

This PR goes hand-in-hand with PR 431 in CIPP-API here

This adds a switch to the Notification settings to optionally include the Tenant ID in alerts:

I've also added the TenantID as a column in the logbook, with handling for null values, falling back on 'None'.

Feedback is appreciated :)

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/KelvinTegelaar/CIPP/pull/1793 **Author:** [@Yoshify](https://github.com/Yoshify) **Created:** 10/9/2023 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `dev-alerts-pr` --- ### 📝 Commits (10+) - [`cd62bde`](https://github.com/KelvinTegelaar/CIPP/commit/cd62bde91b464a9cc2500b333843c0b99c645bf7) Merge pull request #1457 from KelvinTegelaar/dev - [`38d806d`](https://github.com/KelvinTegelaar/CIPP/commit/38d806d2b7ef79f1909f6dc399545a4afebf96d9) Update README.md - [`bd6a41a`](https://github.com/KelvinTegelaar/CIPP/commit/bd6a41a453e59b629ef4c1e51e27845db602fa6e) Merge pull request #1498 from KelvinTegelaar/dev - [`96c0501`](https://github.com/KelvinTegelaar/CIPP/commit/96c05014dd7f464f376970c72d8636a7c5cd6d8c) Merge pull request #1526 from KelvinTegelaar/dev - [`bb32207`](https://github.com/KelvinTegelaar/CIPP/commit/bb322079c11ad25bcce6daf73e5be787d0de541d) Merge pull request #1528 from KelvinTegelaar/dev - [`06907de`](https://github.com/KelvinTegelaar/CIPP/commit/06907de5d66356ea8b403f4a2d8834b4d1b61ef0) Merge pull request #1536 from KelvinTegelaar/dev - [`a007b94`](https://github.com/KelvinTegelaar/CIPP/commit/a007b94d4939eace271d58a51567cf5e68303dcc) Replaced old docs link - [`bc59a45`](https://github.com/KelvinTegelaar/CIPP/commit/bc59a455b6648c81434725bb51d45ade968bc71b) Merge pull request #1543 from rvdwegen/patch-2 - [`f5304da`](https://github.com/KelvinTegelaar/CIPP/commit/f5304da2047aeb22b7d3ea61bea4b826a72c7da1) Merge pull request #1567 from KelvinTegelaar/dev - [`654525c`](https://github.com/KelvinTegelaar/CIPP/commit/654525c1464b91f525e57f3f36215966720e56ff) Merge pull request #1587 from KelvinTegelaar/dev ### 📊 Changes **7 files changed** (+76 additions, -13 deletions) <details> <summary>View changed files</summary> ➕ `.github/ISSUE_TEMPLATE/CLA.yml` (+54 -0) 📝 `.github/workflows/Comment_on_Issues.yml` (+1 -1) 📝 `LICENSE.CustomLicenses` (+3 -1) 📝 `README.md` (+0 -11) 📝 `src/store/api/app.js` (+2 -0) 📝 `src/views/cipp/CIPPSettings.js` (+7 -0) 📝 `src/views/cipp/Logs.js` (+9 -0) </details> ### 📄 Description This PR goes hand-in-hand with PR 431 in CIPP-API [here](https://github.com/KelvinTegelaar/CIPP-API/pull/431) This adds a switch to the Notification settings to optionally include the Tenant ID in alerts: ![image](https://github.com/KelvinTegelaar/CIPP/assets/7742139/9f8861ca-27af-4ac5-8950-5713eb815b4e) I've also added the TenantID as a column in the logbook, with handling for null values, falling back on 'None'. Feedback is appreciated :) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-03-02 13:58:15 +03:00

closed this issue
added the
pull-request
label

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/CIPP#3419

No description provided.

Rows
Columns