starred/CIPP

Fork 0

mirror of https://github.com/KelvinTegelaar/CIPP.git synced 2026-04-25 08:16:01 +03:00

[GH-ISSUE #1223] Exclude tenant icon brings up generic Add Excluded Tenants dialog #682

New issue

Closed

opened 2026-03-02 12:44:18 +03:00 by kerem · 3 comments

kerem commented

2026-03-02 12:44:18 +03:00

Owner

Originally created by @joncz on GitHub (Nov 18, 2022).
Original GitHub issue: https://github.com/KelvinTegelaar/CIPP/issues/1223

Description

CIPP->Settings->Tenants

If I click the eyeball icon beside a tenant to exclude them, I expected to receive a dialog that was basically a confirmation popup:

Exclude
[Continue] [Cancel]

Instead, I got the same popup as if I'd clicked the Add Excluded Tenant button at the top right, giving me the full list.

If you need to use that same dialog, I'd expect to see the selected tenant preselected

Environment data

Hosted

Originally created by @joncz on GitHub (Nov 18, 2022). Original GitHub issue: https://github.com/KelvinTegelaar/CIPP/issues/1223 ### Description CIPP->Settings->Tenants If I click the eyeball icon beside a tenant to exclude them, I expected to receive a dialog that was basically a confirmation popup: Exclude <TenantName> [Continue] [Cancel] Instead, I got the same popup as if I'd clicked the Add Excluded Tenant button at the top right, giving me the full list. If you need to use that same dialog, I'd expect to see the selected tenant preselected ![image](https://user-images.githubusercontent.com/9025749/202726178-c575bac4-2e83-4b3b-8f95-5ad378dbd08d.png) ### Environment data ```PowerShell Hosted ```

kerem

2026-03-02 12:44:18 +03:00

closed this issue
added the
bug

unconfirmed-by-user
labels

kerem commented

2026-03-02 12:44:20 +03:00

Author

Owner

@github-actions[bot] commented on GitHub (Nov 18, 2022):

Thank you for creating a bug. Please make sure your bug is indeed a unique case by checking current and past issues, and reading the complete documentation at https://kelvintegelaar.github.io/CIPP
If your bug is a known documentation issue, it will be closed without notice by a contributor. To confirm that this is not a bug found in the documentation, please copy and paste the following comment: "I confirm that I have checked the documentation thoroughly and believe this to be an actual bug.".

Without confirming, your report will be closed in 24 hours. If you'd like this bug to be assigned to you, please comment "I would like to work on this please!".

@github-actions[bot] commented on GitHub (Nov 18, 2022): Thank you for creating a bug. Please make sure your bug is indeed a unique case by checking current and past issues, and reading the complete documentation at https://kelvintegelaar.github.io/CIPP If your bug is a known documentation issue, it will be closed without notice by a contributor. To confirm that this is not a bug found in the documentation, please copy and paste the following comment: "I confirm that I have checked the documentation thoroughly and believe this to be an actual bug.". Without confirming, your report will be closed in 24 hours. If you'd like this bug to be assigned to you, please comment "I would like to work on this please!".

kerem commented

2026-03-02 12:44:20 +03:00

Author

Owner

@KelvinTegelaar commented on GitHub (Nov 18, 2022):

Whoops, oversight. Should indeed be a "Are you sure you want to exclude this tenant?" Fixing.

@KelvinTegelaar commented on GitHub (Nov 18, 2022): Whoops, oversight. Should indeed be a "Are you sure you want to exclude this tenant?" Fixing.

kerem commented

2026-03-02 12:44:20 +03:00

Author

Owner

@KelvinTegelaar commented on GitHub (Nov 23, 2022):

fixed in dev

@KelvinTegelaar commented on GitHub (Nov 23, 2022): fixed in dev

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#682

No description provided.

Rows
Columns