Skip to content

Add initial setup - #495

Draft
sukhwinder33445 wants to merge 3 commits into
mainfrom
init-setup
Draft

Add initial setup#495
sukhwinder33445 wants to merge 3 commits into
mainfrom
init-setup

Conversation

@sukhwinder33445

@sukhwinder33445 sukhwinder33445 commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

This PR enables users to perform the initial setup when activating the module for the first time.

resolves #351

require Icinga/icingaweb2#5524

@sukhwinder33445 sukhwinder33445 self-assigned this Jun 11, 2026
@cla-bot cla-bot Bot added the cla/signed CLA is signed by all contributors of a PR label Jun 11, 2026
@sukhwinder33445
sukhwinder33445 marked this pull request as draft June 11, 2026 15:50
@sukhwinder33445
sukhwinder33445 requested a review from Copilot June 11, 2026 15:50

This comment was marked as resolved.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.

Comment thread library/Notifications/Widget/InitialSetup.php Outdated
Comment thread library/Notifications/Widget/InitialSetup.php Outdated
Comment thread application/controllers/ContactGroupsController.php Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.

Comment thread library/Notifications/Widget/InitialSetup.php
Comment on lines +88 to +96
foreach ($hooks as $hook) {
/** @var SourceIntegrationSetupHook $hook */
$integrations[] = $hook->getIntegration();

if ($hook->isFinished()) {
$this->integrationAdded = true;
break;
}
}
Comment on lines +98 to +106
$container = new HtmlElement('div', Attributes::create(['class' => 'integrations']));
$container->addHtml(new HtmlElement(
'p',
content: Text::create($this->translate(
'You can either create a new Source or add an existing integration from the following:'
))
));
$container->addHtml(...$integrations);
$this->addHtml($container);
Comment thread public/css/common.less Outdated
@sukhwinder33445
sukhwinder33445 force-pushed the init-setup branch 3 times, most recently from 586fb27 to eace49f Compare June 26, 2026 12:03
- Add message where missing
- Add new line in message with link where missing
- Ensure consistency between messages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla/signed CLA is signed by all contributors of a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Event Rule configuration without contacts, contactgroups and schedules

2 participants