Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Jan 30, 2026

Backport of #57522

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot bot added 3. to review Waiting for reviews pending documentation This pull request needs an associated documentation update feedback-requested labels Jan 30, 2026
@backportbot backportbot bot added this to the Nextcloud 31.0.14 milestone Jan 30, 2026
@printminion-co printminion-co force-pushed the backport/57522/stable31 branch from da07720 to ef8397d Compare January 30, 2026 14:57
@printminion-co printminion-co marked this pull request as ready for review January 30, 2026 15:04
@artonge
Copy link
Contributor

artonge commented Jan 30, 2026

Please amend the PR description if you did fix the backport @printminion-co

@printminion-co printminion-co force-pushed the backport/57522/stable31 branch 2 times, most recently from 99193e4 to 1aebbdb Compare February 2, 2026 08:51
@printminion-co printminion-co force-pushed the backport/57522/stable31 branch 2 times, most recently from 1aebbdb to bb14273 Compare February 2, 2026 09:32
Copy link
Contributor

@susnux susnux left a comment

Choose a reason for hiding this comment

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

We normally do not backport new public API (OCP) 👀

@printminion-co printminion-co force-pushed the backport/57522/stable31 branch from bb14273 to 83d33dd Compare February 3, 2026 08:48
@sorbaugh
Copy link
Contributor

sorbaugh commented Feb 3, 2026

@susnux @come-nc @artonge I think this would be fine since it's not removing or changing existing logic

@provokateurin
Copy link
Member

@printminion-co the last commit still has [skip ci] in the description, so you need to remove that.

@printminion-co printminion-co force-pushed the backport/57522/stable31 branch from 83d33dd to b9f0a7f Compare February 3, 2026 12:25
…ooks

Add InstallationCompletedEvent class in public API (OCP namespace) that
provides installation details: data directory, admin username, and admin
email. Event will be dispatched after successful installation.

Include comprehensive unit tests covering all event scenarios.

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Integrate event dispatching into Setup class:

- Inject IEventDispatcher dependency
- Dispatch InstallationCompletedEvent after successful installation
- Add Setup tests for event integration
- Update composer autoload for new class

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
@printminion-co printminion-co force-pushed the backport/57522/stable31 branch from b9f0a7f to 262dc40 Compare February 3, 2026 12:25
@printminion-co
Copy link
Contributor

@printminion-co the last commit still has [skip ci] in the description, so you need to remove that.

removed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews feedback-requested pending documentation This pull request needs an associated documentation update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants